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

Side by Side Diff: third_party/WebKit/Source/devtools/front_end/sources/XHRBreakpointsSidebarPane.js

Issue 2391773002: [DevTools] Update XHR breakpoint enabled checkbox when needed (Closed)
Patch Set: Created 4 years, 2 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2015 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** 5 /**
6 * @constructor 6 * @constructor
7 * @extends {WebInspector.BreakpointsSidebarPaneBase} 7 * @extends {WebInspector.BreakpointsSidebarPaneBase}
8 * @implements {WebInspector.ContextFlavorListener} 8 * @implements {WebInspector.ContextFlavorListener}
9 * @implements {WebInspector.TargetManager.Observer} 9 * @implements {WebInspector.TargetManager.Observer}
10 * @implements {WebInspector.ToolbarItem.ItemsProvider} 10 * @implements {WebInspector.ToolbarItem.ItemsProvider}
(...skipping 82 matching lines...) Expand 10 before | Expand all | Expand 10 after
93 /** 93 /**
94 * @param {string} url 94 * @param {string} url
95 * @param {boolean} enabled 95 * @param {boolean} enabled
96 * @param {!WebInspector.Target=} target 96 * @param {!WebInspector.Target=} target
97 */ 97 */
98 _setBreakpoint: function(url, enabled, target) 98 _setBreakpoint: function(url, enabled, target)
99 { 99 {
100 if (enabled) 100 if (enabled)
101 this._updateBreakpointOnTarget(url, true, target); 101 this._updateBreakpointOnTarget(url, true, target);
102 102
103 if (this._breakpointElements.has(url)) 103 if (this._breakpointElements.has(url)) {
104 this._breakpointElements.get(url)._checkboxElement.checked = enabled ;
104 return; 105 return;
106 }
105 107
106 var element = createElement("li"); 108 var element = createElement("li");
107 element._url = url; 109 element._url = url;
108 element.addEventListener("contextmenu", this._contextMenu.bind(this, url ), true); 110 element.addEventListener("contextmenu", this._contextMenu.bind(this, url ), true);
109 111
110 var title = url ? WebInspector.UIString("URL contains \"%s\"", url) : We bInspector.UIString("Any XHR"); 112 var title = url ? WebInspector.UIString("URL contains \"%s\"", url) : We bInspector.UIString("Any XHR");
111 var label = createCheckboxLabel(title, enabled); 113 var label = createCheckboxLabel(title, enabled);
112 element.appendChild(label); 114 element.appendChild(label);
113 label.checkboxElement.addEventListener("click", this._checkboxClicked.bi nd(this, url), false); 115 label.checkboxElement.addEventListener("click", this._checkboxClicked.bi nd(this, url), false);
114 element._checkboxElement = label.checkboxElement; 116 element._checkboxElement = label.checkboxElement;
(...skipping 151 matching lines...) Expand 10 before | Expand all | Expand 10 after
266 var breakpoints = this._xhrBreakpointsSetting.get(); 268 var breakpoints = this._xhrBreakpointsSetting.get();
267 for (var i = 0; i < breakpoints.length; ++i) { 269 for (var i = 0; i < breakpoints.length; ++i) {
268 var breakpoint = breakpoints[i]; 270 var breakpoint = breakpoints[i];
269 if (breakpoint && typeof breakpoint.url === "string") 271 if (breakpoint && typeof breakpoint.url === "string")
270 this._setBreakpoint(breakpoint.url, breakpoint.enabled, target); 272 this._setBreakpoint(breakpoint.url, breakpoint.enabled, target);
271 } 273 }
272 }, 274 },
273 275
274 __proto__: WebInspector.BreakpointsSidebarPaneBase.prototype 276 __proto__: WebInspector.BreakpointsSidebarPaneBase.prototype
275 } 277 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698