Updated code docs, Fixed some missing macOS asserts, fix #1447

Author: pichillilorenzo

CHANGELOG.md

@@ -1,3 +1,9 @@
+## 6.0.0-beta.19
+
+- Updated code docs
+- Fixed "Cannot Grant Permission at Android 21" [#1447](https://github.com/pichillilorenzo/flutter_inappwebview/issues/1447)
+- Fixed some missing macOS asserts
+
 ## 6.0.0-beta.18
 
 - Fixed `InAppWebViewSettings` automatic infer if `initialSettings` is `null`
@@ -158,6 +164,10 @@
 - Removed `URLProtectionSpace.iosIsProxy` property
 - `historyUrl` and `baseUrl` of `InAppWebViewInitialData` can be `null`
 
+## 5.7.2+1
+
+- Fixed "Cannot Grant Permission at Android 21" [#1447](https://github.com/pichillilorenzo/flutter_inappwebview/issues/1447)
+
 ## 5.7.2
 
 - Removed Android Hybrid Composition constraint to use the pull-to-refresh feature

android/src/main/java/com/pichillilorenzo/flutter_inappwebview/webview/in_app_webview/InAppWebViewChromeClient.java

@@ -1211,7 +1211,7 @@ private Boolean isArrayEmpty(String[] arr) {
 
   @Override
   public void onPermissionRequest(final PermissionRequest request) {
-    if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.M) {
+    if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) {
       final WebViewChannelDelegate.PermissionRequestCallback callback = new WebViewChannelDelegate.PermissionRequestCallback() {
         @Override
         public boolean nonNullSuccess(@NonNull PermissionResponse response) {

dev_packages/generators/lib/src/util.dart

@@ -34,7 +34,13 @@ abstract class Util {
       final platformName = platform.getField("name")!.toStringValue();
       final note = platform.getField("note")?.toStringValue();
       if (note != null) {
-        platformNoteList.add("///**NOTE for $platformName**: $note");
+        final noteLines = note.split("\n");
+        var platformNote =
+            "///**NOTE for $platformName**: ${noteLines[0].trim()}";
+        for (int i = 1; i < noteLines.length; i++) {
+          platformNote += "\n///${noteLines[i].trim()}";
+        }
+        platformNoteList.add(platformNote);
       }
 
       final apiName = platform.getField("apiName")?.toStringValue();

example/integration_test/chrome_safari_browser/open_and_close.dart

@@ -45,7 +45,7 @@ void openAndClose() {
             activityButton: ActivityButton(
                 templateImage: UIImage(systemName: "sun.max"),
                 extensionIdentifier:
-                    "com.pichillilorenzo.flutter-inappwebview-6-Example.test")));
+                    "com.pichillilorenzo.flutter-inappwebview6-example.test")));
     await chromeSafariBrowser.opened.future;
     expect(chromeSafariBrowser.isOpened(), true);
     expect(() async {

example/ios/Runner.xcodeproj/project.pbxproj

@@ -451,7 +451,7 @@
 				MARKETING_VERSION = 1.0;
 				MTL_ENABLE_DEBUG_INFO = INCLUDE_SOURCE;
 				MTL_FAST_MATH = YES;
-				PRODUCT_BUNDLE_IDENTIFIER = "com.pichillilorenzo.flutter-inappwebview-6-Example.test";
+				PRODUCT_BUNDLE_IDENTIFIER = "com.pichillilorenzo.flutter-inappwebview6-example.test";
 				PRODUCT_NAME = "$(TARGET_NAME)";
 				SKIP_INSTALL = YES;
 				SWIFT_ACTIVE_COMPILATION_CONDITIONS = DEBUG;
@@ -484,7 +484,7 @@
 				LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/Frameworks @executable_path/../../Frameworks";
 				MARKETING_VERSION = 1.0;
 				MTL_FAST_MATH = YES;
-				PRODUCT_BUNDLE_IDENTIFIER = "com.pichillilorenzo.flutter-inappwebview-6-Example.test";
+				PRODUCT_BUNDLE_IDENTIFIER = "com.pichillilorenzo.flutter-inappwebview6-example.test";
 				PRODUCT_NAME = "$(TARGET_NAME)";
 				SKIP_INSTALL = YES;
 				SWIFT_EMIT_LOC_STRINGS = YES;
@@ -627,7 +627,7 @@
 					"$(inherited)",
 					"$(PROJECT_DIR)/Flutter",
 				);
-				PRODUCT_BUNDLE_IDENTIFIER = "com.pichillilorenzo.flutter-inappwebview-6-Example";
+				PRODUCT_BUNDLE_IDENTIFIER = "com.pichillilorenzo.flutter-inappwebview6-example";
 				PRODUCT_NAME = "$(TARGET_NAME)";
 				PROVISIONING_PROFILE_SPECIFIER = "";
 				SWIFT_OBJC_BRIDGING_HEADER = "Runner/Runner-Bridging-Header.h";
@@ -659,7 +659,7 @@
 					"$(inherited)",
 					"$(PROJECT_DIR)/Flutter",
 				);
-				PRODUCT_BUNDLE_IDENTIFIER = "com.pichillilorenzo.flutter-inappwebview-6-Example";
+				PRODUCT_BUNDLE_IDENTIFIER = "com.pichillilorenzo.flutter-inappwebview6-example";
 				PRODUCT_NAME = "$(TARGET_NAME)";
 				PROVISIONING_PROFILE_SPECIFIER = "";
 				SWIFT_OBJC_BRIDGING_HEADER = "Runner/Runner-Bridging-Header.h";

lib/src/chrome_safari_browser/chrome_safari_browser.dart

@@ -208,7 +208,7 @@ class ChromeSafariBrowser {
           // ignore: deprecated_member_use_from_same_package
           ChromeSafariBrowserClassOptions? options,
       ChromeSafariBrowserSettings? settings}) async {
-    if (!kIsWeb && defaultTargetPlatform == TargetPlatform.iOS) {
+    if (Util.isIOS) {
       assert(url != null, 'The specified URL must not be null on iOS.');
       assert(['http', 'https'].contains(url!.scheme),
           'The specified URL has an unsupported scheme. Only HTTP and HTTPS URLs are supported on iOS.');

lib/src/chrome_safari_browser/chrome_safari_browser_settings.dart

@@ -1,6 +1,5 @@
 import 'dart:ui';
 
-import 'package:flutter/foundation.dart';
 import 'package:flutter_inappwebview_internal_annotations/flutter_inappwebview_internal_annotations.dart';
 
 import '../types/activity_button.dart';
@@ -341,10 +340,9 @@ class ChromeSafariBrowserClassOptions {
 
   Map<String, dynamic> toMap() {
     Map<String, dynamic> options = {};
-    if (defaultTargetPlatform == TargetPlatform.android)
+    if (Util.isAndroid)
       options.addAll(this.android?.toMap() ?? {});
-    else if (defaultTargetPlatform == TargetPlatform.iOS)
-      options.addAll(this.ios?.toMap() ?? {});
+    else if (Util.isIOS) options.addAll(this.ios?.toMap() ?? {});
 
     return options;
   }

lib/src/context_menu.dart

@@ -1,7 +1,6 @@
-import 'package:flutter/foundation.dart';
-
 import 'in_app_webview/webview.dart';
 import 'types/main.dart';
+import 'util.dart';
 
 ///Class that represents the WebView context menu. It used by [WebView.contextMenu].
 ///
@@ -89,11 +88,11 @@ class ContextMenuItem {
       @Deprecated("Use id instead") this.iosId,
       required this.title,
       this.action}) {
-    if (defaultTargetPlatform == TargetPlatform.android) {
+    if (Util.isAndroid) {
       // ignore: deprecated_member_use_from_same_package
       this.id = this.id ?? this.androidId;
       assert(this.id is int);
-    } else if (defaultTargetPlatform == TargetPlatform.iOS) {
+    } else if (Util.isIOS) {
       // ignore: deprecated_member_use_from_same_package
       this.id = this.id ?? this.iosId;
     }

lib/src/cookie_manager.dart

@@ -8,6 +8,7 @@ import 'in_app_webview/headless_in_app_webview.dart';
 import 'platform_util.dart';
 
 import 'types/main.dart';
+import 'util.dart';
 import 'web_uri.dart';
 
 ///Class that implements a singleton object (shared instance) which manages the cookies used by WebView instances.
@@ -519,14 +520,13 @@ class CookieManager {
   }
 
   Future<bool> _shouldUseJavascript() async {
-    if (kIsWeb) {
+    if (Util.isWeb) {
       return true;
     }
-    if (defaultTargetPlatform == TargetPlatform.iOS ||
-        defaultTargetPlatform == TargetPlatform.macOS) {
+    if (Util.isIOS || Util.isMacOS) {
       final platformUtil = PlatformUtil.instance();
       final systemVersion = await platformUtil.getSystemVersion();
-      return defaultTargetPlatform == TargetPlatform.iOS
+      return Util.isIOS
           ? systemVersion.compareTo("11") == -1
           : systemVersion.compareTo("10.13") == -1;
     }

lib/src/in_app_browser/in_app_browser.dart

@@ -972,7 +972,7 @@ class InAppBrowser {
   ///[permissionRequest] represents the permission request with an array of resources the web content wants to access
   ///and the origin of the web page which is trying to access the restricted resources.
   ///
-  ///**NOTE for Android**: available only on Android 23+.
+  ///**NOTE for Android**: available only on Android 21+.
   ///
   ///**NOTE for iOS**: available only on iOS 15.0+. The default [PermissionResponse.action] is [PermissionResponseAction.PROMPT].
   ///