Closed Bug 1031444 Opened 8 years ago Closed 8 years ago

Add missing COMPUTED_STYLE_PROP declaration for 'box-decoration-break'


(Core :: CSS Parsing and Computation, defect, P3)




Tracking Status
firefox30 --- unaffected
firefox31 --- unaffected
firefox32 + fixed
firefox33 --- fixed


(Reporter: mats, Assigned: mats)




(1 file)

No description provided.
This was part of the patches for bug 613659 but apparently got lost
while a rebased the patches... :-(
Attachment #8447340 - Flags: review?(roc)
Blocks: 1006326
How did this pass tests?
(In reply to David Baron [:dbaron] (UTC-7) (needinfo? for questions) from comment #3)
> How did this pass tests?

The style system tests derived from property_database.js are
not run for properties that are disabled (they failed when
I enabled it).  The reftests that we do run with the pref
forced to 'true' only use simple CSS rules which apparently
don't need this thing to work.

Fwiw, I did do a full Try run with the pref enabled on an
earlier patch set that passed, but not on the final set
after the rebase.
I don't pretend to have any idea how this could be the case, but either this or bug 1028460 somehow broke /tests/dom/asmjscache/test/test_cachingBasic.html ("asm.js compilation is available") on Android 2.2 Armv6 Opt,

Backed out in
I think it's unlikely that it's this bug that caused the failure
so I pushed it again:
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
Comment on attachment 8447340 [details] [diff] [review]
Add missing COMPUTED_STYLE_PROP declaration for 'box-decoration-break'

Approval Request Comment
[Feature/regressing bug #]: bug 613659
[User impact if declined]: box-decoration-break will not work as intended
[Describe test coverage new/current, TBPL]: on m-c since 2014-06-29
[Risks and why]: zero risk
[String/UUID change made/needed]: none
Attachment #8447340 - Flags: approval-mozilla-aurora?
Comment on attachment 8447340 [details] [diff] [review]
Add missing COMPUTED_STYLE_PROP declaration for 'box-decoration-break'

Attachment #8447340 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.