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

Unified Diff: third_party/WebKit/Source/devtools/front_end/elements/ElementsTreeOutline.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 | « third_party/WebKit/Source/devtools/front_end/elements/ElementsTreeElement.js ('k') | no next file » | 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/ElementsTreeOutline.js
diff --git a/third_party/WebKit/Source/devtools/front_end/elements/ElementsTreeOutline.js b/third_party/WebKit/Source/devtools/front_end/elements/ElementsTreeOutline.js
index 1e32c4f8afc18de4828b1fa9aa118d9fd4e263df..8c058f8c8af4403a9f7583857ac3fdd07fca6b7b 100644
--- a/third_party/WebKit/Source/devtools/front_end/elements/ElementsTreeOutline.js
+++ b/third_party/WebKit/Source/devtools/front_end/elements/ElementsTreeOutline.js
@@ -450,14 +450,6 @@ WebInspector.ElementsTreeOutline.prototype = {
this._revealAndSelectNode(selectedNode, true);
},
- updateSelection: function()
- {
- if (!this.selectedTreeElement)
- return;
- var element = this.selectedTreeElement;
- element.updateSelection();
- },
-
_selectedNodeChanged: function()
{
this.dispatchEventToListeners(WebInspector.ElementsTreeOutline.Events.SelectedNodeChanged, this._selectedDOMNode);
@@ -754,7 +746,6 @@ WebInspector.ElementsTreeOutline.prototype = {
node = node.parentNode;
}
- treeElement.updateSelection();
treeElement.listItemElement.classList.add("elements-drag-over");
this._dragOverTreeElement = treeElement;
event.preventDefault();
@@ -833,7 +824,6 @@ WebInspector.ElementsTreeOutline.prototype = {
_clearDragOverTreeElementMarker: function()
{
if (this._dragOverTreeElement) {
- this._dragOverTreeElement.updateSelection();
this._dragOverTreeElement.listItemElement.classList.remove("elements-drag-over");
delete this._dragOverTreeElement;
}
@@ -1256,7 +1246,6 @@ WebInspector.ElementsTreeOutline.prototype = {
this._element.classList.remove("hidden");
if (originalScrollTop)
treeOutlineContainerElement.scrollTop = originalScrollTop;
- this.updateSelection();
}
this._updateRecords.clear();
@@ -1763,10 +1752,6 @@ WebInspector.ElementsTreeOutline.ShortcutTreeElement.prototype = {
this.listItemElement.classList.toggle("hovered", x);
},
- updateSelection: function()
- {
- },
-
/**
* @return {number}
*/
« no previous file with comments | « third_party/WebKit/Source/devtools/front_end/elements/ElementsTreeElement.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698