Fix webview transformers not working properly on non-web platforms.

Author: BirjuVachhani

lib/src/transformers/node_transformers/passive_embedded_video_transformer.dart

@@ -86,6 +86,11 @@ class _PassiveEmbeddedVideoWidgetState
   @override
   void initState() {
     super.initState();
+    if (!isPlatformSupportedForWebView) {
+      print('Unsupported platform: $defaultTargetPlatform for embedded '
+          'video.');
+      return;
+    }
     final PlatformWebViewControllerCreationParams params;
     if (kIsWeb) {
       // WebView on web only supports loadRequest. Any other method invocation
@@ -177,9 +182,7 @@ class _PassiveEmbeddedVideoWidgetState
 
   Widget buildEmbeddedYoutubeVideo(
       BuildContext context, EmbeddedYoutubeVideoProperties properties) {
-    if (PassiveEmbeddedVideoWidget.supportedPlatforms
-            .contains(Theme.of(context).platform) ||
-        kIsWeb) {
+    if (isPlatformSupportedForWebView) {
       return WebViewWidget(
         controller: _controller,
         key: ValueKey(properties),
@@ -205,9 +208,7 @@ class _PassiveEmbeddedVideoWidgetState
 
   Widget buildEmbeddedVimeoVideo(
       BuildContext context, EmbeddedVimeoVideoProperties properties) {
-    if (PassiveEmbeddedVideoWidget.supportedPlatforms
-            .contains(Theme.of(context).platform) ||
-        kIsWeb) {
+    if (isPlatformSupportedForWebView) {
       return WebViewWidget(
         key: ValueKey(properties),
         controller: _controller,

lib/src/transformers/node_transformers/passive_web_view_transformer.dart

@@ -86,8 +86,12 @@ class _PassiveWebViewWidgetState extends State<PassiveWebViewWidget> {
   @override
   void initState() {
     super.initState();
-    final props = widget.node.properties;
+    if (!isPlatformSupportedForWebView) {
+      print('Unsupported platform: $defaultTargetPlatform for WebView.');
+      return;
+    }
 
+    final props = widget.node.properties;
     if (kIsWeb) {
       // WebView on web only supports loadRequest. Any other method invocation
       // on the controller will result in an exception. Be aware!!
@@ -187,9 +191,7 @@ class _PassiveWebViewWidgetState extends State<PassiveWebViewWidget> {
 
   Widget buildWebpageWebView(
       BuildContext context, WebPageWebViewProperties properties) {
-    if (!PassiveWebViewWidget.supportedPlatforms
-            .contains(Theme.of(context).platform) &&
-        !kIsWeb) {
+    if (!isPlatformSupportedForWebView) {
       return WebViewPreviewWidget(
         icon: Icon(Icons.language_rounded),
         node: widget.node,
@@ -213,9 +215,7 @@ class _PassiveWebViewWidgetState extends State<PassiveWebViewWidget> {
 
   Widget buildGoogleMapsWebView(
       BuildContext context, GoogleMapsWebViewProperties properties) {
-    if (!PassiveWebViewWidget.supportedPlatforms
-            .contains(Theme.of(context).platform) &&
-        !kIsWeb) {
+    if (!isPlatformSupportedForWebView) {
       return WebViewPreviewWidget(
         icon: Icon(Icons.map_outlined),
         node: widget.node,
@@ -230,9 +230,7 @@ class _PassiveWebViewWidgetState extends State<PassiveWebViewWidget> {
 
   Widget buildTwitterWebView(
       BuildContext context, TwitterWebViewProperties properties) {
-    if (!PassiveWebViewWidget.supportedPlatforms
-            .contains(Theme.of(context).platform) &&
-        !kIsWeb) {
+    if (!isPlatformSupportedForWebView) {
       return WebViewPreviewWidget(
         icon: ImageIcon(
             NetworkImage('https://img.icons8.com/color/344/twitter--v2.png')),

lib/src/transformers/transformer_functions.dart

@@ -3,28 +3,11 @@ import 'package:collection/collection.dart';
 import 'package:flutter/foundation.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_svg/flutter_svg.dart';
-import 'package:universal_html/html.dart';
+import 'package:universal_html/html.dart' as html;
 
 import '../utils/extensions.dart';
 import 'transformers.dart';
 
-/// Convenience method that looks up the [BaseNode] of a given [id].
-///
-/// This function uses [nodeDelegate] to look up the node. The node must
-/// be registered before-hand.
-// BaseNode getNodeByID(String id) => nodeDelegate.getNodeByID(id);
-
-/// Convenience method that looks up the [BaseNode] of a given [id].
-///
-/// This function uses [nodeDelegate] to look up the node.
-// bool hasNodeID(String id) => nodeDelegate.hasNodeID(id);
-
-/// Convenience method that gets a transformer from our registry of
-/// passive transformers for the sdk.
-// T getPassiveTransformer<T>() {
-//   return passiveTransformerManager.getTransformer<T>();
-// }
-
 void convertNodeToButtonType(ButtonTypeEnum type, ButtonNode node) {
   switch (type) {
     case ButtonTypeEnum.elevated:
@@ -412,7 +395,7 @@ String getVideoUrl(EmbeddedVideoSource source, String baseUrl) {
   String updatedBaseUrl = baseUrl;
   if (kIsWeb && !kReleaseMode) {
     // This allows to test and debug video player scripts in local environment.
-    var url = Uri.parse(window.location.href);
+    var url = Uri.parse(html.window.location.href);
     updatedBaseUrl = url.origin;
   }
 
@@ -594,3 +577,8 @@ Widget wrapWithScrollable({
     child: child,
   );
 }
+
+bool get isPlatformSupportedForWebView =>
+    kIsWeb ||
+    PassiveEmbeddedVideoWidget.supportedPlatforms
+        .contains(defaultTargetPlatform);

lib/src/transformers/web/web_webview_platform.dart

@@ -1,2 +1,4 @@
+library web_webview_platform;
+
 export 'web_webview_platform_stub.dart'
     if (dart.library.html) 'web_webview_platform_web.dart';

lib/src/transformers/web/web_webview_platform_stub.dart

@@ -1,3 +1,6 @@
 import 'package:webview_flutter/webview_flutter.dart';
 
+/// Stub for [WebViewPlatform] for web. Importing actual [WebWebViewPlatform]
+/// will cause error on non-web platforms. This stub is used to avoid that on
+/// unsupported platforms.
 class WebWebViewPlatform extends WebViewPlatform {}

lib/src/transformers/web/web_webview_platform_web.dart

@@ -1 +1,5 @@
+/// This file is used to import the [WebWebViewPlatform] class from the
+/// `webview_flutter_web` package. This is done to avoid errors on non-web
+/// platforms.
+
 export 'package:webview_flutter_web/webview_flutter_web.dart';