Closed
Bug 1342106
Opened 8 years ago
Closed 7 years ago
stylo: Assertion failure: pseudoContext (should have taken the ReconstructFrame path above), at ServoRestyleManager.cpp:250
Categories
(Core :: CSS Parsing and Computation, defect, P3)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: cbook, Unassigned)
References
()
Details
(Keywords: assertion)
found via bughunter topsite tests and reproduced via latest stylo build from tinderbox
Steps to reproduce:
--> Load http://www.infusionsoft.com
---> Assertion failure
WARN:geckoservo::glue: Resolving style on unstyled element with lazy computation forbidden.
Assertion failure: pseudoContext (should have taken the ReconstructFrame path above), at /home/worker/workspace/build/src/layout/base/ServoRestyleManager.cpp:250
Reporter | ||
Comment 1•8 years ago
|
||
another site where bughunter running into this is also siemens.com
Updated•8 years ago
|
Blocks: stylo-bughunter
Comment 2•8 years ago
|
||
After my fix to animation parsing, this appears in layout/reftests/bugs/367220-1.html as well. I have no idea why.
Comment 3•8 years ago
|
||
So this means that, while restyling an element, its ::before pseudo disappeared (there are no rules matching for it), yet we didn't get a ReconstructFrame hint?
Xidorn, can you reproduce with the patch over bug 1344104 applied?
Comment 4•8 years ago
|
||
(In reply to Emilio Cobos Álvarez [:emilio] from comment #3)
> Xidorn, can you reproduce with the patch over bug 1344104 applied?
Yes, I can still reproduce it with that patch.
Comment 5•8 years ago
|
||
I'm fixing up this kind of pseudo stuff in bug 1331047 (which is taking forever to finish, but slowly making progress). If that's the only test that fails with your animation changes, feel free to disable the test, and file a bug dependent on bug 1331047.
Updated•8 years ago
|
No longer blocks: stylo-nightly
Component: Layout → CSS Parsing and Computation
Depends on: 1331047
Priority: -- → P3
Summary: Assertion failure: pseudoContext (should have taken the ReconstructFrame path above), at ServoRestyleManager.cpp:250 → stylo: Assertion failure: pseudoContext (should have taken the ReconstructFrame path above), at ServoRestyleManager.cpp:250
Comment 6•7 years ago
|
||
This seems to work now.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•