Closed Bug 977248 Opened 6 years ago Closed 6 years ago

Remove _moz-rs-heading hack from quirk.css

Categories

(Core :: CSS Parsing and Computation, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla30

People

(Reporter: sicking, Assigned: dbaron)

References

()

Details

Attachments

(1 file)

We no longer set a "_moz-rs-heading" attribute anywhere, so this rule in quirk.css is a performance-eating no-op.

http://mxr.mozilla.org/mozilla-central/source/layout/style/quirk.css?force=1#209
Assignee: nobody → dbaron
Status: NEW → ASSIGNED
Ooh, this is actually a quirk whose removal gets us something useful.
Component: Layout → CSS Parsing and Computation
Comment on attachment 8382762 [details] [diff] [review]
Remove quirk.css rule that selects on an attribute that was only set by the old HTML parser.

Review of attachment 8382762 [details] [diff] [review]:
-----------------------------------------------------------------

::: layout/style/quirk.css
@@ +212,1 @@
>   margin-bottom: 1em;

If you're going to fix the whitespace here, mind bumping the margin-bottom line over a space?
Attachment #8382762 - Flags: review?(mrbkap) → review+
https://hg.mozilla.org/mozilla-central/rev/b0e36a2e2ee5
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in before you can comment on or make changes to this bug.