Added PR feedback

Author: mvanbeusekom

packages/webview_flutter/webview_flutter_platform_interface/lib/v4/src/javascript_channel_registry.dart

@@ -31,7 +31,7 @@ class JavaScriptChannelRegistry {
     javaScriptChannel.onMessageReceived(JavaScriptMessage(message: message));
   }
 
-  /// Updates the set of [JavaScriptChannel]s with the new set.
+  /// Replaces the set of [JavaScriptChannel]s with the new set.
   void updateJavaScriptChannelsFromSet(Set<JavaScriptChannel>? channels) {
     this.channels.clear();
     if (channels == null) {

packages/webview_flutter/webview_flutter_platform_interface/lib/v4/src/navigation_callback_delegate.dart

@@ -13,7 +13,7 @@ import 'webview_platform.dart';
 ///
 /// The [WebViewControllerDelegate] is notifying this delegate on events that
 /// happened on the platform's webview. Platform implementations should
-/// implement this class and pass an instance to the[WebViewControllerDelegate].
+/// implement this class and pass an instance to the [WebViewControllerDelegate].
 abstract class NavigationCallbackDelegate extends PlatformInterface {
   /// Creates a new [NavigationCallbackDelegate]
   factory NavigationCallbackDelegate(NavigationCallbackCreationParams params) {

packages/webview_flutter/webview_flutter_platform_interface/lib/v4/src/types/javascript_message.dart

@@ -7,7 +7,7 @@ import 'package:flutter/foundation.dart';
 /// A message that was sent by JavaScript code running in a [WebView].
 ///
 /// Platform specific implementations can add additional fields by extending
-/// this class and provide a factory method that takes the
+/// this class and providing a factory method that takes the
 /// [JavaScriptMessage] as a parameter.
 ///
 /// {@tool sample}

packages/webview_flutter/webview_flutter_platform_interface/lib/v4/src/types/web_resource_error.dart

@@ -115,7 +115,5 @@ class WebResourceError {
   final String description;
 
   /// The type this error can be categorized as.
-  ///
-  /// This will never be `null` on Android, but can be `null` on iOS.
   final WebResourceErrorType? errorType;
 }

packages/webview_flutter/webview_flutter_platform_interface/lib/v4/src/types/webview_widget_creation_params.dart

@@ -50,7 +50,7 @@ class WebViewWidgetCreationParams {
   const WebViewWidgetCreationParams({
     this.key,
     required this.controller,
-    this.gestureRecognizers,
+    this.gestureRecognizers = const <Factory<OneSequenceGestureRecognizer>>{},
   });
 
   /// Controls how one widget replaces another widget in the tree.
@@ -72,8 +72,8 @@ class WebViewWidgetCreationParams {
   /// [ListView] will want to handle vertical drags. The web view will claim
   /// gestures that are recognized by any of the recognizers on this list.
   ///
-  /// When `gestureRecognizers` is empty or null, the web view will only handle
+  /// When `gestureRecognizers` is empty (default), the web view will only handle
   /// pointer events for gestures that were not claimed by any other gesture
   /// recognizer.
-  final Set<Factory<OneSequenceGestureRecognizer>>? gestureRecognizers;
+  final Set<Factory<OneSequenceGestureRecognizer>> gestureRecognizers;
 }

packages/webview_flutter/webview_flutter_platform_interface/lib/v4/src/webview_controller_delegate.dart

@@ -146,7 +146,6 @@ abstract class WebViewControllerDelegate extends PlatformInterface {
   ///	2. [Cache API](https://developers.google.com/web/fundamentals/instant-and-offline/web-storage/cache-api) caches.
   ///    These are not yet supported in iOS WkWebView. Service workers tend to use this cache.
   ///	3. Application cache.
-  ///	4. Local Storage.
   Future<void> clearCache() {
     throw UnimplementedError(
         'clearCache is not implemented on the current platform');

packages/webview_flutter/webview_flutter_platform_interface/lib/v4/src/webview_platform.dart

@@ -38,7 +38,7 @@ abstract class WebViewPlatform extends PlatformInterface {
 
   /// Creates a new [WebViewCookieManagerDelegate].
   ///
-  /// This function should only be called by the app-facing plugin.
+  /// This function should only be called by the app-facing package.
   /// Look at using [WebViewCookieManager] in `webview_flutter` instead.
   WebViewCookieManagerDelegate createCookieManagerDelegate(
     WebViewCookieManagerCreationParams params,
@@ -49,7 +49,7 @@ abstract class WebViewPlatform extends PlatformInterface {
 
   /// Creates a new [NavigationCallbackDelegate].
   ///
-  /// This function should only be called by the app-facing plugin.
+  /// This function should only be called by the app-facing package.
   /// Look at using [NavigationCallbackHandler] in `webview_flutter` instead.
   NavigationCallbackDelegate createNavigationCallbackDelegate(
     NavigationCallbackCreationParams params,
@@ -60,7 +60,7 @@ abstract class WebViewPlatform extends PlatformInterface {
 
   /// Create a new [WebViewControllerDelegate].
   ///
-  /// This function should only be called by the app-facing plugin.
+  /// This function should only be called by the app-facing package.
   /// Look at using [WebViewController] in `webview_flutter` instead.
   WebViewControllerDelegate createWebViewControllerDelegate(
     WebViewControllerCreationParams params,
@@ -71,7 +71,7 @@ abstract class WebViewPlatform extends PlatformInterface {
 
   /// Create a new [WebViewWidgetDelegate].
   ///
-  /// This function should only be called by the app-facing plugin.
+  /// This function should only be called by the app-facing package.
   /// Look at using [WebViewWidget] in `webview_flutter` instead.
   WebViewWidgetDelegate createWebViewWidgetDelegate(
     WebViewWidgetCreationParams params,

packages/webview_flutter/webview_flutter_platform_interface/test/src/v4/javascript_channel_registry_test.dart

@@ -3,7 +3,7 @@
 // found in the LICENSE file.
 
 import 'package:flutter_test/flutter_test.dart';
-import 'package:webview_flutter_platform_interface/v4/src/javaScript_channel_registry.dart';
+import 'package:webview_flutter_platform_interface/v4/src/javascript_channel_registry.dart';
 import 'package:webview_flutter_platform_interface/v4/src/types/types.dart';
 
 void main() {