Closed Bug 915053 Opened 6 years ago Closed 6 years ago

remove quirk allowing {} around style attribute


(Core :: CSS Parsing and Computation, defect)

Not set





(Reporter: dbaron, Assigned: dbaron)


(Blocks 1 open bug)


(Keywords: dev-doc-complete, site-compat)


(1 file) shows that other browsers (IE, blink, WebKit) no longer implement the quirk added in bug 99554 to allow {} around the style attribute.  We should remove this.

(Discussed in CSS meeting today, briefly.)
The tests pass with the patch; without the patch
test_style_attribute_quirks.html fails 2 tests.
Attachment #802989 - Flags: review?(cam)
Comment on attachment 802989 [details] [diff] [review]
Remove quirk allowing {} around style attribute.

Review of attachment 802989 [details] [diff] [review]:

(I guess it would have been simpler to write the test as a reftest, but either way.)
Attachment #802989 - Flags: review?(cam) → review+
(In reply to Cameron McCormack (:heycam) from comment #2)
> (I guess it would have been simpler to write the test as a reftest, but
> either way.)

Script tests should be lower-overhead to run, though.
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
Adding dev-doc-needed, I'm not sure this was previously documented, but if so we have to fix the documentation.
Keywords: dev-doc-needed
"In quirks mode, the CSS parser allows {} around the contents of style attributes (bug 99554)."
I added a sentence "This behavior is being removed in Firefox 27, for interoperability.", plus a link to this bug, after the quote in Comment 7.  Feel free to edit/extend as appropriate.

Leaving dev-doc-needed in case there's anything else that needs updating.
Simon, what's the testing situation here?
Flags: needinfo?(zcorpan)
I've submitted (feel free to review so it can be moved to approved)
Flags: needinfo?(zcorpan)
You need to log in before you can comment on or make changes to this bug.