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

Unified Diff: third_party/WebKit/LayoutTests/inspector-protocol/debugger/step-into-inline-event-handler.html

Issue 2242893004: [DevTools] Don't mark scripts from V8LazyEventListener as internal (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: addressed comments Created 4 years, 4 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/debugger/step-into-inline-event-handler.html
diff --git a/third_party/WebKit/LayoutTests/inspector-protocol/debugger/step-into-inline-event-handler.html b/third_party/WebKit/LayoutTests/inspector-protocol/debugger/step-into-inline-event-handler.html
new file mode 100644
index 0000000000000000000000000000000000000000..5c6e12faed2c0e2a2892ab342dead16ca1f338b0
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/inspector-protocol/debugger/step-into-inline-event-handler.html
@@ -0,0 +1,54 @@
+<html>
+<head>
+<script type="text/javascript" src="../../http/tests/inspector-protocol/inspector-protocol-test.js"></script>
+<script>
+
+function testFunction()
+{
+ var e = document.getElementById("div");
+ debugger;
+ e.click();
+}
+
+function shouldNotBeThisFunction()
+{
+ return 239;
+}
+
+function test()
+{
+ InspectorTest.waitForEventPromise("Debugger.paused").then(makeStepping);
+
+ InspectorTest.sendCommandPromise("Debugger.enable", {})
+ .then((result) => InspectorTest.sendCommandPromise("Runtime.evaluate", { expression: "testFunction()" }))
+ .then(() => InspectorTest.completeTest());
+
+ function makeStepping()
+ {
+ sendCommandAndWaitForPause("Debugger.stepInto")
+ .then(() => sendCommandAndWaitForPause("Debugger.stepInto"))
+ .then((result) => dumpTopCallFrame(result))
+ .then(() => InspectorTest.sendCommandPromise("Debugger.resume"));
+ }
+
+ function sendCommandAndWaitForPause(command)
+ {
+ InspectorTest.sendCommand(command, {});
+ return InspectorTest.waitForEventPromise("Debugger.paused");
+ }
+
+ function dumpTopCallFrame(result)
+ {
+ var frame = result.params.callFrames[0];
+ InspectorTest.log("functionName (should be empty): " + (frame.functionName.length ? frame.functionName : "empty"));
+ }
+}
+</script>
+</head>
+<div id="div" onclick="shouldNotBeThisFunction()">
+</div>
+<body onLoad="runTest();">
+Tests that Debugger.stepInto doesn't ignore inline event listeners.
+</body>
+</html>
+

Powered by Google App Engine
This is Rietveld 408576698