Open
Bug 1359263
Opened 6 years ago
Updated 3 months ago
2.17ms uninterruptible reflow at receiveMessage@resource://gre/modules/SelectParentHelper.jsm:214:7
Categories
(Core :: Layout: Form Controls, defect, P4)
Core
Layout: Form Controls
Tracking
()
NEW
Performance Impact | low |
People
(Reporter: rowbot, Unassigned)
References
(Depends on 1 open bug)
Details
(Keywords: perf, perf:frontend, perf:responsiveness, Whiteboard: [ohnoreflow])
Here's the stack: receiveMessage@resource://gre/modules/SelectParentHelper.jsm:214:7
Updated•6 years ago
|
Flags: qe-verify?
Priority: -- → P2
Comment 1•6 years ago
|
||
http://searchfox.org/mozilla-central/rev/baf47b352e873d4516d7656186d6d7c7447d3873/toolkit/modules/SelectParentHelper.jsm#213 // Restore scroll position to what it was prior to the update. scrollBox.scrollTop = scrollTop;
Updated•6 years ago
|
Whiteboard: [ohnoreflow][qf][photon-performance] → [ohnoreflow][qf:p1][photon-performance]
Comment 3•6 years ago
|
||
Do we have any steps to reproduce here? i.e. how do you trigger the reported error, and where do you see it? (Perhaps the browser console, I'm guessing? I'm not familiar with this "X ms uninterruptible reflow" message, offhand.) I'm guessing the STR involve triggering a select dropdown -- but does this happen on a particular page, or any page with a select dropdown, or what? And does it happpen every time, or did you just see this once? (Depending on how complicated the page in question is & how much has changed since the last layout, 2ms may or may not be an unreasonable amount of time for a forced layout flush -- I'm not sure offhand.
Flags: needinfo?(smokey101stair)
Comment 4•6 years ago
|
||
(In reply to Daniel Holbert [:dholbert] (AFK May 3-7, 13-21) from comment #3) > (Depending on how complicated the page in question is & how much has changed > since the last layout, 2ms may or may not be an unreasonable amount of time > for a forced layout flush -- I'm not sure offhand. The "2ms" value is also not very significant without knowing if it's high end of low end hardware. Usually in these bugs the question isn't so much "is the amount of time during the uninterruptible reflow reasonable?", but "can we avoid the uninterrupted reflow in this case?".
Comment 5•6 years ago
|
||
Thanks for the "see-also" linkage - I'll bet this is indeed largely a duplicate of bug 1118086.
Reporter | ||
Comment 6•6 years ago
|
||
Sorry, I don't have STR. I filed this at the end of a browser session and don't know when it occurred since I had the sound disabled. If I find STR, I'll be sure to update this bug.
Flags: needinfo?(smokey101stair)
Reporter | ||
Comment 7•6 years ago
|
||
Sorry Florian, I must have missed your question about my hardware. This was recorded on a machine with an Intel Core i7 930 CPU, 12 GB RAM, nvidia GTX 660 GPU, and SSD. The CPU itself is 7 years old, but likely still falls into the high end I would think. Daniel, the "X ms uninterruptible reflow" message comes from the Oh no! Reflow! addon, which you can find here: https://mikeconley.github.io/ohnoreflow/
Updated•6 years ago
|
Blocks: photon-perf-upforgrabs
Updated•6 years ago
|
No longer blocks: photon-performance-triage
Comment 8•6 years ago
|
||
Trying to make our layout [qf:p1] bug list understandable/actionable/actually-high-priority. For this particular bug, I don't think it makes sense to call this [qf:p1], given that: (a) The reported ~2ms reflow isn't necessarily an unreasonable duration for a reflow, depending on hardware & content (per comment 4) (b) We don't know anything about what page-content triggered this, per comment 6. So we can't really investigate very easily. (c) Based on the JS involved (SelectParentHelper.jsm), it seems likely that whatever was going on here would be addressed by the much-more-painful bug 1118086, which has clear STR and which itself is flagged as [qf:p1] So at this point, I think the right status for this bug is "depends-on/possible-dupe-of bug 1118086" & we should retriage after that bug is fixed.
Depends on: 1118086
Whiteboard: [ohnoreflow][qf:p1][photon-performance] → [ohnoreflow][qf][photon-performance]
Comment 9•6 years ago
|
||
(Sorry, to avoid getting this back into triage, I should've bumped this to [qf:p3]. Fixing that now.)
Whiteboard: [ohnoreflow][qf][photon-performance] → [ohnoreflow][qf:p3][photon-performance]
Updated•6 years ago
|
Flags: qe-verify? → qe-verify-
Updated•6 years ago
|
Priority: P2 → P3
Whiteboard: [ohnoreflow][qf:p3][photon-performance] → [ohnoreflow][qf:p3][reserve-photon-performance]
Updated•6 years ago
|
Priority: P3 → P4
Updated•5 years ago
|
Whiteboard: [ohnoreflow][qf:p3][reserve-photon-performance] → [ohnoreflow][qf:p3][reserve-photon-performance] [fxperf]
Updated•5 years ago
|
Whiteboard: [ohnoreflow][qf:p3][reserve-photon-performance] [fxperf] → [ohnoreflow][qf:p3][reserve-photon-performance] [fxperf:p3]
Updated•5 years ago
|
Whiteboard: [ohnoreflow][qf:p3][reserve-photon-performance] [fxperf:p3] → [ohnoreflow][qf:p3][fxperf:p3]
Updated•4 years ago
|
Whiteboard: [ohnoreflow][qf:p3][fxperf:p3] → [ohnoreflow][qf:p3:responsiveness][fxperf:p3]
Updated•4 years ago
|
No longer blocks: photon-perf-upforgrabs
Updated•1 year ago
|
Performance Impact: --- → P3
Keywords: perf:responsiveness
Whiteboard: [ohnoreflow][qf:p3:responsiveness][fxperf:p3] → [ohnoreflow][fxperf:p3]
Updated•8 months ago
|
Severity: normal → S3
Updated•3 months ago
|
Keywords: perf:frontend
Whiteboard: [ohnoreflow][fxperf:p3] → [ohnoreflow]
You need to log in
before you can comment on or make changes to this bug.
Description
•