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

Unified Diff: third_party/WebKit/Source/devtools/front_end/elements/ElementsPanel.js

Issue 2137313003: DevTools: remove ElementsTreeElement.updateSelection. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 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
« no previous file with comments | « no previous file | third_party/WebKit/Source/devtools/front_end/elements/ElementsTreeElement.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/devtools/front_end/elements/ElementsPanel.js
diff --git a/third_party/WebKit/Source/devtools/front_end/elements/ElementsPanel.js b/third_party/WebKit/Source/devtools/front_end/elements/ElementsPanel.js
index a89c4c1d98ae09dafc34e4394fc5452a67373bba..4b9023d7429fbf7378ceaf0db7cc838bf45bff98 100644
--- a/third_party/WebKit/Source/devtools/front_end/elements/ElementsPanel.js
+++ b/third_party/WebKit/Source/devtools/front_end/elements/ElementsPanel.js
@@ -277,7 +277,6 @@ WebInspector.ElementsPanel.prototype = {
width -= this._splitWidget.sidebarSize();
for (var i = 0; i < this._treeOutlines.length; ++i) {
this._treeOutlines[i].setVisibleWidth(width);
- this._treeOutlines[i].updateSelection();
}
this._breadcrumbs.updateSizes();
},
@@ -315,7 +314,6 @@ WebInspector.ElementsPanel.prototype = {
for (var i = 0; i < this._treeOutlines.length; ++i) {
var treeOutline = this._treeOutlines[i];
- treeOutline.updateSelection();
treeOutline.setVisible(true);
if (!treeOutline.rootDOMNode)
@@ -537,14 +535,6 @@ WebInspector.ElementsPanel.prototype = {
this._contentElement.classList.toggle("elements-wrap", event.data);
for (var i = 0; i < this._treeOutlines.length; ++i)
this._treeOutlines[i].setWordWrap(/** @type {boolean} */ (event.data));
-
- var selectedNode = this.selectedDOMNode();
- if (!selectedNode)
- return;
-
- var treeElement = this._treeElementForNode(selectedNode);
- if (treeElement)
- treeElement.updateSelection(); // Recalculate selection highlight dimensions.
},
switchToAndFocus: function(node)
« no previous file with comments | « no previous file | third_party/WebKit/Source/devtools/front_end/elements/ElementsTreeElement.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698