Closed Bug 1427614 Opened 2 years ago Closed 2 years ago

It takes about 8 seconds for navigation to start after click a link


(Core :: DOM: Core & HTML, defect, P2)

Windows 10



Tracking Status
firefox-esr52 --- unaffected
firefox57 --- unaffected
firefox58 --- unaffected
firefox59 + fixed


(Reporter: alice0775, Assigned: farre)



(Keywords: perf, regression, reproducible, Whiteboard: [parity-Chrome][parity-Edge])


(1 file)

[Tracking Requested - why for this release]: Performance regression. UX degradation.

It takes about 8 seconds(too long) for navigation to start (The tab loading icon starts animation).
So, The user feels that mouse click failed....

Chrome and Edge works as expected.

Reproducible: always

Steps To Reproduce:
1. Open in a foreground tab
        (I also experienced on several sites)
2. Wait for 3-4 minutes
3. Click one of link "社会" "経済" "国際" "エンタメ" "スポーツ" "話題" "ランキング" "写真" "動画"

Actual Results:
It takes about 8 seconds for navigation to start (The tab loading icon starts animation)

Expected Results:
It should start immediately.

Repression window:

Suspect:Bug 1372951

Your patch seems to cause the performance regression. Could you look into this?
Flags: needinfo?(afarre)
I can confirm that setting dom.min_tracking_timeout_value=4 fixes the problem.
Version: 56 Branch → Trunk
Note that dom.min_tracking_timeout_value is only set to enable foreground throttling on nightly. Still, it is as I feared a point against throttling on the foreground. I'll debug and verify that I'm correct and that there's nothing to do about it. Maybe throttling could be disabled if we can determine that user interaction is the source, but I'm not sure of if even that would be enough.
Flags: needinfo?(afarre)
Assignee: nobody → afarre
Priority: -- → P2
Does that mean the issue will stay in nightly? Or is it going to move to beta for 59?
Flags: needinfo?(afarre)
The issue will not move to beta, and I'm going to fix it for nightly.
Flags: needinfo?(afarre)
Turn off throttling even though it's only turned on for Nightly.
Attachment #8943624 - Flags: review?(bkelly)
Attachment #8943624 - Flags: review?(bkelly) → review+
Keywords: checkin-needed
Pushed by
Turn off foreground throttling of tp timeouts. r=bkelly
Keywords: checkin-needed
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.