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

Unified Diff: LayoutTests/inspector/elements/styles/styles-disable-then-enable-overriden-ua.html

Issue 1058723002: Remove testRunner.injectStyleSheet usage from layout tests. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 9 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: LayoutTests/inspector/elements/styles/styles-disable-then-enable-overriden-ua.html
diff --git a/LayoutTests/inspector/elements/styles/styles-disable-then-enable-overriden-ua.html b/LayoutTests/inspector/elements/styles/styles-disable-then-enable-overriden-ua.html
deleted file mode 100644
index dfc9fa7489ab52dad5925c5aa4aacf600153ed62..0000000000000000000000000000000000000000
--- a/LayoutTests/inspector/elements/styles/styles-disable-then-enable-overriden-ua.html
+++ /dev/null
@@ -1,63 +0,0 @@
-<html>
dcheng 2015/04/02 04:41:18 Given that this also seems to test style sheet inj
pfeldman 2015/04/02 05:34:12 According to the description, the test is about ov
dcheng 2015/04/02 05:39:12 I don't think it's worth keeping around ~500 lines
pfeldman 2015/04/02 05:46:16 By no means I am suggesting to keep the code. I am
-<head>
-<script src="../../../http/tests/inspector/inspector-test.js"></script>
-<script src="../../../http/tests/inspector/elements-test.js"></script>
-
-<script>
-
-function injectStyleSheet()
-{
- var styleSheet = "#container { margin: 8px }";
- if (window.testRunner)
- window.testRunner.injectStyleSheet(styleSheet, true);
-}
-
-function test()
-{
- InspectorTest.selectNodeAndWaitForStyles("container", step0);
-
- function step0()
- {
- InspectorTest.evaluateInPage("injectStyleSheet()", step1);
- }
-
- function step1()
- {
- InspectorTest.waitForStyles("container", step2);
- }
-
- function step2()
- {
- InspectorTest.addResult("Before disable");
- InspectorTest.dumpSelectedElementStyles(true, false, true);
- InspectorTest.toggleStyleProperty("margin", false);
- InspectorTest.waitForStyles("container", step3);
- }
-
- function step3()
- {
- InspectorTest.addResult("After disable");
- InspectorTest.dumpSelectedElementStyles(true, false, true);
- InspectorTest.toggleStyleProperty("margin", true);
- InspectorTest.waitForStyles("container", step4);
- }
-
- function step4()
- {
- InspectorTest.addResult("After enable");
- InspectorTest.dumpSelectedElementStyles(true, false, true);
- InspectorTest.completeTest();
- }
-}
-</script>
-</head>
-
-<body onload="runTest()" id="body-id">
-<p>
-Tests that disabling shorthand removes the "overriden" mark from the UA shorthand it overrides.
-</p>
-
-<div id="container" style="margin: 10px"></div>
-
-</body>
-</html>

Powered by Google App Engine
This is Rietveld 408576698