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

Unified Diff: third_party/WebKit/LayoutTests/inspector-protocol/css/css-set-media-text.html

Issue 2942573003: [DevTools] New harness for inspector-protocol layout tests (Closed)
Patch Set: Protocol -> dp Created 3 years, 6 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/inspector-protocol/css/css-set-media-text.html
diff --git a/third_party/WebKit/LayoutTests/inspector-protocol/css/css-set-media-text.html b/third_party/WebKit/LayoutTests/inspector-protocol/css/css-set-media-text.html
index 1c8f5795729f9b5fe312ba130bbe6bea0b8c3e8b..d7ac4fe4575b5e704e23333e2eb9b68238c57d5e 100644
--- a/third_party/WebKit/LayoutTests/inspector-protocol/css/css-set-media-text.html
+++ b/third_party/WebKit/LayoutTests/inspector-protocol/css/css-set-media-text.html
@@ -1,7 +1,7 @@
<html>
<head>
-<script type="text/javascript" src="../../http/tests/inspector-protocol/inspector-protocol-test.js"></script>
-<script type="text/javascript" src="../../http/tests/inspector-protocol/css-protocol-test.js"></script>
+<script type="text/javascript" src="../../http/tests/inspector-protocol/resources/inspector-protocol-test.js"></script>
+<script type="text/javascript" src="../resources/css-protocol-test.js"></script>
<script type="text/javascript">
function test()
{
@@ -10,7 +10,7 @@ function test()
InspectorTest.sendCommandOrDie("CSS.enable", {});
var setMediaText;
- var verifyProtocolError;
+ var verifydpError;
var dumpStyleSheet;
var firstMediaRange = {
@@ -30,7 +30,7 @@ function test()
{
var styleSheetId = result.params.header.styleSheetId;
setMediaText = InspectorTest.setMediaText.bind(InspectorTest, styleSheetId, false);
- verifyProtocolError = InspectorTest.setMediaText.bind(InspectorTest, styleSheetId, true);
+ verifydpError = InspectorTest.setMediaText.bind(InspectorTest, styleSheetId, true);
dumpStyleSheet = InspectorTest.dumpStyleSheetText.bind(null, styleSheetId);
InspectorTest.sendCommandOrDie("CSS.getStyleSheetText", { styleSheetId: styleSheetId }, onInitialStyleSheetText);
}
@@ -69,7 +69,7 @@ function test()
function testInvalidParameters(next)
{
- verifyProtocolError({
+ verifydpError({
range: { startLine: "three", startColumn: 0, endLine: 4, endColumn: 0 },
text: "no matter what is here"
}, next);
@@ -77,7 +77,7 @@ function test()
function testInvalidText(next)
{
- verifyProtocolError({
+ verifydpError({
range: firstMediaRange,
text: "something /* is wrong here"
}, next);
@@ -85,7 +85,7 @@ function test()
function testInvalidText2(next)
{
- verifyProtocolError({
+ verifydpError({
range: firstMediaRange,
text: "something { is { wrong: here} }"
}, next);
@@ -93,7 +93,7 @@ function test()
function testInvalidText3(next)
{
- verifyProtocolError({
+ verifydpError({
range: firstMediaRange,
text: "something { wrong"
}, next);

Powered by Google App Engine
This is Rietveld 408576698