Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Gecko Bug 1545823] Implement non-standard CSSStyleSheet.rules, CSSStyleSheet.addRule and CSSStyleSheet.removeRule. #16876

Merged
merged 1 commit into from May 16, 2019

Conversation

moz-wptsync-bot
Copy link
Collaborator

It's not worth dying on this hill. Both Blink and WebKit pass the tests.

(Well, WebKit actually fails one of the latest ones I wrote, cssRules and rules
are not the same JS object, WebKit returns a new rule list. I'll file)

Spec PR in w3c/csswg-drafts#3900.

Differential Revision: https://phabricator.services.mozilla.com/D30348

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1545823
gecko-commit: a93ed2a80220b9ba15fb9b59ea79b5f1ee8f7693
gecko-integration-branch: central
gecko-reviewers: bzbarsky

… CSSStyleSheet.removeRule.

It's not worth dying on this hill. Both Blink and WebKit pass the tests.

(Well, WebKit actually fails one of the latest ones I wrote, cssRules and rules
are not the same JS object, WebKit returns a new rule list. I'll file)

Spec PR in w3c/csswg-drafts#3900.

Differential Revision: https://phabricator.services.mozilla.com/D30348

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1545823
gecko-commit: a93ed2a80220b9ba15fb9b59ea79b5f1ee8f7693
gecko-integration-branch: central
gecko-reviewers: bzbarsky
Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Already reviewed downstream.

@moz-wptsync-bot moz-wptsync-bot merged commit 5f6ae2d into master May 16, 2019
@moz-wptsync-bot moz-wptsync-bot deleted the gecko/1545823 branch May 16, 2019 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants