From 775191381424a96cea3a2cba42213ffa09ae25d4 Mon Sep 17 00:00:00 2001 From: Chris Bracken Date: Sat, 25 May 2019 14:23:01 -0700 Subject: [PATCH] Removed outdated deprecation comments In breaking change flutter/engine#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. --- .../flutter/app/FlutterActivityDelegate.java | 9 ----- .../io/flutter/app/FlutterPluginRegistry.java | 18 ---------- .../flutter/plugin/common/PluginRegistry.java | 36 ------------------- 3 files changed, 63 deletions(-) diff --git a/shell/platform/android/io/flutter/app/FlutterActivityDelegate.java b/shell/platform/android/io/flutter/app/FlutterActivityDelegate.java index 1ab43db5f792f..24aa3e15889df 100644 --- a/shell/platform/android/io/flutter/app/FlutterActivityDelegate.java +++ b/shell/platform/android/io/flutter/app/FlutterActivityDelegate.java @@ -127,15 +127,6 @@ public boolean onRequestPermissionsResult( return flutterView.getPluginRegistry().onRequestPermissionsResult(requestCode, permissions, grantResults); } - /* - * Method onRequestPermissionResult(int, String[], int[]) was made - * unavailable on 2018-02-28, following deprecation. This comment is left as - * a temporary tombstone for reference, to be removed on 2018-03-28 (or at - * least four weeks after release of unavailability). - * - * https://github.com/flutter/flutter/wiki/Changelog#typo-fixed-in-flutter-engine-android-api - */ - @Override public boolean onActivityResult(int requestCode, int resultCode, Intent data) { return flutterView.getPluginRegistry().onActivityResult(requestCode, resultCode, data); diff --git a/shell/platform/android/io/flutter/app/FlutterPluginRegistry.java b/shell/platform/android/io/flutter/app/FlutterPluginRegistry.java index 13297ae5921ab..072e8ccf8a163 100644 --- a/shell/platform/android/io/flutter/app/FlutterPluginRegistry.java +++ b/shell/platform/android/io/flutter/app/FlutterPluginRegistry.java @@ -156,15 +156,6 @@ public Registrar publish(Object value) { return this; } - /* - * Method addRequestPermissionResultListener(RequestPermissionResultListener) - * was made unavailable on 2018-02-28, following deprecation. - * This comment is left as a temporary tombstone for reference, to be removed - * on 2018-03-28 (or at least four weeks after release of unavailability). - * - * https://github.com/flutter/flutter/wiki/Changelog#typo-fixed-in-flutter-engine-android-api - */ - @Override public Registrar addRequestPermissionsResultListener( RequestPermissionsResultListener listener) { @@ -207,15 +198,6 @@ public boolean onRequestPermissionsResult(int requestCode, String[] permissions, return false; } - /* - * Method onRequestPermissionResult(int, String[], int[]) was made - * unavailable on 2018-02-28, following deprecation. This comment is left as - * a temporary tombstone for reference, to be removed on 2018-03-28 (or at - * least four weeks after release of unavailability). - * - * https://github.com/flutter/flutter/wiki/Changelog#typo-fixed-in-flutter-engine-android-api - */ - @Override public boolean onActivityResult(int requestCode, int resultCode, Intent data) { for (ActivityResultListener listener : mActivityResultListeners) { diff --git a/shell/platform/android/io/flutter/plugin/common/PluginRegistry.java b/shell/platform/android/io/flutter/plugin/common/PluginRegistry.java index 7853982e87928..3b1e4a1855ea5 100644 --- a/shell/platform/android/io/flutter/plugin/common/PluginRegistry.java +++ b/shell/platform/android/io/flutter/plugin/common/PluginRegistry.java @@ -163,26 +163,6 @@ interface Registrar { */ Registrar addRequestPermissionsResultListener(RequestPermissionsResultListener listener); - /* - * Method addRequestPermissionResultListener(RequestPermissionResultListener listener) - * was made unavailable on 2018-02-28, leaving this comment as a temporary - * tombstone for reference. This comment will be removed on 2018-03-28 - * (or at least four weeks after the unavailability is released). - * - * https://github.com/flutter/flutter/wiki/Changelog#typo-fixed-in-flutter-engine-android-api - * - * Adds a callback allowing the plugin to take part in handling incoming - * calls to {@code Activity#onRequestPermissionsResult(int, String[], int[])} - * or {@code android.support.v4.app.ActivityCompat.OnRequestPermissionsResultCallback#onRequestPermissionsResult(int, String[], int[])}. - * - * @param listener a {@link RequestPermissionResultListener} callback. - * @return this {@link Registrar}. - - * @deprecated on 2018-01-02 because of misspelling. This method will be made unavailable - * on 2018-02-06 (or at least four weeks after the deprecation is released). Use - * {@link #addRequestPermissionsResultListener(RequestPermissionsResultListener)} instead. - */ - /** * Adds a callback allowing the plugin to take part in handling incoming * calls to {@link Activity#onActivityResult(int, int, Intent)}. @@ -231,22 +211,6 @@ interface RequestPermissionsResultListener { boolean onRequestPermissionsResult(int requestCode, String[] permissions, int[] grantResults); } - /* - * interface RequestPermissionResultListener was made unavailable on - * 2018-02-28, leaving this comment as a temporary tombstone for reference. - * This comment will be removed on 2018-03-28 (or at least four weeks after - * the unavailability is released). - * - * https://github.com/flutter/flutter/wiki/Changelog#typo-fixed-in-flutter-engine-android-api - * - * Delegate interface for handling result of permissions requests on - * behalf of the main {@link Activity}. - * - * Deprecated on 2018-01-02 because of misspelling. This interface will be made - * unavailable on 2018-02-06 (or at least four weeks after the deprecation is released). - * Use {@link RequestPermissionsResultListener} instead. - */ - /** * Delegate interface for handling activity results on behalf of the main * {@link Activity}.