[Android][iOS] WebView Navigated event not triggered when using HtmlWebViewSource (#28354)

Tamilarasan-Paranthaman · PureWeen · commit ea45872932d6 · 2025-04-28T15:08:19.000Z
* WebView navigated event fix

* Test sample changes.
diff --git a/src/Controls/tests/TestCases.HostApp/Issues/Issue23502.cs b/src/Controls/tests/TestCases.HostApp/Issues/Issue23502.cs
@@ -0,0 +1,49 @@
+﻿namespace Maui.Controls.Sample.Issues;
+
+[Issue(IssueTracker.Github, 23502, "WebView Navigated event is not triggered", PlatformAffected.Android, isInternetRequired: true)]
+public partial class Issue23502 : ContentPage
+{
+	public Issue23502()
+	{
+
+		var verticalStackLayout = new VerticalStackLayout();
+		verticalStackLayout.Spacing = 20;
+		var webview = new WebView()
+		{
+			HeightRequest = 300,
+			AutomationId = "webView",
+			Source = new HtmlWebViewSource
+			{
+				Html = @"<HTML><BODY><H1>.NET MAUI</H1><P>Welcome to WebView.</P></BODY></HTML>"
+			},
+		};
+
+		var label1 = new Label
+		{
+			Text = "WebView Navigating event is not triggered",
+			AutomationId = "navigatingLabel"
+		};
+
+		var label = new Label
+		{
+			Text = "WebView Navigated event is not triggered",
+			AutomationId = "navigatedLabel"
+		};
+
+		webview.Navigating += (s, e) =>
+		{
+			label1.Text = "Navigating event is triggered";
+		};
+
+		webview.Navigated += (s, e) =>
+		{
+			label.Text = "Navigated event is triggered";
+		};
+
+		verticalStackLayout.Add(label1);
+		verticalStackLayout.Add(label);
+		verticalStackLayout.Add(webview);
+
+		Content = verticalStackLayout;
+	}
+}
diff --git a/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue23502.cs b/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue23502.cs
@@ -0,0 +1,27 @@
+﻿using NUnit.Framework;
+using UITest.Appium;
+using UITest.Core;
+
+namespace Microsoft.Maui.TestCases.Tests.Issues
+{
+	public class Issue23502 : _IssuesUITest
+	{
+		public override string Issue => "WebView Navigated event is not triggered";
+
+		public Issue23502(TestDevice device)
+		: base(device)
+		{ }
+
+		[Test]
+		[Category(UITestCategories.WebView)]
+		public void VerifyWebViewNavigatedEvent()
+		{
+			VerifyInternetConnectivity();
+			var navigatingLabel = App.WaitForElement("navigatingLabel");
+			var navigatedLabel = App.WaitForElement("navigatedLabel");
+
+			Assert.That(navigatingLabel.GetText(), Is.EqualTo("Navigating event is triggered"));
+			Assert.That(navigatedLabel.GetText(), Is.EqualTo("Navigated event is triggered"));
+		}
+	}
+}
diff --git a/src/Core/src/Handlers/WebView/WebViewHandler.Android.cs b/src/Core/src/Handlers/WebView/WebViewHandler.Android.cs
@@ -146,9 +146,6 @@ protected internal bool NavigatingCanceled(string? url)
 			if (VirtualView == null || string.IsNullOrWhiteSpace(url))
 				return true;
 
-			if (url == AssetBaseUrl)
-				return false;
-
 			SyncPlatformCookies(url);
 			bool cancel = VirtualView.Navigating(CurrentNavigationEvent, url);
 
diff --git a/src/Core/src/Platform/Android/MauiWebViewClient.cs b/src/Core/src/Platform/Android/MauiWebViewClient.cs
@@ -22,7 +22,7 @@ public override bool ShouldOverrideUrlLoading(WebView? view, IWebResourceRequest
 
 		public override void OnPageStarted(WebView? view, string? url, Bitmap? favicon)
 		{
-			if (!_handler.TryGetTarget(out var handler) || handler.VirtualView == null || url == WebViewHandler.AssetBaseUrl)
+			if (!_handler.TryGetTarget(out var handler) || handler.VirtualView == null)
 				return;
 
 			if (!string.IsNullOrWhiteSpace(url))
@@ -51,7 +51,7 @@ public override void OnPageStarted(WebView? view, string? url, Bitmap? favicon)
 
 		public override void OnPageFinished(WebView? view, string? url)
 		{
-			if (!_handler.TryGetTarget(out var handler) || handler.VirtualView == null || string.IsNullOrWhiteSpace(url) || url == WebViewHandler.AssetBaseUrl)
+			if (!_handler.TryGetTarget(out var handler) || handler.VirtualView == null || string.IsNullOrWhiteSpace(url))
 				return;
 
 			bool navigate = _navigationResult != WebNavigationResult.Failure || !GetValidUrl(url).Equals(_lastUrlNavigatedCancel, StringComparison.OrdinalIgnoreCase);
diff --git a/src/Core/src/Platform/iOS/MauiWebViewNavigationDelegate.cs b/src/Core/src/Platform/iOS/MauiWebViewNavigationDelegate.cs
@@ -43,9 +43,6 @@ public void DidFinishNavigation(WKWebView webView, WKNavigation navigation)
 
 			var url = GetCurrentUrl();
 
-			if (url == $"file://{NSBundle.MainBundle.BundlePath}/")
-				return;
-
 			virtualView.Navigated(_lastEvent, url, WebNavigationResult.Success);
 
 			// ProcessNavigatedAsync calls UpdateCanGoBackForward

Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ public override bool ShouldOverrideUrlLoading(WebView? view, IWebResourceRequest`
`22`	`22`
`23`	`23`	`public override void OnPageStarted(WebView? view, string? url, Bitmap? favicon)`
`24`	`24`	`{`
`25`		`- if (!_handler.TryGetTarget(out var handler) \|\| handler.VirtualView == null \|\| url == WebViewHandler.AssetBaseUrl)`
	`25`	`+ if (!_handler.TryGetTarget(out var handler) \|\| handler.VirtualView == null)`
`26`	`26`	`return;`
`27`	`27`
`28`	`28`	`if (!string.IsNullOrWhiteSpace(url))`
`@@ -51,7 +51,7 @@ public override void OnPageStarted(WebView? view, string? url, Bitmap? favicon)`
`51`	`51`
`52`	`52`	`public override void OnPageFinished(WebView? view, string? url)`
`53`	`53`	`{`
`54`		`- if (!_handler.TryGetTarget(out var handler) \|\| handler.VirtualView == null \|\| string.IsNullOrWhiteSpace(url) \|\| url == WebViewHandler.AssetBaseUrl)`
	`54`	`+ if (!_handler.TryGetTarget(out var handler) \|\| handler.VirtualView == null \|\| string.IsNullOrWhiteSpace(url))`
`55`	`55`	`return;`
`56`	`56`
`57`	`57`	`bool navigate = _navigationResult != WebNavigationResult.Failure \|\| !GetValidUrl(url).Equals(_lastUrlNavigatedCancel, StringComparison.OrdinalIgnoreCase);`