Skip to content

Commit

Permalink
Bug 1379577 - stylo: test custom properties computed values consisten…
Browse files Browse the repository at this point in the history
…t order. r=heycam

MozReview-Commit-ID: HTg9ynXJNRK

--HG--
extra : rebase_source : ac0a94bd5d19d417deb5fe7f50bb5fcb69e21611
  • Loading branch information
ferjm committed Aug 7, 2017
1 parent d58fe88 commit 3e43af6
Show file tree
Hide file tree
Showing 2 changed files with 55 additions and 0 deletions.
2 changes: 2 additions & 0 deletions layout/style/test/mochitest.ini
Original file line number Diff line number Diff line change
Expand Up @@ -320,6 +320,8 @@ skip-if = toolkit == 'android'
[test_variable_serialization_specified.html]
[test_variables.html]
support-files = support/external-variable-url.css
[test_variables_order.html]
support-files = support/external-variable-url.css
[test_video_object_fit.html]
[test_viewport_scrollbar_causing_reflow.html]
[test_viewport_units.html]
Expand Down
53 changes: 53 additions & 0 deletions layout/style/test/test_variables_order.html
Original file line number Diff line number Diff line change
@@ -0,0 +1,53 @@
<!DOCTYPE type>
<title>CSS variables order tests</title>
<script src="/MochiKit/MochiKit.js"></script>
<script src="/tests/SimpleTest/SimpleTest.js"></script>
<link rel="stylesheet" href="/tests/SimpleTest/test.css" type="text/css">

<style id="test">
</style>

<div id="t4"></div>

<script>

/*
* Although the spec does not enforce any specific order, Gecko and Servo
* implement a consistent ordering for CSSDeclaration objects in the DOM.
* CSSDeclarations expose property names as indexed properties, which need
* to be stable. This order is the order that properties are cascaded in.
*
* We have this test just to prevent regressions, rather than testing specific
* mandated behavior.
*/

function prepareTest() {
var e = document.createElement("link");
e.addEventListener("load", runTest);
e.setAttribute("rel", "stylesheet");
e.setAttribute("href", "support/external-variable-url.css");
document.head.appendChild(e);
}

function runTest() {
var test = document.getElementById("test");
test.textContent = "div { --SomeVariableName: a; }";

var declaration = test.sheet.cssRules[0].style;
is(declaration.item(0), "--SomeVariableName", "custom property name returned by item() on style declaration");
is(declaration[0], "--SomeVariableName", "custom property name returned by indexed getter on style declaration");

var element = document.getElementById("t4");
var cs = window.getComputedStyle(element);

["--SomeVariableName", "--a"].forEach((varName, index) => {
is(cs.item(cs.length - (index + 1)), varName, "custom property name returned by item() on computed style");
is(cs[cs.length - (index + 1)], varName, "custom property name returned by indexed getter on style declaration");
});

SimpleTest.finish();
}

SimpleTest.waitForExplicitFinish();
prepareTest();
</script>

0 comments on commit 3e43af6

Please sign in to comment.