2.61ms uninterruptible reflow at onxblmousedown@chrome://browser/content/tabbrowser.xml:7392:11

RESOLVED FIXED in Firefox 58

Status

()

enhancement
P1
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: marco, Assigned: enndeakin)

Tracking

(Blocks 1 bug)

unspecified
Firefox 58
Points:
---
Dependency tree / graph
Bug Flags:
qe-verify -

Firefox Tracking Flags

(firefox58 fixed)

Details

(Whiteboard: [ohnoreflow][qf:p2][reserve-photon-performance])

Attachments

(1 attachment)

Here's the stack:

onxblmousedown@chrome://browser/content/tabbrowser.xml:7392:11
Component: Untriaged → Tabbed Browser
Flags: qe-verify?
Priority: -- → P2
          this.clientTop; // just using this to flush style updates
http://searchfox.org/mozilla-central/rev/ce5ccb6a8ca803271c946ccb8b43b7e7d8b64e7a/browser/base/content/tabbrowser.xml#7391

This is on the mousedown event handler of the tabbrowser-tab binding, when clicking the currently selected tab.
Blocks: 462289
Whiteboard: [ohnoreflow][qf][photon-performance] → [ohnoreflow][qf:p1][photon-performance]
Flags: qe-verify? → qe-verify-
Priority: P2 → P3
Whiteboard: [ohnoreflow][qf:p1][photon-performance] → [ohnoreflow][qf:p1][reserve-photon-performance]
Whiteboard: [ohnoreflow][qf:p1][reserve-photon-performance] → [ohnoreflow][qf:p2][reserve-photon-performance]
Priority: P3 → P4
I don't see any issue on Linux when I remove this line. Perhaps 743877 made this unnecessary.
Assignee: nobody → enndeakin
Status: NEW → ASSIGNED
Attachment #8910808 - Flags: review?(mconley)
Priority: P4 → P1
Comment on attachment 8910808 [details] [diff] [review]
Don't flush here

Review of attachment 8910808 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks!
Attachment #8910808 - Flags: review?(mconley) → review+
Pushed by neil@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e1f5ab037fb6
remove flush when clicking tab, r=mconley
https://hg.mozilla.org/mozilla-central/rev/e1f5ab037fb6
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 58
You need to log in before you can comment on or make changes to this bug.