[MBL-17170][Parent] Updated WebView dependency (#2243)

refs: MBL-17170 affects: Parent release note: Fixed a crash on web content screens on Android 10. * Updated webview_flutter to 3.0.4 * Fixed dark mode. * Tests. * Cleanup.
instructure · Nov 9, 2023 · df03276 · df03276
1 parent 3dd6c9d
commit df03276
Show file tree

Hide file tree

Showing 32 changed files with 450 additions and 3,549 deletions.
diff --git a/apps/flutter_parent/assets/html/html_wrapper.html b/apps/flutter_parent/assets/html/html_wrapper.html
@@ -26,6 +26,8 @@
         height: auto;
         margin: 0;
         padding: 0;
+        background-color: {BACKGROUND};
+        color: {COLOR};
     }
 
     img {
@@ -65,6 +67,7 @@
 
     a {
         word-wrap: break-word;
+        color: {LINK_COLOR}
     }
 
     .lti_button {
@@ -81,6 +84,10 @@
         font-size: 13px;
         margin: auto;
     }
+
+    a:visited {
+        color: {VISITED_LINK_COLOR}
+    }
 </style>
 <body>
 <div id="content">

diff --git a/apps/flutter_parent/lib/main.dart b/apps/flutter_parent/lib/main.dart
@@ -33,7 +33,6 @@ import 'package:flutter_parent/utils/notification_util.dart';
 import 'package:flutter_parent/utils/old_app_migration.dart';
 import 'package:flutter_parent/utils/remote_config_utils.dart';
 import 'package:flutter_parent/utils/service_locator.dart';
-import 'package:webview_flutter/webview_flutter.dart';
 import 'package:flutter_downloader/flutter_downloader.dart';
 
 void main() async {

diff --git a/apps/flutter_parent/lib/screens/help/terms_of_use_screen.dart b/apps/flutter_parent/lib/screens/help/terms_of_use_screen.dart
@@ -75,10 +75,12 @@ class _TermsOfUseScreenState extends State<TermsOfUseScreen> {
 
             // Content
             return WebView(
-              darkMode: ParentTheme.of(context)?.isWebViewDarkMode,
               onWebViewCreated: (controller) {
-                controller.loadHtml(snapshot.data!.content!, horizontalPadding: 16);
-              },
+                controller.loadHtml(snapshot.data!.content!,
+                      horizontalPadding: 16,
+                      darkMode:
+                          ParentTheme.of(context)?.isWebViewDarkMode ?? false);
+                },
               navigationDelegate: _handleNavigation
             );
           },

diff --git a/apps/flutter_parent/lib/screens/web_login/web_login_screen.dart b/apps/flutter_parent/lib/screens/web_login/web_login_screen.dart
@@ -134,7 +134,6 @@ class _WebLoginScreenState extends State<WebLoginScreen> {
             navigationDelegate: (request) =>
                 _navigate(context, request, verifyResult),
             javascriptMode: JavascriptMode.unrestricted,
-            darkMode: ParentTheme.of(context)?.isWebViewDarkMode,
             userAgent: ApiPrefs.getUserAgent(),
             onPageFinished: (url) => _pageFinished(url, verifyResult),
             onPageStarted: (url) => _pageStarted(url),
@@ -243,16 +242,15 @@ class _WebLoginScreenState extends State<WebLoginScreen> {
   /// Load the authenticated url with any necessary cookies
   void _loadAuthUrl() async {
     _showLoadingState();
-    CookieManager().clearCookies();
+    final cookieManager = CookieManager();
+    cookieManager.clearCookies();
 
     if (widget.loginFlow == LoginFlow.siteAdmin) {
-      await _controller?.setAcceptThirdPartyCookies(true);
       if (_domain.contains('.instructure.com')) {
-        String cookie = 'canvas_sa_delegated=1;domain=.instructure.com;path=/;';
-        await _controller?.setCookie(_domain, cookie);
-        await _controller?.setCookie('.instructure.com', cookie);
+        cookieManager.setCookie(WebViewCookie(name: 'canvas_sa_delegated', value: '1', domain: _domain));
+        cookieManager.setCookie(WebViewCookie(name: 'canvas_sa_delegated', value: '1', domain: '.instructure.com'));
       } else {
-        await _controller?.setCookie(_domain, 'canvas_sa_delegated=1');
+        cookieManager.setCookie(WebViewCookie(name: 'canvas_sa_delegated', value: '1', domain: _domain));
       }
     }
 

diff --git a/apps/flutter_parent/lib/utils/common_widgets/web_view/canvas_web_view.dart b/apps/flutter_parent/lib/utils/common_widgets/web_view/canvas_web_view.dart
@@ -230,14 +230,15 @@ class _ResizingWebViewState extends State<_ResizingWebView> with WidgetsBindingO
     if (_content != widgetContent) {
       _height = widget.initialHeight;
       _content = widgetContent!;
-      _controller?.loadHtml(_content, horizontalPadding: widget.horizontalPadding);
+      _controller?.loadHtml(_content,
+          horizontalPadding: widget.horizontalPadding,
+          darkMode: ParentTheme.of(context)?.isWebViewDarkMode ?? false);
     }
 
     Widget child = WebView(
       javascriptMode: JavascriptMode.unrestricted,
       onPageFinished: _handlePageLoaded,
       onWebViewCreated: _handleWebViewCreated,
-      darkMode: ParentTheme.of(context)?.isWebViewDarkMode == true,
       navigationDelegate: _handleNavigation,
       gestureRecognizers: _webViewGestures(),
       javascriptChannels: _webViewChannels(),
@@ -260,7 +261,10 @@ class _ResizingWebViewState extends State<_ResizingWebView> with WidgetsBindingO
   }
 
   void _handleWebViewCreated(WebViewController webViewController) async {
-    webViewController.loadHtml(_content, baseUrl: ApiPrefs.getDomain(), horizontalPadding: widget.horizontalPadding);
+    webViewController.loadHtml(_content,
+        baseUrl: ApiPrefs.getDomain(),
+        horizontalPadding: widget.horizontalPadding,
+        darkMode: ParentTheme.of(context)?.isWebViewDarkMode ?? false);
     _controller = webViewController;
   }
 

diff --git a/apps/flutter_parent/lib/utils/common_widgets/web_view/simple_web_view_screen.dart b/apps/flutter_parent/lib/utils/common_widgets/web_view/simple_web_view_screen.dart
@@ -48,7 +48,6 @@ class _SimpleWebViewScreenState extends State<SimpleWebViewScreen> {
         body: WebView(
           javascriptMode: JavascriptMode.unrestricted,
           userAgent: ApiPrefs.getUserAgent(),
-          darkMode: ParentTheme.of(context)?.isWebViewDarkMode,
           gestureRecognizers: Set()..add(Factory<WebViewGestureRecognizer>(() => WebViewGestureRecognizer())),
           navigationDelegate: _handleNavigation,
           onWebViewCreated: (controller) {

diff --git a/apps/flutter_parent/lib/utils/web_view_utils.dart b/apps/flutter_parent/lib/utils/web_view_utils.dart
@@ -30,15 +30,20 @@ extension WebViewUtils on WebViewController {
     String? html, {
     String? baseUrl,
     Map<String, String>? headers,
-    double horizontalPadding = 0})
+    double horizontalPadding = 0,
+    bool darkMode = false})
     async {
       String fileText = await rootBundle.loadString('assets/html/html_wrapper.html');
       html = _applyWorkAroundForDoubleSlashesAsUrlSource(html);
       html = _addProtocolToLinks(html);
       html = _checkForMathTags(html);
       html = fileText.replaceAll('{CANVAS_CONTENT}', html);
+      html = html.replaceAll('{BACKGROUND}', darkMode ? '#000000' : '#ffffff');
+      html = html.replaceAll('{COLOR}', darkMode ? '#ffffff' : '#000000');
+      html = html.replaceAll('{LINK_COLOR}', darkMode ? '#1283C4' : '#0374B5');
+      html = html.replaceAll('{VISITED_LINK_COLOR}', darkMode ? '#C74BAF' : '#BF32A4');
       html = html.replaceAll('{PADDING}', horizontalPadding.toString());
-      this.loadData(baseUrl, html, 'text/html', 'utf-8');
+      this.loadHtmlString(html, baseUrl: baseUrl);
     }
 
   /**

diff --git a/apps/flutter_parent/plugins/webview_flutter/LICENSE b/apps/flutter_parent/plugins/webview_flutter/LICENSE
diff --git a/apps/flutter_parent/plugins/webview_flutter/android/build.gradle b/apps/flutter_parent/plugins/webview_flutter/android/build.gradle
diff --git a/apps/flutter_parent/plugins/webview_flutter/android/settings.gradle b/apps/flutter_parent/plugins/webview_flutter/android/settings.gradle
diff --git a/apps/flutter_parent/plugins/webview_flutter/android/src/main/AndroidManifest.xml b/apps/flutter_parent/plugins/webview_flutter/android/src/main/AndroidManifest.xml
diff --git a/...flutter/android/src/main/java/io/flutter/plugins/webviewflutter/DisplayListenerProxy.java b/...flutter/android/src/main/java/io/flutter/plugins/webviewflutter/DisplayListenerProxy.java