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.

Sign up for GitHub

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

Jump to bottom

[Android] Slow loading FB timeline #31894

Closed
timchilds opened this issue Jul 26, 2023 · 4 comments · Fixed by brave/brave-core#19544
Closed

[Android] Slow loading FB timeline #31894

timchilds opened this issue Jul 26, 2023 · 4 comments · Fixed by brave/brave-core#19544
Assignees
@SergeyZhukovsky
Labels
bug OS/Android Fixes related to Android browser functionality priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass - Android ARM QA/Yes release-notes/include
Milestone
1.56.x - Release #7

Comments

@timchilds
Copy link

timchilds commented Jul 26, 2023 •
edited
Loading

Description

We got a report that the FB timeline takes ~5 seconds to load in Brave Android. The timeline loads almost immediately in Brave Desktop, Brave iOS and other browsers.
https://community.brave.com/t/brave-loads-facebook-really-slowly/498222/2

STR

  1. Open m.facebook.com.
  2. Observe how long it takes for the timeline to load.
@timchilds timchilds added bug priority/P4 Planned work. We expect to get to it "soon". QA/Yes OS/Android Fixes related to Android browser functionality labels Jul 26, 2023
@atuchin-m
Copy link
Contributor

As investigated in the slack thread, SafeBrowsing is the root of the issue. CheckUrl creates a dramatical slowdown for some resources check.

image

@SergeyZhukovsky SergeyZhukovsky self-assigned this Aug 2, 2023
@SergeyZhukovsky SergeyZhukovsky mentioned this issue Aug 2, 2023
Merged
25 tasks
@SergeyZhukovsky
Copy link
Member

SergeyZhukovsky commented Aug 2, 2023 •
edited by fmarier
Loading

the API returns
INTERNAL_ERROR, code: 8 on links like that, I assume because of wss schema
wss://kaios-d.facebook.com

@brave-builds brave-builds added this to the 1.58.x - Nightly milestone Aug 2, 2023
This was referenced Aug 2, 2023
Merged
Merged
@SergeyZhukovsky SergeyZhukovsky added release-notes/include priority/P2 A bad problem. We might uplift this to the next planned release. and removed priority/P4 Planned work. We expect to get to it "soon". labels Aug 2, 2023
@kjozwiak
Copy link
Member

kjozwiak commented Aug 3, 2023

The above requires 1.56.120 or higher for 1.56.x verification 👍

@Uni-verse
Copy link
Contributor

Verified on Samsung Galaxy S21 using version:

Brave	1.56.20 Chromium: 115.0.5790.171 (Official Build) (32-bit) 
Revision	cf9067bf10d8f798c24643029af1d24e275646d6-refs/branch-heads/5790@{#1924}
OS	Android 13; Build/TP1A.220624.014; 33; REL
  • Verified loading facebook.com and timeline is instant

Screenshot 2023-08-03 at 12 38 05 PM

@kjozwiak kjozwiak mentioned this issue Aug 14, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SergeyZhukovsky SergeyZhukovsky

Labels
bug OS/Android Fixes related to Android browser functionality priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass - Android ARM QA/Yes release-notes/include
Projects
None yet
Milestone
1.56.x - Release #7
Development

Successfully merging a pull request may close this issue.

Handles better failure cases on Android Safe Browsing brave/brave-core
6 participants
@kjozwiak @SergeyZhukovsky @Uni-verse @timchilds @brave-builds @atuchin-m