Closed Bug 1523712 Opened 5 years ago Closed 5 years ago

Restrict parsing of -moz-binding to chrome and UA sheets.

Categories

(Core :: CSS Parsing and Computation, task)

task
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla67
Tracking Status
firefox67 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

Details

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

Attachments

(1 file)

No description provided.

Tentatively blocking on Bug 1465457, although it sounds like there could be a workaround for reftests to enable -moz-binding there.

Depends on: 1465457

We probably do need to turn this off in all tests, there are even -moz-binding: being set in inline styles.

https://searchfox.org/mozilla-central/search?q=style%3D%22-moz-binding&case=false&regexp=false&path=

Hmm, yeah, that's right... That's unfortunate, but probably not worth to rewrite tests.

But enable it in all tests because a lot of them rely on using it in the
style="" attribute for example, or in inline stylesheets, which will no longer
parse this (even in chrome documents), and we don't want to rewrite all the XUL
and XBL tests.

Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b866f7486b8f
Make -moz-binding chrome / UA only. r=boris
Keywords: site-compat
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
Assignee: nobody → emilio

Note to MDN writers: I've added a note about this to the 67 rel notes: https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/67#CSS

In terms of other things, might need to just update BCD.

I have finished off the documentation of this by updating the compat data: https://github.com/mdn/browser-compat-data/pull/4092

Type: enhancement → task
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: