Closed
Bug 1370153
Opened 7 years ago
Closed 7 years ago
stylo: Re-enable dom/xul/crashtests/461917-1.xhtml
Categories
(Core :: CSS Parsing and Computation, defect, P4)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla58
People
(Reporter: TYLin, Assigned: xidorn)
References
Details
Attachments
(1 file)
I disabled dom/xul/crashtests/461917-1.xhtml in bug 1290276 because it crashes on try, but I cannot reproduce it locally for now. Let's re-enabled this crashtest for stylo here.
Updated•7 years ago
|
Priority: -- → P1
Comment 1•7 years ago
|
||
Just did a try run with this test re-enabled, it's still crashing, now with this assertion: thread '<unnamed>' panicked at 'Resolving style on element without current styles', /home/worker/workspace/build/src/servo/ports/geckolib/glue.rs:2806 https://treeherder.mozilla.org/#/jobs?repo=try&revision=774847439410cdd32cee715b94b43d38f18014f3&selectedJob=116821222
Comment 2•7 years ago
|
||
As of [1] this is the last failing crashtest on stylo. We should fix it! [1] https://hg.mozilla.org/integration/autoland/rev/91727c7a6f5bb08ea5297e6b82690ece6ca2cd38
Comment 3•7 years ago
|
||
Oddly I still can't reproduce this locally, but I do get a few of these assertions: [Child 1426] WARNING: stylo: requesting a Gecko declaration block?: file /z/moz/d/layout/style/ServoBindings.cpp, line 450
Assignee: nobody → cam
Status: NEW → ASSIGNED
Comment 4•7 years ago
|
||
Well, no luck in reproducing this locally, or on the build machine with logs enabled. It would be nice if there were a way to dynamically turn on RUST_LOG=debug just for a certain test, but it looks like there's no way. I do have a patch to fix the (unrelated) Gecko declaration block issue.
Updated•7 years ago
|
Priority: P1 → --
Comment 5•7 years ago
|
||
I guess it's something funny with how the <tabs> or <wizards> XBL bindings work. Since they can't be used in regular content documents, it's not high priority.
Assignee: cam → nobody
Status: ASSIGNED → NEW
Priority: -- → P4
Assignee | ||
Comment 6•7 years ago
|
||
Doesn't seem to crash anymore: https://treeherder.mozilla.org/#/jobs?repo=try&revision=ace451fcdd16e3024de294aeb6c6d6f824deb5dc We can probably just enable it now.
Comment hidden (mozreview-request) |
Assignee | ||
Comment 8•7 years ago
|
||
mozreview-review |
Comment on attachment 8922715 [details] Bug 1370153 - Re-enable dom/xul/crashtests/461917-1.xhtml. https://reviewboard.mozilla.org/r/193886/#review198958
Attachment #8922715 -
Flags: review+
Pushed by xquan@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b6b2536fa997 Re-enable dom/xul/crashtests/461917-1.xhtml. r=xidorn
Comment 10•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b6b2536fa997
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox58:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
Updated•7 years ago
|
Assignee: nobody → xidorn+moz
status-firefox57:
--- → wontfix
You need to log in
before you can comment on or make changes to this bug.
Description
•