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

Unified Diff: third_party/WebKit/LayoutTests/inspector/device-mode/forced-viewport-far-layered.html

Issue 2960263002: [DevTools] Move pixel tests from inspector-protocol to inspector (Closed)
Patch Set: rebased 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/device-mode/forced-viewport-far-layered.html
diff --git a/third_party/WebKit/LayoutTests/inspector-protocol/emulation/forced-viewport-far-layered.html b/third_party/WebKit/LayoutTests/inspector/device-mode/forced-viewport-far-layered.html
similarity index 67%
rename from third_party/WebKit/LayoutTests/inspector-protocol/emulation/forced-viewport-far-layered.html
rename to third_party/WebKit/LayoutTests/inspector/device-mode/forced-viewport-far-layered.html
index 1b7b3326f421af8496c121dc862aea9c8ca8bb82..c2f69135c48643e2a8a1acb9a092c8893682a029 100644
--- a/third_party/WebKit/LayoutTests/inspector-protocol/emulation/forced-viewport-far-layered.html
+++ b/third_party/WebKit/LayoutTests/inspector/device-mode/forced-viewport-far-layered.html
@@ -1,7 +1,7 @@
<html>
<head>
-<script src="../../http/tests/inspector-protocol/resources/inspector-protocol-test.js"></script>
+<script src="../../http/tests/inspector/inspector-test.js"></script>
<style>
html {
@@ -37,14 +37,15 @@ body {
<script>
function test()
{
- InspectorTest.sendCommand("Runtime.evaluate", { "expression": "testRunner.dumpAsTextWithPixelResults();"} );
-
- InspectorTest.sendCommand("Emulation.forceViewport", { "x": 200, "y": 9000, "scale": 2.0 }, overrideActive);
+ InspectorTest.evaluateInPage("testRunner.dumpAsTextWithPixelResults();", () => {
+ InspectorTest.EmulationAgent.invoke_forceViewport({x: 200, y: 9000, scale: 2.0}).then(overrideActive);
+ });
function overrideActive(msg)
{
// Complete the test without closing the inspector, so that the override stays active for the picture.
- InspectorTest.sendCommand("Runtime.evaluate", { "expression": "testRunner.notifyDone();"} );
+ InspectorTest.flushResults();
+ InspectorTest.evaluateInPage("testRunner.notifyDone();");
}
}
</script>

Powered by Google App Engine
This is Rietveld 408576698