Closed
Bug 1544590
Opened 5 years ago
Closed 5 years ago
Dynamic changes to 'counter-set' doesn't trigger reflow properly
Categories
(Core :: CSS Parsing and Computation, defect, P3)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla68
People
(Reporter: MatsPalmgren_bugz, Assigned: MatsPalmgren_bugz)
References
Details
(Keywords: regression, testcase)
Attachments
(3 files)
STR
- load the attached testcase in Nightly
ACTUAL RESULT
"3."
EXPECTED RESULT
"1."
ADDITIONAL INFO
Just opening devtools (F12) makes the testcase render correctly.
|
Assignee | |
Comment 1•5 years ago
|
||
|
|
Assignee | |
Comment 2•5 years ago
|
||
Comment on attachment 9058438 [details]
'counter-increment' testcase
The same test using 'counter-increment' instead works correctly though...
Attachment #9058438 -
Attachment description: 'counter-increment → 'counter-increment' testcase
|
|
Assignee | |
Comment 3•5 years ago
|
||
Oops, I forgot to add it here somehow:
https://searchfox.org/mozilla-central/source/layout/style/nsStyleStruct.cpp#3654
Assignee: nobody → mats
|
|
Assignee | |
Comment 4•5 years ago
|
||
Hmm, I did have that originally actually:
https://bugzilla.mozilla.org/attachment.cgi?id=9034771&action=diff#a/layout/style/nsStyleStruct.cpp_sec2
However, the patch that landed is missing the changes to nsStyleStruct.cpp:
https://hg.mozilla.org/integration/mozilla-inbound/rev/467284f16c74
Weird... I wonder if I was bitten by the clang-format extension bug there?
|
|
Assignee | |
Comment 5•5 years ago
|
||
|
|
Assignee | |
Updated•5 years ago
|
Blocks: 1542871
Keywords: regression
|
|
Assignee | |
Comment 6•5 years ago
|
||
Pushed by mpalmgren@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/30ab155d08d2 Add the missing change hint for 'counter-set'. r=emilio
|
|
Assignee | |
Updated•5 years ago
|
Flags: in-testsuite+
OS: Linux → All
Hardware: x86_64 → All
|
||
Comment 8•5 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
|
||
Updated•5 years ago
|
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/16500 for changes under testing/web-platform/tests
You need to log in
before you can comment on or make changes to this bug.
Description
•