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

Unified Diff: third_party/WebKit/LayoutTests/inspector/sources/debugger/source-frame-inline-breakpoint-decorations.html

Issue 2545813003: [DevTools] fix minor issue with inline breakpoints (Closed)
Patch Set: Created 4 years 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/source-frame-inline-breakpoint-decorations.html
diff --git a/third_party/WebKit/LayoutTests/inspector/sources/debugger/source-frame-inline-breakpoint-decorations.html b/third_party/WebKit/LayoutTests/inspector/sources/debugger/source-frame-inline-breakpoint-decorations.html
index 93329e48bed3a3fb434790836025e8dbfded7421..92a7ce8f1cbc1fd3b043c6dd9fdc140d04367dfa 100644
--- a/third_party/WebKit/LayoutTests/inspector/sources/debugger/source-frame-inline-breakpoint-decorations.html
+++ b/third_party/WebKit/LayoutTests/inspector/sources/debugger/source-frame-inline-breakpoint-decorations.html
@@ -101,6 +101,35 @@ function test()
waitAndDumpDecorations(javaScriptSourceFrame).then(() => next());
InspectorTest.clickJavaScriptSourceFrameBreakpoint(javaScriptSourceFrame, 3, 1);
}
+ },
+
+ function toggleBreakpointInAnotherLineWontRemoveExisting(next) {
+ var javaScriptSourceFrame;
+ InspectorTest.showScriptSource("foo.js", addBreakpoint);
+
+ function addBreakpoint(sourceFrame)
+ {
+ javaScriptSourceFrame = sourceFrame;
+ InspectorTest.prepareSourceFrameForBreakpointTest(sourceFrame);
+ InspectorTest.addResult("Setting breakpoint in line 4");
+ InspectorTest.toggleBreakpoint(sourceFrame, 4, false);
+ waitAndDumpDecorations(javaScriptSourceFrame).then(toggleBreakpointInAnotherLine);
+ }
+
+ function toggleBreakpointInAnotherLine()
+ {
+ InspectorTest.addResult("Setting breakpoint in line 3");
+ waitAndDumpDecorations(javaScriptSourceFrame).then(removeBreakpoints);
+ InspectorTest.toggleBreakpoint(javaScriptSourceFrame, 3, false);
+ }
+
+ function removeBreakpoints()
+ {
+ InspectorTest.addResult("Click by first inline breakpoints");
+ waitAndDumpDecorations(javaScriptSourceFrame).then(() => next());
+ InspectorTest.clickJavaScriptSourceFrameBreakpoint(javaScriptSourceFrame, 3, 0);
+ InspectorTest.clickJavaScriptSourceFrameBreakpoint(javaScriptSourceFrame, 4, 0);
+ }
}
]);
};

Powered by Google App Engine
This is Rietveld 408576698