Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Commit 0a0f330

Browse files
authored
Removed outdated deprecation comments (#9087)
In breaking change #4487, a typo was fixed in our Android APIs correcting RequestPermissionResult to RequestPermissionsResult (note the 's' on Permissions) for consistency with the Android SDK. Various tombstone comments were left to help guide developers in the right direction. These comments were slated for removal in March of 2018; at over a year later, we can probably safely remove them.
1 parent a44cbbf commit 0a0f330

File tree

3 files changed

+0
-63
lines changed

3 files changed

+0
-63
lines changed

shell/platform/android/io/flutter/app/FlutterActivityDelegate.java

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -127,15 +127,6 @@ public boolean onRequestPermissionsResult(
127127
return flutterView.getPluginRegistry().onRequestPermissionsResult(requestCode, permissions, grantResults);
128128
}
129129

130-
/*
131-
* Method onRequestPermissionResult(int, String[], int[]) was made
132-
* unavailable on 2018-02-28, following deprecation. This comment is left as
133-
* a temporary tombstone for reference, to be removed on 2018-03-28 (or at
134-
* least four weeks after release of unavailability).
135-
*
136-
* https://github.com/flutter/flutter/wiki/Changelog#typo-fixed-in-flutter-engine-android-api
137-
*/
138-
139130
@Override
140131
public boolean onActivityResult(int requestCode, int resultCode, Intent data) {
141132
return flutterView.getPluginRegistry().onActivityResult(requestCode, resultCode, data);

shell/platform/android/io/flutter/app/FlutterPluginRegistry.java

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -156,15 +156,6 @@ public Registrar publish(Object value) {
156156
return this;
157157
}
158158

159-
/*
160-
* Method addRequestPermissionResultListener(RequestPermissionResultListener)
161-
* was made unavailable on 2018-02-28, following deprecation.
162-
* This comment is left as a temporary tombstone for reference, to be removed
163-
* on 2018-03-28 (or at least four weeks after release of unavailability).
164-
*
165-
* https://github.com/flutter/flutter/wiki/Changelog#typo-fixed-in-flutter-engine-android-api
166-
*/
167-
168159
@Override
169160
public Registrar addRequestPermissionsResultListener(
170161
RequestPermissionsResultListener listener) {
@@ -207,15 +198,6 @@ public boolean onRequestPermissionsResult(int requestCode, String[] permissions,
207198
return false;
208199
}
209200

210-
/*
211-
* Method onRequestPermissionResult(int, String[], int[]) was made
212-
* unavailable on 2018-02-28, following deprecation. This comment is left as
213-
* a temporary tombstone for reference, to be removed on 2018-03-28 (or at
214-
* least four weeks after release of unavailability).
215-
*
216-
* https://github.com/flutter/flutter/wiki/Changelog#typo-fixed-in-flutter-engine-android-api
217-
*/
218-
219201
@Override
220202
public boolean onActivityResult(int requestCode, int resultCode, Intent data) {
221203
for (ActivityResultListener listener : mActivityResultListeners) {

shell/platform/android/io/flutter/plugin/common/PluginRegistry.java

Lines changed: 0 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -163,26 +163,6 @@ interface Registrar {
163163
*/
164164
Registrar addRequestPermissionsResultListener(RequestPermissionsResultListener listener);
165165

166-
/*
167-
* Method addRequestPermissionResultListener(RequestPermissionResultListener listener)
168-
* was made unavailable on 2018-02-28, leaving this comment as a temporary
169-
* tombstone for reference. This comment will be removed on 2018-03-28
170-
* (or at least four weeks after the unavailability is released).
171-
*
172-
* https://github.com/flutter/flutter/wiki/Changelog#typo-fixed-in-flutter-engine-android-api
173-
*
174-
* Adds a callback allowing the plugin to take part in handling incoming
175-
* calls to {@code Activity#onRequestPermissionsResult(int, String[], int[])}
176-
* or {@code android.support.v4.app.ActivityCompat.OnRequestPermissionsResultCallback#onRequestPermissionsResult(int, String[], int[])}.
177-
*
178-
* @param listener a {@link RequestPermissionResultListener} callback.
179-
* @return this {@link Registrar}.
180-
181-
* @deprecated on 2018-01-02 because of misspelling. This method will be made unavailable
182-
* on 2018-02-06 (or at least four weeks after the deprecation is released). Use
183-
* {@link #addRequestPermissionsResultListener(RequestPermissionsResultListener)} instead.
184-
*/
185-
186166
/**
187167
* Adds a callback allowing the plugin to take part in handling incoming
188168
* calls to {@link Activity#onActivityResult(int, int, Intent)}.
@@ -231,22 +211,6 @@ interface RequestPermissionsResultListener {
231211
boolean onRequestPermissionsResult(int requestCode, String[] permissions, int[] grantResults);
232212
}
233213

234-
/*
235-
* interface RequestPermissionResultListener was made unavailable on
236-
* 2018-02-28, leaving this comment as a temporary tombstone for reference.
237-
* This comment will be removed on 2018-03-28 (or at least four weeks after
238-
* the unavailability is released).
239-
*
240-
* https://github.com/flutter/flutter/wiki/Changelog#typo-fixed-in-flutter-engine-android-api
241-
*
242-
* Delegate interface for handling result of permissions requests on
243-
* behalf of the main {@link Activity}.
244-
*
245-
* Deprecated on 2018-01-02 because of misspelling. This interface will be made
246-
* unavailable on 2018-02-06 (or at least four weeks after the deprecation is released).
247-
* Use {@link RequestPermissionsResultListener} instead.
248-
*/
249-
250214
/**
251215
* Delegate interface for handling activity results on behalf of the main
252216
* {@link Activity}.

0 commit comments

Comments
 (0)