[webview_flutter] Add listener for content offset

packages/webview_flutter/webview_flutter_android/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 3.4.3
+
+* Added `setOnContentOffsetChanged` method to the `AndroidWebViewController`.
+
 ## 3.4.2
 
 * Clarifies explanation of endorsement in README.

packages/webview_flutter/webview_flutter_android/android/src/main/java/io/flutter/plugins/webviewflutter/ContentOffsetChangedListener.java

@@ -0,0 +1,9 @@
+// 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.
+
+package io.flutter.plugins.webviewflutter;
+
+public interface ContentOffsetChangedListener {
+  void onContentOffsetChange(int left, int top, int oldLeft, int oldTop);
+}

packages/webview_flutter/webview_flutter_android/android/src/main/java/io/flutter/plugins/webviewflutter/GeneratedAndroidWebView.java

@@ -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.0.4), do not edit directly.
+// Autogenerated from Pigeon (v9.0.6), do not edit directly.
 // See also: https://pub.dev/packages/pigeon
 
 package io.flutter.plugins.webviewflutter;
@@ -706,6 +706,8 @@ void removeJavaScriptChannel(
 
     void setBackgroundColor(@NonNull Long instanceId, @NonNull Long color);
 
+    void enableContentOffsetChangedListener(@NonNull Long instanceId, @NonNull Boolean enabled);
+
     /** The codec used by WebViewHostApi. */
     static MessageCodec<Object> getCodec() {
       return WebViewHostApiCodec.INSTANCE;
@@ -1572,6 +1574,40 @@ public void error(Throwable error) {
           channel.setMessageHandler(null);
         }
       }
+      {
+        BasicMessageChannel<Object> channel =
+            new BasicMessageChannel<>(
+                binaryMessenger,
+                "dev.flutter.pigeon.WebViewHostApi.enableContentOffsetChangedListener",
+                getCodec());
+        if (api != null) {
+          channel.setMessageHandler(
+              (message, reply) -> {
+                ArrayList<Object> wrapped = new ArrayList<Object>();
+                try {
+                  ArrayList<Object> args = (ArrayList<Object>) message;
+                  assert args != null;
+                  Number instanceIdArg = (Number) args.get(0);
+                  if (instanceIdArg == null) {
+                    throw new NullPointerException("instanceIdArg unexpectedly null.");
+                  }
+                  Boolean enabledArg = (Boolean) args.get(1);
+                  if (enabledArg == null) {
+                    throw new NullPointerException("enabledArg unexpectedly null.");
+                  }
+                  api.enableContentOffsetChangedListener(
+                      (instanceIdArg == null) ? null : instanceIdArg.longValue(), enabledArg);
+                  wrapped.add(0, null);
+                } catch (Error | RuntimeException exception) {
+                  ArrayList<Object> wrappedError = wrapError(exception);
+                  wrapped = wrappedError;
+                }
+                reply.reply(wrapped);
+              });
+        } else {
+          channel.setMessageHandler(null);
+        }
+      }
     }
   }
   /**
@@ -1591,6 +1627,7 @@ public WebViewFlutterApi(BinaryMessenger argBinaryMessenger) {
       this.binaryMessenger = argBinaryMessenger;
     }
 
+    /** Public interface for sending reply. */
     public interface Reply<T> {
       void reply(T reply);
     }
