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

Unified Diff: third_party/WebKit/LayoutTests/inspector/sources/debugger-breakpoints/disable-breakpoints.html

Issue 2491983002: [DevTools] reworked JavaScriptBreakpointsSidebarPane.js (Closed)
Patch Set: addressed offline 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-breakpoints/disable-breakpoints.html
diff --git a/third_party/WebKit/LayoutTests/inspector/sources/debugger-breakpoints/disable-breakpoints.html b/third_party/WebKit/LayoutTests/inspector/sources/debugger-breakpoints/disable-breakpoints.html
index bb0b5d5da94d369f1c80387939931edfaae4ecbd..c7da0c9580eba488447ccdca6418c4624ebde13d 100644
--- a/third_party/WebKit/LayoutTests/inspector/sources/debugger-breakpoints/disable-breakpoints.html
+++ b/third_party/WebKit/LayoutTests/inspector/sources/debugger-breakpoints/disable-breakpoints.html
@@ -92,11 +92,11 @@ var test = function()
{
InspectorTest.addResult("Test function finished.");
InspectorTest.dumpBreakpointSidebarPane();
- InspectorTest.addSniffer(currentSourceFrame, "_removeBreakpointDecoration", breakpointRemoved);
+ InspectorTest.addSniffer(WebInspector.JavaScriptBreakpointsSidebarPane.prototype, "didUpdateForTest", dumpBreakpointSidebarPane.bind(this));
InspectorTest.removeBreakpoint(currentSourceFrame, 7);
}
- function breakpointRemoved()
+ function dumpBreakpointSidebarPane()
{
InspectorTest.addResult("Breakpoints removed.");
InspectorTest.dumpBreakpointSidebarPane();

Powered by Google App Engine
This is Rietveld 408576698