[Android][iOS] WebView Navigated event not triggered when using HtmlWebViewSource by Tamilarasan-Paranthaman · Pull Request #28354 · dotnet/maui

Tamilarasan-Paranthaman · 2025-03-12T14:23:22Z

Root Cause of the issue

On Android and iOS, the Navigated event is not triggered when using HtmlWebViewSource without a BaseUrl. This occurs because when BaseUrl is null, a default value is assigned (AssetBaseUrl for Android and NSBundle.MainBundle.BundlePath for iOS). In the Navigated event, execution exits early when these default values are used, preventing the event from being triggered.

Description of Change

I have removed these conditions on both Android and iOS to fix the issue. This check seems unnecessary because the default value is explicitly set while loading the HTML when BaseUrl is null, so there is no need to restrict it in the Navigated event. Additionally, on iOS, the Navigating event was called without any such condition restriction.

Issues Fixed

Fixes #23502
Fixed #21800

Tested the behaviour in the following platforms

iOS
Android
Windows
Mac

Screenshot

Platform	Before Issue Fix	After Issue Fix
Android	Before-Fix-Android.mov	After-Fix-Android.mov
iOS	Before-Fix-iOS.mov	After-Fix-iOS.mov

dotnet-policy-service · 2025-03-12T14:23:43Z

Hey there @Tamilarasan-Paranthaman! Thank you so much for your PR! Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

jsuarezruiz · 2025-03-12T14:46:40Z

/azp run

azure-pipelines · 2025-03-12T14:47:06Z

Azure Pipelines successfully started running 3 pipeline(s).

Copilot

Pull Request Overview

This PR fixes an issue where the WebView Navigated event was not being triggered when using HtmlWebViewSource without a BaseUrl on Android and iOS. The changes remove conditions that prevented the event from firing when default asset URLs were used.

Removed conditional checks for default asset URLs in the Navigated event on Android and iOS.
Added new UI test cases in TestCases.HostApp and automated tests in TestCases.Shared.Tests to verify the fix.
Updated platform-specific code in MauiWebViewClient.cs, WebViewHandler.Android.cs, and MauiWebViewNavigationDelegate.cs to align with the intended behavior.

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
src/Controls/tests/TestCases.HostApp/Issues/Issue23502.cs	Added UI test page demonstrating WebView events.
src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue23502.cs	Added automated test to verify WebView events.
src/Core/src/Platform/Android/MauiWebViewClient.cs	Removed check for AssetBaseUrl in OnPageStarted and OnPageFinished to ensure proper event triggering.
src/Core/src/Handlers/WebView/WebViewHandler.Android.cs	Removed condition checking AssetBaseUrl in NavigatingCanceled.
src/Core/src/Platform/iOS/MauiWebViewNavigationDelegate.cs	Removed early return when URL equals the default asset URL to allow the Navigated event to trigger.

Comments suppressed due to low confidence (2)

src/Core/src/Handlers/WebView/WebViewHandler.Android.cs:146

Ensure that removing the check for AssetBaseUrl in NavigatingCanceled is fully validated so that asset-based URLs are handled appropriately without unintended side effects.

if (VirtualView == null || string.IsNullOrWhiteSpace(url))

src/Core/src/Platform/iOS/MauiWebViewNavigationDelegate.cs:43

The removal of the early return for the default asset URL in iOS is appropriate for resolving the issue; please ensure that this change does not adversely affect valid file-based navigations.

if (url == "file://{NSBundle.MainBundle.BundlePath}/")

Copilot · 2025-03-14T03:34:16Z

src/Core/src/Platform/Android/MauiWebViewClient.cs

 		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)


Removal of the AssetBaseUrl check in OnPageStarted appears intentional for fixing the bug; please ensure that this change does not introduce regressions when handling local asset URLs.

Copilot · 2025-03-14T03:34:17Z

src/Core/src/Platform/Android/MauiWebViewClient.cs

 		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))


Confirm that the removal of the AssetBaseUrl condition in OnPageFinished does not affect scenarios where asset-based URLs might be used, ensuring consistency with the Android behavior.

