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

Unified Diff: third_party/WebKit/Source/devtools/front_end/resources/ResourcesSection.js

Issue 2819183002: [DevTools] Consolidate overlay-related functionality in Overlay domain (Closed)
Patch Set: rebased bad merge Created 3 years, 8 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/resources/ResourcesSection.js
diff --git a/third_party/WebKit/Source/devtools/front_end/resources/ResourcesSection.js b/third_party/WebKit/Source/devtools/front_end/resources/ResourcesSection.js
index 047cb83789337f3bd39aa42a3f86ecacef7d55fe..2ffe0362b3565c45d25ea1873b7eeabf146469d9 100644
--- a/third_party/WebKit/Source/devtools/front_end/resources/ResourcesSection.js
+++ b/third_party/WebKit/Source/devtools/front_end/resources/ResourcesSection.js
@@ -155,18 +155,17 @@ Resources.FrameTreeElement = class extends Resources.BaseStorageTreeElement {
this._panel.showCategoryView(this.titleAsText());
this.listItemElement.classList.remove('hovered');
- SDK.DOMModel.hideDOMNodeHighlight();
+ SDK.OverlayModel.hideDOMNodeHighlight();
return false;
}
set hovered(hovered) {
if (hovered) {
this.listItemElement.classList.add('hovered');
- var domModel = this._frame.resourceTreeModel().domModel();
- domModel.highlightFrame(this._frameId);
+ this._frame.resourceTreeModel().domModel().overlayModel().highlightFrame(this._frameId);
} else {
this.listItemElement.classList.remove('hovered');
- SDK.DOMModel.hideDOMNodeHighlight();
+ SDK.OverlayModel.hideDOMNodeHighlight();
}
}

Powered by Google App Engine
This is Rietveld 408576698