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

Unified Diff: third_party/WebKit/LayoutTests/inspector/sources/debugger/live-edit.html

Issue 2505413002: [DevTools] Prepare JavaScriptSourceFrame for multiple breakpoints per line (Closed)
Patch Set: addressed comments Created 4 years, 1 month 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/sources/debugger/live-edit.html
diff --git a/third_party/WebKit/LayoutTests/inspector/sources/debugger/live-edit.html b/third_party/WebKit/LayoutTests/inspector/sources/debugger/live-edit.html
index de3682e0aa1c8b288660030c09d7024127d92808..526bd949e6424c18f23f6d440690db2476804239 100644
--- a/third_party/WebKit/LayoutTests/inspector/sources/debugger/live-edit.html
+++ b/third_party/WebKit/LayoutTests/inspector/sources/debugger/live-edit.html
@@ -113,7 +113,7 @@ function test()
function didShowScriptSource(sourceFrame)
{
- InspectorTest.addSniffer(sourceFrame, "_addBreakpointDecoration", breakpointAdded);
+ InspectorTest.waitJavaScriptSourceFrameBreakpoints(sourceFrame).then(breakpointAdded);
InspectorTest.setBreakpoint(sourceFrame, 2, "", true);
}
@@ -140,14 +140,10 @@ function test()
function didShowScriptSource(sourceFrame)
{
testSourceFrame = sourceFrame;
- InspectorTest.addSniffer(sourceFrame, "_addBreakpointDecoration", uiBreakpointAdded);
+ InspectorTest.waitJavaScriptSourceFrameBreakpoints(testSourceFrame).then(breakpointAdded);
InspectorTest.setBreakpoint(sourceFrame, 3, "", true);
}
- function uiBreakpointAdded() {
- InspectorTest.addSniffer(testSourceFrame, "_addBreakpointDecoration", breakpointAdded);
- }
-
function breakpointAdded()
{
InspectorTest.waitUntilPaused(pausedInF);

Powered by Google App Engine
This is Rietveld 408576698