…ebViewSource (#28354) * WebView navigated event fix * Test sample changes.

…ebViewSource (dotnet#28354) * WebView navigated event fix * Test sample changes.

…ebViewSource (#28354) * WebView navigated event fix * Test sample changes.

…ebViewSource (dotnet#28354) * WebView navigated event fix * Test sample changes.

Tamilarasan-Paranthaman added 2 commits March 5, 2025 17:48

WebView navigated event fix

be31031

Test sample changes.

a86d82c

dotnet-policy-service bot added the community ✨ Community Contribution label Mar 12, 2025

dotnet-policy-service bot added the partner/syncfusion Issues / PR's with Syncfusion collaboration label Mar 12, 2025

jsuarezruiz added area-controls-webview WebView platform/android platform/ios labels Mar 12, 2025

Tamilarasan-Paranthaman marked this pull request as ready for review March 14, 2025 03:33

Copilot AI review requested due to automatic review settings March 14, 2025 03:33

Tamilarasan-Paranthaman requested a review from a team as a code owner March 14, 2025 03:33

Tamilarasan-Paranthaman requested review from rmarinho and tj-devel709 March 14, 2025 03:33

Copilot AI reviewed Mar 14, 2025

View reviewed changes

jsuarezruiz approved these changes Apr 16, 2025

View reviewed changes

PureWeen changed the base branch from main to inflight/current April 18, 2025 09:35

PureWeen merged commit 3a41053 into dotnet:inflight/current Apr 18, 2025
128 of 130 checks passed

PureWeen pushed a commit that referenced this pull request Apr 23, 2025

[Android][iOS] WebView Navigated event not triggered when using HtmlW…

ec5373f

…ebViewSource (#28354) * WebView navigated event fix * Test sample changes.

prakashKannanSf3972 pushed a commit to prakashKannanSf3972/maui that referenced this pull request Apr 24, 2025

[Android][iOS] WebView Navigated event not triggered when using HtmlW…

01dba61

…ebViewSource (dotnet#28354) * WebView navigated event fix * Test sample changes.

NanthiniMahalingam pushed a commit to NanthiniMahalingam/maui that referenced this pull request Apr 24, 2025

[Android][iOS] WebView Navigated event not triggered when using HtmlW…

9e221c6

…ebViewSource (dotnet#28354) * WebView navigated event fix * Test sample changes.

prakashKannanSf3972 pushed a commit to prakashKannanSf3972/maui that referenced this pull request Apr 25, 2025

[Android][iOS] WebView Navigated event not triggered when using HtmlW…

c6135cb

…ebViewSource (dotnet#28354) * WebView navigated event fix * Test sample changes.

prakashKannanSf3972 pushed a commit to prakashKannanSf3972/maui that referenced this pull request Apr 28, 2025

[Android][iOS] WebView Navigated event not triggered when using HtmlW…

a4d97fc

…ebViewSource (dotnet#28354) * WebView navigated event fix * Test sample changes.

github-actions bot pushed a commit that referenced this pull request Apr 28, 2025

[Android][iOS] WebView Navigated event not triggered when using HtmlW…

ea45872

…ebViewSource (#28354) * WebView navigated event fix * Test sample changes.

PureWeen pushed a commit that referenced this pull request May 2, 2025

[Android][iOS] WebView Navigated event not triggered when using HtmlW…

a7c605c

…ebViewSource (#28354) * WebView navigated event fix * Test sample changes.

PureWeen mentioned this pull request May 5, 2025

[2025/04/28] Candidate - In Flight Branch #29158

Merged

SuthiYuvaraj pushed a commit to SuthiYuvaraj/maui that referenced this pull request May 9, 2025

[Android][iOS] WebView Navigated event not triggered when using HtmlW…

f162d85

…ebViewSource (dotnet#28354) * WebView navigated event fix * Test sample changes.

github-actions bot locked and limited conversation to collaborators May 19, 2025

Tamilarasan-Paranthaman deleted the fix-23502 branch September 24, 2025 07:12

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Comments

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

[Android][iOS] WebView Navigated event not triggered when using HtmlWebViewSource#28354
PureWeen merged 2 commits intodotnet:inflight/currentfrom
Tamilarasan-Paranthaman:fix-23502

Tamilarasan-Paranthaman commented Mar 12, 2025 •

edited

Loading

Uh oh!

dotnet-policy-service bot commented Mar 12, 2025

Uh oh!

jsuarezruiz commented Mar 12, 2025

Uh oh!

azure-pipelines bot commented Mar 12, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Mar 14, 2025

Uh oh!

Copilot AI Mar 14, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Comments

Conversation

Tamilarasan-Paranthaman commented Mar 12, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Root Cause of the issue

Description of Change

Issues Fixed

Tested the behaviour in the following platforms

Screenshot

Uh oh!

dotnet-policy-service bot commented Mar 12, 2025

Uh oh!

jsuarezruiz commented Mar 12, 2025

Uh oh!

azure-pipelines bot commented Mar 12, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Mar 14, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 14, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Tamilarasan-Paranthaman commented Mar 12, 2025 •

edited

Loading