Skip to content

[iOS] Make sure always call decisionHandler even if no handler#18934

Merged
PureWeen merged 1 commit intomainfrom
fix-DecidePolicy-ios
Nov 22, 2023
Merged

[iOS] Make sure always call decisionHandler even if no handler#18934
PureWeen merged 1 commit intomainfrom
fix-DecidePolicy-ios

Conversation

@rmarinho
Copy link
Copy Markdown
Member

@rmarinho rmarinho commented Nov 21, 2023

Description of Change

When running DeviceTests on 15.4 to 14.5 we got some random failures like:

Objective-C exception thrown.  Name: NSInternalInconsistencyException Reason: Completion handler passed to -[Microsoft_Maui_Platform_MauiWebViewNavigationDelegate webView:decidePolicyForNavigationAction:decisionHandler:] was not called

This makes sure we always call the decisionHandler

Fixes #18749

@rmarinho rmarinho requested a review from a team as a code owner November 21, 2023 17:14
@rmarinho rmarinho added area-testing Unit tests, device tests testing-flakiness labels Nov 21, 2023
@rmarinho rmarinho requested a review from PureWeen November 21, 2023 17:15
@PureWeen PureWeen enabled auto-merge (squash) November 21, 2023 19:12
@PureWeen PureWeen merged commit b8f99f4 into main Nov 22, 2023
@PureWeen PureWeen deleted the fix-DecidePolicy-ios branch November 22, 2023 12:27
@github-actions github-actions bot locked and limited conversation to collaborators Dec 23, 2023
@samhouts samhouts added the fixed-in-8.0.6 Look for this fix in 8.0.6 SR1! label Aug 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-testing Unit tests, device tests fixed-in-8.0.6 Look for this fix in 8.0.6 SR1! platform/ios testing-flakiness

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Testing] Failing tests on iOS 14.5 - webView:decidePolicyForNavigationAction:decisionHandler:] was not called

3 participants