@@ -1605,9 +1642,24 @@ public void create(@NonNull Long identifierArg, Reply<Void> callback) {
               binaryMessenger, "dev.flutter.pigeon.WebViewFlutterApi.create", getCodec());
       channel.send(
           new ArrayList<Object>(Collections.singletonList(identifierArg)),
-          channelReply -> {
-            callback.reply(null);
-          });
+          channelReply -> callback.reply(null));
+    }
+
+    public void onScrollPosChange(
+        @NonNull Long webViewInstanceIdArg,
+        @NonNull Long xArg,
+        @NonNull Long yArg,
+        @NonNull Long oldXArg,
+        @NonNull Long oldYArg,
+        Reply<Void> callback) {
+      BasicMessageChannel<Object> channel =
+          new BasicMessageChannel<>(
+              binaryMessenger,
+              "dev.flutter.pigeon.WebViewFlutterApi.onScrollPosChange",
+              getCodec());
+      channel.send(
+          new ArrayList<Object>(Arrays.asList(webViewInstanceIdArg, xArg, yArg, oldXArg, oldYArg)),
+          channelReply -> callback.reply(null));
     }
   }
   /** Generated interface from Pigeon that represents a handler of messages from Flutter. */

packages/webview_flutter/webview_flutter_android/android/src/main/java/io/flutter/plugins/webviewflutter/WebViewExtendedApi.java

@@ -0,0 +1,12 @@
+// 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.
+
+package io.flutter.plugins.webviewflutter;
+
+import androidx.annotation.Nullable;
+/** Define extending APIs for {@link android.webkit.WebView} */
+public interface WebViewExtendedApi {
+  void setContentOffsetChangedListener(
+      @Nullable ContentOffsetChangedListener contentOffsetChangedListener);
+}

packages/webview_flutter/webview_flutter_android/android/src/main/java/io/flutter/plugins/webviewflutter/WebViewFlutterApiImpl.java

@@ -16,11 +16,7 @@
  * <p>This class may handle adding native instances that are attached to a Dart instance or passing
  * arguments of callbacks methods to a Dart instance.
  */
