Closed
Bug 1454532
Opened 7 years ago
Closed 6 years ago
multi second delay between visual page load finish and progress bar animation finish
Categories
(Firefox for Android Graveyard :: General, defect, P2)
Tracking
(firefox-esr52 wontfix, firefox59 wontfix, firefox60 wontfix, firefox61 wontfix, firefox65 wontfix, firefox66 ?, firefox67 ?)
People
(Reporter: kbrosnan, Unassigned)
References
(Blocks 1 open bug)
Details
Using a Galaxy S6 I saw a 7s delay between the end of the page visually painting and the completion of the animation for the progress bar. see bug 1454474 comment 2 for some initial findings.
Comment 1•7 years ago
|
||
Progress bar progress is currently set to 100 % when our onStateChange() listener says that loading has stopped [1][2].
[1] https://dxr.mozilla.org/mozilla-central/rev/6276ec7ebbf33e3484997b189f20fc1511534187/mobile/android/chrome/content/browser.js#4416
[2] https://dxr.mozilla.org/mozilla-central/rev/6276ec7ebbf33e3484997b189f20fc1511534187/mobile/android/base/java/org/mozilla/gecko/Tabs.java#635
Comment 2•7 years ago
|
||
For comparison, Chrome shows a progress bar for about 2 seconds when loading cnn.com on my Moto G5. Firefox shows a progress bar for 10+ seconds even though the visible page content appears to be all loaded and displayed.
status-firefox59:
--- → wontfix
status-firefox60:
--- → affected
status-firefox-esr52:
--- → wontfix
Comment 3•6 years ago
|
||
When tracking protection is turned off this is even more common. There are often scripts running in the background holding the progress bar from going to 100% even though the pages loads visually completed.
Can we change the progress bar to only take into account main content being loaded?
Flags: needinfo?(snorp)
See Also: → 1503119
Eugen has done some work on this previously. There may be some knobs we can turn. I don't think, though, that we really know what "main content" is in all cases.
Flags: needinfo?(snorp) → needinfo?(esawin)
Comment 5•6 years ago
|
||
I think this is at least a P2 since it blocks perf "apples to apples" test work.
Eugen do we need anything from gecko here or can we fix more directly?
Priority: -- → P2
Comment 6•6 years ago
|
||
Note: this is an old pre-ProgressTracker (bug 1437988) report, but it may still apply to some individual sites.
We already have bug 1454477 for the centralized discussion on progress bar differences between other engines and GeckoView.
To make this issue actionable, we need to pick an individual (major) site to address a specific issue since the page load progress characteristics heavily vary between sites.
Flags: needinfo?(esawin)
Comment 7•6 years ago
|
||
Eugen, is this bug still relevant after your new progress bar changes in bug 1531179 land?
status-firefox65:
--- → wontfix
status-firefox66:
--- → ?
status-firefox67:
--- → ?
Flags: needinfo?(esawin)
Comment 8•6 years ago
|
||
This bug hasn't been relevant since bug 1437988, I'm closing it.
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(esawin)
Resolution: --- → INVALID
Assignee | ||
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•