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

Unified Diff: third_party/WebKit/LayoutTests/inspector-protocol/css/css-set-rule-selector.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-rule-selector.html
diff --git a/third_party/WebKit/LayoutTests/inspector-protocol/css/css-set-rule-selector.html b/third_party/WebKit/LayoutTests/inspector-protocol/css/css-set-rule-selector.html
index 6eea64a5a36c39f4975a27a6e677f1b9c4378d61..af061505fa48611a925425d0959b0251d8807de7 100644
--- a/third_party/WebKit/LayoutTests/inspector-protocol/css/css-set-rule-selector.html
+++ b/third_party/WebKit/LayoutTests/inspector-protocol/css/css-set-rule-selector.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 setRuleSelector;
- var verifyProtocolError;
+ var verifydpError;
var dumpStyleSheet;
var firstSelectorRange = {
@@ -42,7 +42,7 @@ function test()
{
var styleSheetId = result.params.header.styleSheetId;
setRuleSelector = InspectorTest.setRuleSelector.bind(InspectorTest, styleSheetId, false);
- verifyProtocolError = InspectorTest.setRuleSelector.bind(InspectorTest, styleSheetId, true);
+ verifydpError = InspectorTest.setRuleSelector.bind(InspectorTest, styleSheetId, true);
dumpStyleSheet = InspectorTest.dumpStyleSheetText.bind(null, styleSheetId);
InspectorTest.sendCommandOrDie("CSS.getStyleSheetText", { styleSheetId: styleSheetId }, onInitialStyleSheetText);
}
@@ -98,7 +98,7 @@ function test()
function testInvalidSelectorText(next)
{
- verifyProtocolError({
+ verifydpError({
range: firstSelectorRange,
selector: "123"
}, next);
@@ -106,7 +106,7 @@ function test()
function testEmptySelector(next)
{
- verifyProtocolError({
+ verifydpError({
range: firstSelectorRange,
selector: ""
}, next);
@@ -114,7 +114,7 @@ function test()
function testSelectorTextWithUnclosedComment(next)
{
- verifyProtocolError({
+ verifydpError({
range: firstSelectorRange,
selector: "body /*"
}, next);
@@ -122,7 +122,7 @@ function test()
function testSelectorTextWithBrace(next)
{
- verifyProtocolError({
+ verifydpError({
range: firstSelectorRange,
selector: "body {"
}, next);
@@ -130,7 +130,7 @@ function test()
function testSelectorTextWithBraces(next)
{
- verifyProtocolError({
+ verifydpError({
range: firstSelectorRange,
selector: "body {}"
}, next);
@@ -138,7 +138,7 @@ function test()
function testSelectorTextWithExtraRule(next)
{
- verifyProtocolError({
+ verifydpError({
range: firstSelectorRange,
selector: "body {} body"
}, next);
@@ -146,7 +146,7 @@ function test()
function testEditSubSelector(next)
{
- verifyProtocolError({
+ verifydpError({
range: { startLine: 12, startColumn: 8, endLine: 12, endColumn: 14 },
selector: "line"
}, next);
@@ -154,7 +154,7 @@ function test()
function testInvalidParameters(next)
{
- verifyProtocolError({
+ verifydpError({
range: { startLine: "three", startColumn: 0, endLine: 4, endColumn: 0 },
selector: "no matter what is here"
}, next);
@@ -162,7 +162,7 @@ function test()
function testNegativeRangeParameters(next)
{
- verifyProtocolError({
+ verifydpError({
range: { startLine: -1, startColumn: -1, endLine: 1, endColumn: 0 },
selector: "html > body > div"
}, next);
@@ -170,7 +170,7 @@ function test()
function testStartLineOutOfBounds(next)
{
- verifyProtocolError({
+ verifydpError({
range: { startLine: 100, startColumn: 0, endLine: 100, endColumn: 0 },
selector: "html > body > div"
}, next);
@@ -178,7 +178,7 @@ function test()
function testEndLineOutOfBounds(next)
{
- verifyProtocolError({
+ verifydpError({
range: { startLine: 0, startColumn: 0, endLine: 100, endColumn: 0 },
selector: "html > body > div"
}, next);
@@ -186,7 +186,7 @@ function test()
function testStartColumnBeyondLastLineCharacter(next)
{
- verifyProtocolError({
+ verifydpError({
range: { startLine: 3, startColumn: 1000, endLine: 3, endColumn: 1000 },
selector: "html > body > div"
}, next);
@@ -194,7 +194,7 @@ function test()
function testEndColumnBeyondLastLineCharacter(next)
{
- verifyProtocolError({
+ verifydpError({
range: { startLine: 3, startColumn: 0, endLine: 3, endColumn: 1000 },
selector: "html > body > div"
}, next);
@@ -202,7 +202,7 @@ function test()
function testInsertBeyondLastCharacterOfLastLine(next)
{
- verifyProtocolError({
+ verifydpError({
range: { startLine: 3, startColumn: 2, endLine: 3, endColumn: 2 },
selector: "html > body > div"
}, next);
@@ -210,7 +210,7 @@ function test()
function testReversedRange(next)
{
- verifyProtocolError({
+ verifydpError({
range: { startLine: 2, startColumn: 0, endLine: 0, endColumn: 0 },
selector: "html > body > div"
}, next);

Powered by Google App Engine
This is Rietveld 408576698