Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Follow up to #22849] Improve URL bar state when opening an App Store link from another app #37533

Open
hffvld opened this issue Apr 12, 2024 · 1 comment
Labels
bug OS/iOS Fixes related to iOS browser functionality QA/Yes

Comments

@hffvld
Copy link
Contributor

hffvld commented Apr 12, 2024

Description:

Follow-up to #22849

Steps to Reproduce

  1. Open Brave > Go to any website, for example Wikipedia.org
  2. Close Brave
  3. Open any other browser (Safari, Chrome, etc.) > Go to www.apple.com
  4. Search for any app, for example Whatsapp
  5. Tap and hold > Share... > Choose Brave
  6. Tap Don't Allow > Observe

OR

  1. Launch Brave > Go to www.apple.com
  2. Search for any app, for example Whatsapp
  3. Close and relaunch Brave
  4. Wait for the transfer to App Store
  5. Return to Brave > Observe

Actual result:

Observed Your connection to this site is not secure on the transition page when opening the App Store link from other app.

2024-04-11_16-23-21.mp4
2024-04-11_17-02-41.mp4

Expected result:

Improve the not secure cert state on the transition page when opening the App Store link from another app

Reproduces how often: [Easily reproduced, Intermittent Issue]

Easily reproduced

Brave Version:

Brave build: 1.67 (2)
BraveCore: 1.67.2 (124.0.6367.29)
  • Can you reproduce this issue with the most recent build from TestFlight? YES
  • Can you reproduce this issue with the previous version of the current build from TestFlight? YES
  • Can you reproduce this issue with the current build from AppStore? YES

Device details:

iPhone 14 / iOS 17.5

Website problems only:

  • Does the issue resolve itself when disabling Brave Shields?
  • Is the issue reproducible on the latest version of Mobile Safari?

Additional Information

@hffvld hffvld added bug QA/Yes OS/iOS Fixes related to iOS browser functionality labels Apr 12, 2024
@kylehickinson kylehickinson changed the title [Follow up to #22849] Improve not secure cert state on transition page when opening App Store link from another app [Follow up to #22849] Improve URL bar state when opening an App Store link from another app Apr 12, 2024
@kylehickinson
Copy link
Collaborator

kylehickinson commented Apr 12, 2024

To clarify this will occur when we cancel a provisional load when there is no NTP to fall back on. Since the web view doesn't have any committed URL we don't have anything to revert back to. We have a few options to solve this:

  • Close the empty tab
  • Clean up the URL bar in empty tab state and leave it in an about:blank state
  • Load an NTP by default in the back/forward history somehow before actually loading external URLs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug OS/iOS Fixes related to iOS browser functionality QA/Yes
Projects
None yet
Development

No branches or pull requests

2 participants