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

Unified Diff: third_party/WebKit/Source/devtools/front_end/components/BreakpointsSidebarPaneBase.js

Issue 2234193002: DevTools: migrate some of the sources panel sidebar panes to view management, allow view toolbars. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: review comments addressed 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/Source/devtools/front_end/components/BreakpointsSidebarPaneBase.js
diff --git a/third_party/WebKit/Source/devtools/front_end/components/BreakpointsSidebarPaneBase.js b/third_party/WebKit/Source/devtools/front_end/components/BreakpointsSidebarPaneBase.js
index 251248e9252571bf83e7c0ecb98a8a3b1a70b520..949a68d240d5f538a51f92bf73259abaac578245 100644
--- a/third_party/WebKit/Source/devtools/front_end/components/BreakpointsSidebarPaneBase.js
+++ b/third_party/WebKit/Source/devtools/front_end/components/BreakpointsSidebarPaneBase.js
@@ -30,12 +30,11 @@
/**
* @constructor
- * @extends {WebInspector.SimpleView}
- * @param {string} title
+ * @extends {WebInspector.VBox}
*/
-WebInspector.BreakpointsSidebarPaneBase = function(title)
+WebInspector.BreakpointsSidebarPaneBase = function()
{
- WebInspector.SimpleView.call(this, title);
+ WebInspector.VBox.call(this);
this.registerRequiredCSS("components/breakpointsList.css");
this.listElement = createElement("ol");
@@ -46,6 +45,9 @@ WebInspector.BreakpointsSidebarPaneBase = function(title)
this.emptyElement.textContent = WebInspector.UIString("No Breakpoints");
this.element.appendChild(this.emptyElement);
+ WebInspector.targetManager.addModelListener(WebInspector.DebuggerModel, WebInspector.DebuggerModel.Events.DebuggerPaused, this._update, this);
+ WebInspector.targetManager.addModelListener(WebInspector.DebuggerModel, WebInspector.DebuggerModel.Events.DebuggerResumed, this._update, this);
+ WebInspector.context.addFlavorChangeListener(WebInspector.Target, this._update, this);
}
WebInspector.BreakpointsSidebarPaneBase.prototype = {
@@ -92,5 +94,19 @@ WebInspector.BreakpointsSidebarPaneBase.prototype = {
}
},
- __proto__: WebInspector.SimpleView.prototype
+ _update: function()
+ {
+ var target = WebInspector.context.flavor(WebInspector.Target);
+ var debuggerModel = WebInspector.DebuggerModel.fromTarget(target);
+ var details = debuggerModel ? debuggerModel.debuggerPausedDetails() : null;
+ this.highlightDetails(details);
+ },
+
+ /**
+ * @param {?WebInspector.DebuggerPausedDetails} details
+ * @protected
+ */
+ highlightDetails: function(details) { },
+
+ __proto__: WebInspector.VBox.prototype
}

Powered by Google App Engine
This is Rietveld 408576698