-public class WebViewFlutterApiImpl {
-  // To ease adding additional methods, this value is added prematurely.
-  @SuppressWarnings({"unused", "FieldCanBeLocal"})
-  private final BinaryMessenger binaryMessenger;
-
+public class WebViewFlutterApiImpl extends WebViewFlutterApi {
   private final InstanceManager instanceManager;
   private WebViewFlutterApi api;
 
@@ -30,9 +26,9 @@ public class WebViewFlutterApiImpl {
    * @param binaryMessenger used to communicate with Dart over asynchronous messages
    * @param instanceManager maintains instances stored to communicate with attached Dart objects
    */
-  public WebViewFlutterApiImpl(
+  public WebViewFContentOffsetChangedListener lutterApiImpl(
       @NonNull BinaryMessenger binaryMessenger, @NonNull InstanceManager instanceManager) {
-    this.binaryMessenger = binaryMessenger;
+    super(binaryMessenger);
     this.instanceManager = instanceManager;
     api = new WebViewFlutterApi(binaryMessenger);
   }

packages/webview_flutter/webview_flutter_android/android/src/main/java/io/flutter/plugins/webviewflutter/WebViewFlutterPlugin.java

@@ -92,6 +92,7 @@ private void setUp(
             instanceManager,
             binaryMessenger,
             new WebViewHostApiImpl.WebViewProxy(),
+            new WebViewFlutterApiImpl(binaryMessenger, instanceManager),
             context,
             containerView);
     javaScriptChannelHostApi =

packages/webview_flutter/webview_flutter_android/android/src/main/java/io/flutter/plugins/webviewflutter/WebViewHostApiImpl.java

@@ -16,6 +16,7 @@
 import androidx.annotation.Nullable;
 import io.flutter.plugin.common.BinaryMessenger;
 import io.flutter.plugin.platform.PlatformView;
+import io.flutter.plugins.webviewflutter.GeneratedAndroidWebView.WebViewFlutterApi;
 import io.flutter.plugins.webviewflutter.GeneratedAndroidWebView.WebViewHostApi;
 import java.util.Map;
 import java.util.Objects;
@@ -31,6 +32,7 @@ public class WebViewHostApiImpl implements WebViewHostApi {
   // Only used with WebView using virtual displays.
   @Nullable private final View containerView;
   private final BinaryMessenger binaryMessenger;
+  private final WebViewFlutterApi webViewFlutterApi;
 
   private Context context;
 
@@ -77,9 +79,11 @@ public void setWebContentsDebuggingEnabled(boolean enabled) {
   }
 
   /** Implementation of {@link WebView} that can be used as a Flutter {@link PlatformView}s. */
-  public static class WebViewPlatformView extends WebView implements PlatformView {
+  public static class WebViewPlatformView extends WebView
+      implements PlatformView, WebViewExtendedApi {
     private WebViewClient currentWebViewClient;
     private WebChromeClientHostApiImpl.SecureWebChromeClient currentWebChromeClient;
+    private @Nullable ContentOffsetChangedListener contentOffsetChangedListener;
 
     /**
      * Creates a {@link WebViewPlatformView}.
@@ -129,6 +133,20 @@ public void setWebChromeClient(WebChromeClient client) {
     public WebChromeClient getWebChromeClient() {
       return currentWebChromeClient;
     }
+
+    @Override
+    protected void onScrollChanged(int l, int t, int oldL, int oldT) {
+      super.onScrollChanged(l, t, oldL, oldT);
+      if (contentOffsetChangedListener != null) {
+        contentOffsetChangedListener.onContentOffsetChange(l, t, oldL, oldT);
+      }
+    }
+
+    @Override
+    public void setContentOffsetChangedListener(
+        ContentOffsetChangedListener contentOffsetChangedListener) {
+      this.contentOffsetChangedListener = contentOffsetChangedListener;
+    }
   }
 
   /**
@@ -137,9 +155,10 @@ public WebChromeClient getWebChromeClient() {
    */
   @SuppressLint("ViewConstructor")
   public static class InputAwareWebViewPlatformView extends InputAwareWebView
-      implements PlatformView {
+      implements PlatformView, WebViewExtendedApi {
     private WebViewClient currentWebViewClient;
     private WebChromeClientHostApiImpl.SecureWebChromeClient currentWebChromeClient;
+    private @Nullable ContentOffsetChangedListener contentOffsetChangedListener;
 
     /**
      * Creates a {@link InputAwareWebViewPlatformView}.
@@ -206,6 +225,20 @@ public void setWebChromeClient(WebChromeClient client) {
       currentWebChromeClient = (WebChromeClientHostApiImpl.SecureWebChromeClient) client;
       currentWebChromeClient.setWebViewClient(currentWebViewClient);
     }
+
+    @Override
+    protected void onScrollChanged(int l, int t, int oldl, int oldt) {
+      super.onScrollChanged(l, t, oldl, oldt);
+      if (contentOffsetChangedListener != null) {
+        contentOffsetChangedListener.onContentOffsetChange(l, t, oldl, oldt);
+      }
+    }
+
+    @Override
+    public void setContentOffsetChangedListener(
+        ContentOffsetChangedListener contentOffsetChangedListener) {
+      this.contentOffsetChangedListener = contentOffsetChangedListener;
+    }
   }
 
   /**
@@ -221,11 +254,13 @@ public WebViewHostApiImpl(
       InstanceManager instanceManager,
       BinaryMessenger binaryMessenger,
       WebViewProxy webViewProxy,
+      WebViewFlutterApi webViewFlutterApi,
       Context context,
       @Nullable View containerView) {
     this.instanceManager = instanceManager;
     this.binaryMessenger = binaryMessenger;
     this.webViewProxy = webViewProxy;
+    this.webViewFlutterApi = webViewFlutterApi;
     this.context = context;
     this.containerView = containerView;
   }
@@ -420,6 +455,30 @@ public void setBackgroundColor(Long instanceId, Long color) {
     webView.setBackgroundColor(color.intValue());
   }
 
+  @Override
+  public void enableContentOffsetChangedListener(
+      @NonNull Long instanceId, @NonNull Boolean enabled) {
+    final WebView webView = (WebView) instanceManager.getInstance(instanceId);
+    if (webView instanceof WebViewExtendedApi) {
+      if (enabled) {
+        ((WebViewExtendedApi) webView)
+            .setContentOffsetChangedListener(
+                (left, top, oldLeft, oldTop) -> {
+                  webViewFlutterApi.onScrollPosChange(
+                      instanceId,
+                      (long) left,
+                      (long) top,
+                      (long) oldLeft,
+                      (long) oldTop,
+                      reply -> {});
+                });
+
+      } else {
+        ((WebViewExtendedApi) webView).setContentOffsetChangedListener(null);
+      }
+    }
+  }
+
   /** Maintains instances used to communicate with the corresponding WebView Dart object. */
   public InstanceManager getInstanceManager() {
     return instanceManager;

packages/webview_flutter/webview_flutter_android/android/src/test/java/io/flutter/plugins/webviewflutter/WebViewTest.java

@@ -6,6 +6,7 @@
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertTrue;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.ArgumentMatchers.eq;
@@ -39,6 +40,8 @@ public class WebViewTest {
 
   @Mock WebViewHostApiImpl.WebViewProxy mockWebViewProxy;
 
+  @Mock public GeneratedAndroidWebView.WebViewFlutterApi mockWebViewFlutterApi;
+
   @Mock Context mockContext;
 
   @Mock BinaryMessenger mockBinaryMessenger;
@@ -54,7 +57,12 @@ public void setUp() {
         .thenReturn(mockWebView);
     testHostApiImpl =
         new WebViewHostApiImpl(
-            testInstanceManager, mockBinaryMessenger, mockWebViewProxy, mockContext, null);
+            testInstanceManager,
+            mockBinaryMessenger,
+            mockWebViewProxy,
+            mockWebViewFlutterApi,
+            mockContext,
+            null);
     testHostApiImpl.create(0L, true);
   }
 
@@ -335,4 +343,21 @@ public void flutterApiCreate() {
 
     instanceManager.close();
   }
+
+  @Test
+  public void disableContentOffsetChangedListener() {
+    testHostApiImpl.enableContentOffsetChangedListener(0L, false);
+    verify(mockWebView).setContentOffsetChangedListener(null);
+  }
+
+  @Test
+  public void enableContentOffsetChangedListener() {
+    final ArgumentCaptor<ContentOffsetChangedListener> modeCaptor =
+        ArgumentCaptor.forClass(ContentOffsetChangedListener.class);
+    testHostApiImpl.enableContentOffsetChangedListener(0L, true);
+    verify(mockWebView).setContentOffsetChangedListener(modeCaptor.capture());
+    assertNotNull(modeCaptor.getValue());
+    modeCaptor.getValue().onContentOffsetChange(0, 1, 2, 3);
+    verify(mockWebViewFlutterApi).onScrollPosChange(eq(0L), eq(0L), eq(1L), eq(2L), eq(3L), any());
+  }
 }

packages/webview_flutter/webview_flutter_android/example/lib/main.dart

@@ -135,7 +135,11 @@ Page resource error:
       ))
       ..loadRequest(LoadRequestParams(
         uri: Uri.parse('https://flutter.dev'),
-      ));
+      ))
+      ..setOnContentOffsetChanged((int left, int top, int oldLeft, int oldTop) {
+        debugPrint(
+            'Scroll offset change to left = $left y = $top oldX = $oldLeft oldY = $oldTop');
+      });
   }
 
   @override

packages/webview_flutter/webview_flutter_android/example/pubspec.yaml

@@ -35,3 +35,9 @@ flutter:
     - assets/sample_video.mp4
     - assets/www/index.html
     - assets/www/styles/style.css
+
+
+# FOR TESTING ONLY. DO NOT MERGE.
+dependency_overrides:
+  webview_flutter_platform_interface:
+    path: ../../../webview_flutter/webview_flutter_platform_interface