flutter · auto-submit · Apr 13, 2023 · Apr 10, 2023 · Apr 10, 2023 · Apr 10, 2023
@@ -1,3 +1,8 @@
+## 3.5.1
+
+* Updates pigeon dev dependency to `9.2.4`.
+* Fixes Android lint warnings.
+
 ## 3.5.0
 
 * Adds support for `PlatformNavigationDelegate.onUrlChange`.

@@ -57,12 +57,12 @@ import io.flutter.plugins.webviewflutter.WebViewFlutterAndroidExternalApi;
 This package uses [pigeon][3] to generate the communication layer between Flutter and the host
 platform (Android). The communication interface is defined in the `pigeons/android_webview.dart`
 file. After editing the communication interface regenerate the communication layer by running
-`flutter pub run pigeon --input pigeons/android_webview.dart`.
+`dart run pigeon --input pigeons/android_webview.dart`.
 
 Besides [pigeon][3] this package also uses [mockito][4] to generate mock objects for testing
 purposes. To generate the mock objects run the following command:
 ```bash
-flutter pub run build_runner build --delete-conflicting-outputs
+dart run build_runner build --delete-conflicting-outputs
 ```
 
 If you would like to contribute to the plugin, check out our [contribution guide][5].

@@ -33,7 +33,6 @@ android {
         checkAllWarnings true
         warningsAsErrors true
         disable 'AndroidGradlePluginVersion', 'InvalidPackage', 'GradleDependency'
-        baseline file("lint-baseline.xml")
     }
 
     dependencies {

@@ -6,10 +6,12 @@
 
 import android.os.Build;
 import android.webkit.CookieManager;
+import androidx.annotation.NonNull;
+import io.flutter.plugins.webviewflutter.GeneratedAndroidWebView.CookieManagerHostApi;
 
-class CookieManagerHostApiImpl implements GeneratedAndroidWebView.CookieManagerHostApi {
+class CookieManagerHostApiImpl implements CookieManagerHostApi {
   @Override
-  public void clearCookies(GeneratedAndroidWebView.Result<Boolean> result) {
+  public void clearCookies(@NonNull GeneratedAndroidWebView.Result<Boolean> result) {
     CookieManager cookieManager = CookieManager.getInstance();
     if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) {
       cookieManager.removeAllCookies(result::success);
@@ -19,7 +21,7 @@ public void clearCookies(GeneratedAndroidWebView.Result<Boolean> result) {
   }
 
   @Override
-  public void setCookie(String url, String value) {
+  public void setCookie(@NonNull String url, @NonNull String value) {
     CookieManager.getInstance().setCookie(url, value);
   }
 

@@ -6,7 +6,6 @@
 
 import static android.hardware.display.DisplayManager.DisplayListener;
 
-import android.annotation.TargetApi;
 import android.hardware.display.DisplayManager;
 import android.os.Build;
 import android.util.Log;
@@ -39,7 +38,6 @@
  * first initialization of a webview within the process the difference between the lists is the
  * webview's display listener.
  */
-@TargetApi(Build.VERSION_CODES.KITKAT)
 class DisplayListenerProxy {
   private static final String TAG = "DisplayListenerProxy";
 

@@ -5,6 +5,7 @@
 package io.flutter.plugins.webviewflutter;
 
 import android.webkit.DownloadListener;
+import androidx.annotation.NonNull;
 import io.flutter.plugin.common.BinaryMessenger;
 import io.flutter.plugins.webviewflutter.GeneratedAndroidWebView.DownloadListenerFlutterApi;
 
@@ -23,20 +24,20 @@ public class DownloadListenerFlutterApiImpl extends DownloadListenerFlutterApi {
    * @param instanceManager maintains instances stored to communicate with Dart objects
    */
   public DownloadListenerFlutterApiImpl(
-      BinaryMessenger binaryMessenger, InstanceManager instanceManager) {
+      @NonNull BinaryMessenger binaryMessenger, @NonNull InstanceManager instanceManager) {
     super(binaryMessenger);
     this.instanceManager = instanceManager;
   }
 
   /** Passes arguments from {@link DownloadListener#onDownloadStart} to Dart. */
   public void onDownloadStart(
-      DownloadListener downloadListener,
-      String url,
-      String userAgent,
-      String contentDisposition,
-      String mimetype,
+      @NonNull DownloadListener downloadListener,
+      @NonNull String url,
+      @NonNull String userAgent,
+      @NonNull String contentDisposition,
+      @NonNull String mimetype,
       long contentLength,
-      Reply<Void> callback) {
+      @NonNull Reply<Void> callback) {
     onDownloadStart(
         getIdentifierForListener(downloadListener),
         url,

@@ -35,10 +35,10 @@ public DownloadListenerImpl(@NonNull DownloadListenerFlutterApiImpl flutterApi)
 
     @Override
     public void onDownloadStart(
-        String url,
-        String userAgent,
-        String contentDisposition,
-        String mimetype,
+        @NonNull String url,
+        @NonNull String userAgent,
+        @NonNull String contentDisposition,
+        @NonNull String mimetype,
         long contentLength) {
       flutterApi.onDownloadStart(
           this, url, userAgent, contentDisposition, mimetype, contentLength, reply -> {});
@@ -53,7 +53,9 @@ public static class DownloadListenerCreator {
      * @param flutterApi handles sending messages to Dart
      * @return the created {@link DownloadListenerImpl}
      */
-    public DownloadListenerImpl createDownloadListener(DownloadListenerFlutterApiImpl flutterApi) {
+    @NonNull
+    public DownloadListenerImpl createDownloadListener(
+        @NonNull DownloadListenerFlutterApiImpl flutterApi) {
       return new DownloadListenerImpl(flutterApi);
     }
   }
@@ -66,16 +68,16 @@ public DownloadListenerImpl createDownloadListener(DownloadListenerFlutterApiImp
    * @param flutterApi handles sending messages to Dart
    */
   public DownloadListenerHostApiImpl(
-      InstanceManager instanceManager,
-      DownloadListenerCreator downloadListenerCreator,
-      DownloadListenerFlutterApiImpl flutterApi) {
+      @NonNull InstanceManager instanceManager,
+      @NonNull DownloadListenerCreator downloadListenerCreator,
+      @NonNull DownloadListenerFlutterApiImpl flutterApi) {
     this.instanceManager = instanceManager;
     this.downloadListenerCreator = downloadListenerCreator;
     this.flutterApi = flutterApi;
   }
 
   @Override
-  public void create(Long instanceId) {
+  public void create(@NonNull Long instanceId) {
     final DownloadListener downloadListener =
         downloadListenerCreator.createDownloadListener(flutterApi);
     instanceManager.addDartCreatedInstance(downloadListener, instanceId);

@@ -6,6 +6,7 @@
 
 import android.os.Build;
 import android.webkit.WebChromeClient;
+import androidx.annotation.NonNull;
 import androidx.annotation.RequiresApi;
 import io.flutter.plugin.common.BinaryMessenger;
 import java.util.Arrays;
@@ -28,7 +29,7 @@ public class FileChooserParamsFlutterApiImpl
    * @param instanceManager maintains instances stored to communicate with Dart objects
    */
   public FileChooserParamsFlutterApiImpl(
-      BinaryMessenger binaryMessenger, InstanceManager instanceManager) {
+      @NonNull BinaryMessenger binaryMessenger, @NonNull InstanceManager instanceManager) {
     super(binaryMessenger);
     this.instanceManager = instanceManager;
   }
@@ -58,7 +59,8 @@ private static GeneratedAndroidWebView.FileChooserModeEnumData toFileChooserEnum
    * Stores the FileChooserParams instance and notifies Dart to create a new FileChooserParams
    * instance that is attached to this one.
    */
-  public void create(WebChromeClient.FileChooserParams instance, Reply<Void> callback) {
+  public void create(
+      @NonNull WebChromeClient.FileChooserParams instance, @NonNull Reply<Void> callback) {
     if (!instanceManager.containsInstance(instance)) {
       create(
           instanceManager.addHostCreatedInstance(instance),

@@ -11,6 +11,7 @@
 import java.io.IOException;
 
 /** Provides access to the assets registered as part of the App bundle. */
+@SuppressWarnings({"deprecation", "DeprecatedIsStillUsed"})
 abstract class FlutterAssetManager {
   final AssetManager assetManager;
 

@@ -5,6 +5,7 @@
 package io.flutter.plugins.webviewflutter;
 
 import android.webkit.WebView;
+import androidx.annotation.NonNull;
 import io.flutter.plugins.webviewflutter.GeneratedAndroidWebView.FlutterAssetManagerHostApi;
 import java.io.IOException;
 import java.util.ArrayList;
@@ -20,12 +21,13 @@ public class FlutterAssetManagerHostApiImpl implements FlutterAssetManagerHostAp
   final FlutterAssetManager flutterAssetManager;
 
   /** Constructs a new instance of {@link FlutterAssetManagerHostApiImpl}. */
-  public FlutterAssetManagerHostApiImpl(FlutterAssetManager flutterAssetManager) {
+  public FlutterAssetManagerHostApiImpl(@NonNull FlutterAssetManager flutterAssetManager) {
     this.flutterAssetManager = flutterAssetManager;
   }
 
+  @NonNull
   @Override
-  public List<String> list(String path) {
+  public List<String> list(@NonNull String path) {
     try {
       String[] paths = flutterAssetManager.list(path);
 
@@ -39,8 +41,9 @@ public List<String> list(String path) {
     }
   }
 
+  @NonNull
   @Override
-  public String getAssetFilePathByName(String name) {
+  public String getAssetFilePathByName(@NonNull String name) {
     return flutterAssetManager.getAssetFilePathByName(name);
   }
 }
@@ -5,6 +5,8 @@
 package io.flutter.plugins.webviewflutter;
 
 import android.content.Context;
+import androidx.annotation.NonNull;
+import androidx.annotation.Nullable;
 import io.flutter.plugin.common.StandardMessageCodec;
 import io.flutter.plugin.platform.PlatformView;
 import io.flutter.plugin.platform.PlatformViewFactory;
@@ -17,9 +19,15 @@ class FlutterWebViewFactory extends PlatformViewFactory {
     this.instanceManager = instanceManager;
   }
 
+  @NonNull
   @Override
-  public PlatformView create(Context context, int id, Object args) {
-    final PlatformView view = (PlatformView) instanceManager.getInstance((Integer) args);
+  public PlatformView create(Context context, int viewId, @Nullable Object args) {
+    final Integer identifier = (Integer) args;
+    if (identifier == null) {
+      throw new IllegalStateException("An identifier is required to retrieve WebView instance.");
+    }
+
+    final PlatformView view = instanceManager.getInstance(identifier);
     if (view == null) {
       throw new IllegalStateException("Unable to find WebView instance: " + args);
     }

@@ -1,7 +1,7 @@
 // Copyright 2013 The Flutter Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
-// Autogenerated from Pigeon (v9.2.3), do not edit directly.
+// Autogenerated from Pigeon (v9.2.4), do not edit directly.
 // See also: https://pub.dev/packages/pigeon
 
 package io.flutter.plugins.webviewflutter;

@@ -7,6 +7,7 @@
 import android.os.Handler;
 import android.os.Looper;
 import android.util.Log;
+import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
 import java.lang.ref.ReferenceQueue;
 import java.lang.ref.WeakReference;
@@ -69,7 +70,8 @@ public interface FinalizationListener {
    * @param finalizationListener the listener for garbage collected weak references.
    * @return a new `InstanceManager`.
    */
-  public static InstanceManager open(FinalizationListener finalizationListener) {
+  @NonNull
+  public static InstanceManager open(@NonNull FinalizationListener finalizationListener) {
     return new InstanceManager(finalizationListener);
   }
 
@@ -113,7 +115,7 @@ public <T> T remove(long identifier) {
    *     null if the manager doesn't contain the value or the manager is closed.
    */
   @Nullable
-  public Long getIdentifierForStrongReference(Object instance) {
+  public Long getIdentifierForStrongReference(@Nullable Object instance) {
     if (assertNotClosed()) {
       return null;
     }
@@ -137,7 +139,7 @@ public Long getIdentifierForStrongReference(Object instance) {
    * @param identifier the identifier to be paired with instance. This value must be >= 0 and
    *     unique.
    */
-  public void addDartCreatedInstance(Object instance, long identifier) {
+  public void addDartCreatedInstance(@NonNull Object instance, long identifier) {
     if (assertNotClosed()) {
       return;
     }
@@ -151,7 +153,7 @@ public void addDartCreatedInstance(Object instance, long identifier) {
    * @return the unique identifier stored with instance. If the manager is closed, returns -1.
    *     Otherwise, returns a value >= 0.
    */
-  public long addHostCreatedInstance(Object instance) {
+  public long addHostCreatedInstance(@NonNull Object instance) {
     if (assertNotClosed()) {
       return INSTANCE_CLOSED;
     }
@@ -193,7 +195,7 @@ public <T> T getInstance(long identifier) {
    * @return whether this manager contains the given `instance`. If the manager is closed, returns
    *     `false`.
    */
-  public boolean containsInstance(Object instance) {
+  public boolean containsInstance(@Nullable Object instance) {
     if (assertNotClosed()) {
       return false;
     }

@@ -21,7 +21,7 @@ public class JavaObjectHostApiImpl implements GeneratedAndroidWebView.JavaObject
    *
    * @param instanceManager maintains instances stored to communicate with Dart objects
    */
-  public JavaObjectHostApiImpl(InstanceManager instanceManager) {
+  public JavaObjectHostApiImpl(@NonNull InstanceManager instanceManager) {
     this.instanceManager = instanceManager;
   }
 

@@ -30,8 +30,8 @@ public class JavaScriptChannel {
    */
   public JavaScriptChannel(
       @NonNull JavaScriptChannelFlutterApiImpl flutterApi,
-      String channelName,
-      Handler platformThreadHandler) {
+      @NonNull String channelName,
+      @NonNull Handler platformThreadHandler) {
     this.flutterApi = flutterApi;
     this.javaScriptChannelName = channelName;
     this.platformThreadHandler = platformThreadHandler;
@@ -40,11 +40,9 @@ public JavaScriptChannel(
   // Suppressing unused warning as this is invoked from JavaScript.
   @SuppressWarnings("unused")
   @JavascriptInterface
-  public void postMessage(final String message) {
+  public void postMessage(@NonNull final String message) {
     final Runnable postMessageRunnable =
-        () -> {
-          flutterApi.postMessage(JavaScriptChannel.this, message, reply -> {});
-        };
+        () -> flutterApi.postMessage(JavaScriptChannel.this, message, reply -> {});
 
     if (platformThreadHandler.getLooper() == Looper.myLooper()) {
       postMessageRunnable.run();

@@ -4,6 +4,7 @@
 
 package io.flutter.plugins.webviewflutter;
 
+import androidx.annotation.NonNull;
 import io.flutter.plugin.common.BinaryMessenger;
 import io.flutter.plugins.webviewflutter.GeneratedAndroidWebView.JavaScriptChannelFlutterApi;
 
@@ -22,14 +23,16 @@ public class JavaScriptChannelFlutterApiImpl extends JavaScriptChannelFlutterApi
    * @param instanceManager Maintains instances stored to communicate with Dart objects.
    */
   public JavaScriptChannelFlutterApiImpl(
-      BinaryMessenger binaryMessenger, InstanceManager instanceManager) {
+      @NonNull BinaryMessenger binaryMessenger, @NonNull InstanceManager instanceManager) {
     super(binaryMessenger);
     this.instanceManager = instanceManager;
   }
 
   /** Passes arguments from {@link JavaScriptChannel#postMessage} to Dart. */
   public void postMessage(
-      JavaScriptChannel javaScriptChannel, String messageArg, Reply<Void> callback) {
+      @NonNull JavaScriptChannel javaScriptChannel,
+      @NonNull String messageArg,
+      @NonNull Reply<Void> callback) {
     super.postMessage(getIdentifierForJavaScriptChannel(javaScriptChannel), messageArg, callback);
   }