Open
Bug 1360929
Opened 7 years ago
Updated 1 year ago
Add serialization tests for combination units of "%" and "em" of flex-basis and word-spacing properties
Categories
(Core :: CSS Parsing and Computation, enhancement, P3)
Core
CSS Parsing and Computation
Tracking
()
NEW
People
(Reporter: chenpighead, Unassigned)
References
Details
Per Bug 1359786 comment 16: > (In reply to Hiroyuki Ikezoe (:hiro) from comment #16) > I think the problem does not happen when the computed value is calculated > through in StyleAnimationValue::AddWeighted() (i.e not at offset 0 or 1). > So it's probably just a serialization problem (I mean it's not an animation > problem). > A bad sideeffect of this change what I can think of is that it's hard to > notice if someone fix bug 1291187 unintentionally. So it would be good to > add an automation test for it. We have already such test cases somewhere in > layout/style/test? > Even we re-enabled the animation tests, it doesn't mean that the serialization issue is resolved (note that Bug 1291187 still remains). So, it would be better to add tests to really reflect the serialization issues in Bug 1291187. I think we can add combination units of "%" and "em" as valid other values for flex-basis and word-spacing properties in property_database.js. Then, we shall be able to test them through test_value_computation.html or test_compute_data_with_start_struct.html. Though I haven't walked through all the layout/style/test_*.html files....
Updated•7 years ago
|
Priority: -- → P3
Updated•1 year ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•