Skip to content

Commit

Permalink
Add a WPT for removeProperty("all").
Browse files Browse the repository at this point in the history
This fails in Chrome, see bug 1939643 and https://crbug.com/387030982

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1939701
gecko-commit: c4344adf1971d3c00d1273c5c01a36ca4c8094b8
gecko-reviewers: firefox-style-system-reviewers, zrhoffman
  • Loading branch information
emilio authored and moz-wptsync-bot committed Jan 3, 2025
1 parent 837678e commit 71e9f98
Showing 1 changed file with 17 additions and 0 deletions.
17 changes: 17 additions & 0 deletions css/cssom/cssstyledeclaration-removeProperty-all.html
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
<!DOCTYPE html>
<meta charset="utf-8">
<title>CSSStyleDeclaration.removeProperty("all")</title>
<link rel="author" title="Emilio Cobos Álvarez" href="mailto:[email protected]">
<link rel="author" title="Mozilla" href="https://mozilla.org">
<link rel="help" href="https://drafts.csswg.org/cssom-1/#dom-cssstyledeclaration-removeproperty">
<script src="/resources/testharness.js"></script>
<script src="/resources/testharnessreport.js"></script>
<script>
test(function() {
let style = document.createElement("div").style;
style.width = "40px";
assert_equals(style.length, 1, "setter should work as expected");
style.removeProperty("all");
assert_equals(style.length, 0, "all is a shorthand of all properties, so should remove the property");
});
</script>

0 comments on commit 71e9f98

Please sign in to comment.