brave / brave-ios

Brave iOS Browser
https://brave.com
Mozilla Public License 2.0
1.7k stars 440 forks source link

Intermittent Incorrect URL Bar state and various fixes related with state and url #8635

Closed soner-yuksel closed 10 months ago

soner-yuksel commented 10 months ago

Description:

URL Bar update moved to didFailProvisionalNavigation to support spoofing prevention properly. For cancelation didFailProvisionalNavigation should be used.

This way we can cover both cancelation url update problem and also blank address bar before loading starts.

Issues closed related with this were

https://github.com/brave/brave-ios/issues/8393 https://github.com/brave/brave-ios/issues/8576

Also the async trigger for observe webview key values inside didCommit is removed. In addition verifyTrust is forced to run async and the awaited result is used on updateURL bar.

We are thinking these 2 changes will solve the intermittent problem related with loading urls stuck in insecure state.

In addition this change also should be tested where we show secure icon when insecure website is loaded.

https://github.com/brave/brave-ios/issues/7403

Test Plan

Test all the STRs in these 3 tickets, fresh test all over again.

https://github.com/brave/brave-ios/issues/8393 https://github.com/brave/brave-ios/issues/8576 https://github.com/brave/brave-ios/issues/7403

In addition always keep an eye on if a insecure website indicator shown on a secure website.

Uni-verse commented 10 months ago

Verified on iPhone 12 Pro running iOS 17.1.1 using version 1.62 (24.1.11.20)

Example Example Example Example Example Example
IMG_5568 IMG_5570 IMG_5571 IMG_5572 IMG_5574 IMG_5576