Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1021)

Unified Diff: third_party/WebKit/LayoutTests/http/tests/css/ignore-pending-link-import.html

Issue 1867513002: Don't apply style elements or PIs with loading imports. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Do the same for processing instructions Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/http/tests/css/ignore-pending-link-import.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/css/ignore-pending-link-import.html b/third_party/WebKit/LayoutTests/http/tests/css/ignore-pending-link-import.html
new file mode 100644
index 0000000000000000000000000000000000000000..80e39bd56d100fe4ed216e7c378cdfd204819cf3
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/http/tests/css/ignore-pending-link-import.html
@@ -0,0 +1,50 @@
+<!DOCTYPE html>
+<h1 id="h1"></h1>
+<pre id="res"></pre>
+<script>
+ if (window.testRunner) {
+ testRunner.dumpAsText();
+ testRunner.waitUntilDone();
+ }
+
+ var result = "";
+
+ function expectComputedOnH1(name, expected) {
+ var value = getComputedStyle(h1)[name];
+ result += name + ": ";
+ if (value == expected)
+ result += "PASS\n";
+ else
+ result += "FAIL - " + value + ", expected " + expected + "\n";
+ }
+
+ function sheetLoaded() {
+ // Need to remove the stylesheet used to trigger the ignore-pending-stylesheet
+ // behavior in order to not have pending stylesheets here.
+ document.head.removeChild(document.head.querySelector("style"));
+
+ expectComputedOnH1("backgroundColor", "rgb(0, 128, 0)");
+ expectComputedOnH1("color", "rgb(0, 128, 0)");
+
+ res.innerHTML = result;
+ if (window.testRunner)
+ testRunner.notifyDone();
+ }
+
+ // This style element is required to trigger updateLayoutTreeIgnorePendingStylesheets
+ // while the main link css file has loaded while its import has not.
+ var style = document.createElement("style");
+ style.appendChild(document.createTextNode("@import url(resources/slow-loading-sheet.php?sleep=10000&color=pink);"));
+ document.head.appendChild(style);
+
+ var link = document.createElement("link");
+ link.setAttribute("onload", "sheetLoaded()");
+ link.setAttribute("rel", "stylesheet");
+ link.setAttribute("href", "data:text/css,@import url(http://localhost:8080/css/resources/slow-loading-sheet.php?color=green); h1{color:green}");
+ document.head.appendChild(link);
+
+ document.body.offsetTop;
+ expectComputedOnH1("backgroundColor", "rgba(0, 0, 0, 0)");
+ expectComputedOnH1("color", "rgb(0, 0, 0)");
+
+</script>

Powered by Google App Engine
This is Rietveld 408576698