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

Unified Diff: third_party/WebKit/LayoutTests/inspector-protocol/runtime/runtime-console-timestamp.html

Issue 2139543002: [DevTools] Report console API calls through Runtime. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebased Created 4 years, 5 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/runtime/runtime-console-timestamp.html
diff --git a/third_party/WebKit/LayoutTests/inspector-protocol/console/console-timestamp.html b/third_party/WebKit/LayoutTests/inspector-protocol/runtime/runtime-console-timestamp.html
similarity index 81%
rename from third_party/WebKit/LayoutTests/inspector-protocol/console/console-timestamp.html
rename to third_party/WebKit/LayoutTests/inspector-protocol/runtime/runtime-console-timestamp.html
index 1d957267d1af841c16307094a64fa96da9c85bcc..e58a8d7d8f201f0624643d2eab101221a9dff731 100644
--- a/third_party/WebKit/LayoutTests/inspector-protocol/console/console-timestamp.html
+++ b/third_party/WebKit/LayoutTests/inspector-protocol/runtime/runtime-console-timestamp.html
@@ -10,7 +10,7 @@ function test()
function messageAdded(data)
{
- var payload = data.params.message;
+ var payload = data.params;
if (messages.length > 0)
InspectorTest.log("Message " + messages.length + " has non-decreasing timestamp: " + (payload.timestamp >= messages[messages.length - 1].timestamp));
@@ -18,13 +18,13 @@ function test()
messages.push(payload);
InspectorTest.log("Message has timestamp: " + !!payload.timestamp);
- InspectorTest.log("Message timestamp doesn't differ too much from current time (one minute interval): " + (Math.abs(new Date().getTime() / 1000 - payload.timestamp) < 60));
+ InspectorTest.log("Message timestamp doesn't differ too much from current time (one minute interval): " + (Math.abs(new Date().getTime() - payload.timestamp) < 60000));
if (messages.length === 3)
InspectorTest.completeTest();
}
- InspectorTest.eventHandler["Console.messageAdded"] = messageAdded;
- InspectorTest.sendCommand("Console.enable", {});
+ InspectorTest.eventHandler["Runtime.consoleAPICalled"] = messageAdded;
+ InspectorTest.sendCommand("Runtime.enable", {});
InspectorTest.sendCommand("Runtime.evaluate", { expression: "console.log('testUnique'); for (var i = 0; i < 2; ++i) console.log('testDouble');" });
}
@@ -34,3 +34,4 @@ function test()
<body onload="runTest()">
</body>
</html>
+

Powered by Google App Engine
This is Rietveld 408576698