Closed
Bug 1447875
Opened 7 years ago
Closed 7 years ago
Back out bug 1439875 until we fix bug 1446264
Categories
(Core :: XUL, defect)
Core
XUL
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: emilio, Assigned: emilio)
References
Details
Attachments
(1 file)
Comment hidden (mozreview-request) |
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8961226 [details]
Bug 1447875: Partially back out bug 1439875 until we figure out bug 1446264.
https://reviewboard.mozilla.org/r/230008/#review235696
Attachment #8961226 -
Flags: review?(xidorn+moz) → review+
Pushed by ecoal95@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/becfb5f6388d
Partially back out bug 1439875 until we figure out bug 1446264. r=xidorn
Comment 4•7 years ago
|
||
Backed out for failing browser chrome at browser/base/content/test/performance/browser_windowopen_reflows.js
Push that cased the failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=becfb5f6388d29d7c978147cb3d374211b068027
Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=169602247&repo=autoland&lineNumber=4454
Backout: https://hg.mozilla.org/integration/autoland/rev/f82d56c64966a27cd87ab53025ebc774936b3657
Flags: needinfo?(emilio)
Comment 5•7 years ago
|
||
Bug 1439875 changed browser_windowopen_reflows.js but it is not included in the backout patch.
Maybe the test should await BrowserTestUtils.waitForEvent depending on the "xul.size_window_before_layout" pref?
Assignee | ||
Comment 6•7 years ago
|
||
(In reply to Masatoshi Kimura [:emk] from comment #5)
> Bug 1439875 changed browser_windowopen_reflows.js but it is not included in
> the backout patch.
> Maybe the test should await BrowserTestUtils.waitForEvent depending on the
> "xul.size_window_before_layout" pref?
I suspect that waitForEvent was wallpapering these reflows, since they're exactly the same failures as seen in bug 1447056.
Comment 7•7 years ago
|
||
Bug 1446264 has been fixed now.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
Assignee | ||
Updated•7 years ago
|
Flags: needinfo?(emilio)
You need to log in
before you can comment on or make changes to this bug.
Description
•