stylo: Assertion failure: array->Count() == 2 (eCSSUnit_Calc_Plus should have a 2-length array), at nsCSSValue.cpp:873
RESOLVED
FIXED
in Firefox 55
Status
()
People
(Reporter: cbook, Assigned: hiro)
Tracking
(Blocks: 1 bug, {assertion})
Firefox Tracking Flags
(firefox52 unaffected, firefox-esr52 unaffected, firefox53 unaffected, firefox54 disabled, firefox55 fixed)
Details
(URL)
Attachments
(4 attachments)
Created attachment 8840748 [details] bughunter stack found via topsite bughunter tests on http://www.disqus.com and reproduced with latest stylo-debug build from taskcluster Steps to reproduce: -> Load http://www.disqus.com ---> Assertion failure after ~ 5 to 10 seconds on my fedora vm Assertion failure: array->Count() == 2 (eCSSUnit_Calc_Plus should have a 2-length array), at /home/worker/workspace/build/src/layout/style/nsCSSValue.cpp:873
(Reporter) | ||
Comment 1•2 years ago
|
||
note, this seem to happen on a lot of urls according to bughunter
status-firefox54: --- → affected
(Assignee) | ||
Comment 2•2 years ago
|
||
This is animation bug.
Component: Layout → CSS Parsing and Computation
Updated•2 years ago
|
Summary: Assertion failure: array->Count() == 2 (eCSSUnit_Calc_Plus should have a 2-length array), at nsCSSValue.cpp:873 → [stylo] Assertion failure: array->Count() == 2 (eCSSUnit_Calc_Plus should have a 2-length array), at nsCSSValue.cpp:873
(Assignee) | ||
Comment 3•2 years ago
|
||
Leaving ni to me. Bug 1341985 or bug 1340322 fixes this. I will check this again after those of bugs are closed.
Flags: needinfo?(hikezoe)
(Assignee) | ||
Comment 4•2 years ago
|
||
Bug 1341985 fixed this.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Flags: needinfo?(hikezoe)
Resolution: --- → FIXED
Updated•2 years ago
|
status-firefox53: --- → unaffected
status-firefox54: affected → disabled
status-firefox55: --- → fixed
status-firefox-esr52: --- → unaffected
Updated•2 years ago
|
status-firefox52: --- → unaffected
(Reporter) | ||
Comment 5•2 years ago
|
||
seems this is not fixed, bughunter run into the same assertion again with todays build. Its a stylo bug i guess because on win 7 i didn't crash but with the lastest stylo build from taskcluster i did. Steps to reproduce: Load http://www.wheelcovers.com wait for 10-30 seconds --> Assertion failure: array->Count() == 2 (eCSSUnit_Calc_Plus should have a 2-length array), at /home/worker/workspace/build/src/layout/style/nsCSSValue.cpp:880
Status: RESOLVED → REOPENED
status-firefox55: fixed → affected
Flags: needinfo?(hikezoe)
Resolution: FIXED → ---
(Reporter) | ||
Comment 6•2 years ago
|
||
Created attachment 8856840 [details]
new bughunter stack
Updated•2 years ago
|
Summary: [stylo] Assertion failure: array->Count() == 2 (eCSSUnit_Calc_Plus should have a 2-length array), at nsCSSValue.cpp:873 → stylo: Assertion failure: array->Count() == 2 (eCSSUnit_Calc_Plus should have a 2-length array), at nsCSSValue.cpp:873
Comment 7•2 years ago
|
||
Hiro, looks like this assertion failure was not fixed by bug 1341985 or bug 1340322.
Assignee: nobody → hikezoe
Priority: -- → P2
(Assignee) | ||
Comment 8•2 years ago
|
||
I will look into this after I fixed bug 1354947.
(Assignee) | ||
Comment 9•2 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=744f252d805441d018e10e5210e5351eea28662c
Flags: needinfo?(hikezoe)
(Assignee) | ||
Comment 10•2 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=e26ec592ae9ce4ebfc3223b6326a1836fa8c10b7 The commit of crash test in the previous try was empty!
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 13•2 years ago
|
||
mozreview-review |
Comment on attachment 8862659 [details] Bug 1342316 - Check eCSSUnit_Calc_Plus' array length in assertion. https://reviewboard.mozilla.org/r/134524/#review137866
Attachment #8862659 -
Flags: review?(cam) → review+
Comment 14•2 years ago
|
||
mozreview-review |
Comment on attachment 8862660 [details] Bug 1342316 - A crashtest which has calc() method in translate3d function as base style of transform animation. https://reviewboard.mozilla.org/r/134526/#review137870
Attachment #8862660 -
Flags: review?(cam) → review+
Comment 15•2 years ago
|
||
Pushed by hikezoe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/2cf0ebc74751 Check eCSSUnit_Calc_Plus' array length in assertion. r=heycam https://hg.mozilla.org/integration/autoland/rev/24032eef27b3 A crashtest which has calc() method in translate3d function as base style of transform animation. r=heycam
(Assignee) | ||
Comment 16•2 years ago
|
||
Thank you!
![]() |
||
Comment 17•2 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/2cf0ebc74751 https://hg.mozilla.org/mozilla-central/rev/24032eef27b3
Status: REOPENED → RESOLVED
Last Resolved: 2 years ago → 2 years ago
status-firefox55: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in
before you can comment on or make changes to this bug.
Description
•