brave / brave-browser

Brave browser for Android, iOS, Linux, macOS, Windows.
https://brave.com
Mozilla Public License 2.0
17.64k stars 2.3k forks source link

[IOS] Use Chromium's web navigation delegates #38667

Open iccub opened 4 months ago

iccub commented 4 months ago

Platforms

iOS

Description

It may help with the security indicator state bugs and allow us to reuse more chromiums code

bsclifton commented 3 months ago

Marking as a P2 - there are a lot of details not captured here... but team has already been discussing how this could be handled (ex: taking an API based approach). We should definitely update this issue as we have more information 😄

The long term goal would be to leverage as much of the Chromium code as we can (similar to Android and Desktop). This issue can be the first big step we take towards this long term goal as it addresses a very important security issue that has been particularly hard to troubleshoot / fix (https://github.com/brave/brave-browser/issues/38671 - also marked as a P2).