Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Loading some websites hanging because of a Intermittent error caused by injected scripts #6888

Closed
soner-yuksel opened this issue Feb 6, 2023 · 1 comment · Fixed by #6910 or #6986
Assignees
Labels
bug priority/P2 A bad problem. We might uplift this to the next planned release. privacy QA Pass - iPad QA Pass - iPhone QA/Yes release/blocking
Milestone

Comments

@soner-yuksel
Copy link
Contributor

soner-yuksel commented Feb 6, 2023

Description:

Weird behaviour while loading a webpage where webpage hangs and freezes entire Webview. Please check reproduction steps, videos and additional information section for extra details.

Steps to Reproduce

Steps 1:

  1. Load twitch.tv
  2. Refresh the website
  3. Website hangs - Cant scroll, press cancel or refresh

Steps 2:

  1. Load twitch.tv
  2. After it finishes loading, tap on the URL bar, click on YouTube
  3. Website hangs

Actual result:

Website hangs - Cant scroll, press cancel or refresh

Simulator.Screen.Recording.-.iPhone.14.Pro.-.2023-02-06.at.10.25.40.mp4
Screen.Recording.2023-02-03.at.4.22.42.PM.mov

Expected result:

No infinite loads or hangs on the website- loads properly

Reproduces how often: [Easily reproduced, Intermittent Issue]

Intermittent Issue however If you can’t reproduce try deleting app/resetting iOS sim (Device > Erase all contents & settings) before trying

Brave Version:

  • 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:

N/A however the tests are being done using a iPhone iOS 16.3 and simulator 16.2

Website problems only:

  • did you check with Brave Shields down? Disabling adblock shield (cross-site trackers) fixes the problem
  • did you check in Safari/Firefox (WkWebView-based browsers)? Yes, obviously this doesnt happen in other browsers but at this context it is not applicable.

Additional Information

image (7)

This can be reproduced reliably on simulator / device. Logs in Safari are showing “kek” logs repeat recursively and the number of kekFinity increases continuously and tab loses it capability of refresh and stop.

So according to the tests performed the problem is being suspected in RequestBlocking script however it is not 100 percent.

When the website successfully loads once, this will not happen again because WebKit is probably caching state of current webview.

So after getting the error If Block Cross Site trackers are disabled, the problem might stop.

image (8)

To reproduce as mentioned above Erase All content should be invoked in simulator, while testing is being done on simulator.

This is a very likely the CAUSE and CULPRIT for external links is causing application to freeze in black screen. It has been reproduced while doing tests related the problem.

1

cc @cuba @brave/ios @kjozwiak @Uni-verse

@soner-yuksel soner-yuksel added bug priority/P1 A very extremely bad problem. We might push a hotfix for it. release/blocking labels Feb 6, 2023
@soner-yuksel soner-yuksel added this to the 1.48 milestone Feb 6, 2023
@iccub iccub assigned cuba and unassigned cuba Feb 6, 2023
@iccub iccub added priority/P2 A bad problem. We might uplift this to the next planned release. and removed priority/P1 A very extremely bad problem. We might push a hotfix for it. labels Feb 8, 2023
@Brandon-T Brandon-T reopened this Feb 21, 2023
@Brandon-T Brandon-T self-assigned this Feb 21, 2023
@Uni-verse
Copy link
Contributor

Verified on iPhone 12, iOS 16.1 & iPad (9th Gen), IPadOS 15.6.1 using version 1.48 (23.2.22.16)

  • Reproduced issue by visiting https://twitch.tv using version 1.48 (23.2.21.21)
RPReplay_Final1677104848.MP4
RPReplay_Final1677104733.MP4

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.