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

Unified Diff: third_party/WebKit/Source/devtools/front_end/main/Main.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/main/Main.js
diff --git a/third_party/WebKit/Source/devtools/front_end/main/Main.js b/third_party/WebKit/Source/devtools/front_end/main/Main.js
index 840462fe6e8b98384c88b435519230fab812ad7d..2aa82f788e0ea948e5cacd1ae07ea56cf617914f 100644
--- a/third_party/WebKit/Source/devtools/front_end/main/Main.js
+++ b/third_party/WebKit/Source/devtools/front_end/main/Main.js
@@ -197,7 +197,6 @@ Main.Main = class {
Persistence.persistence =
new Persistence.Persistence(Workspace.workspace, Bindings.breakpointManager, Workspace.fileSystemMapping);
- new Main.OverlayController();
new Main.ExecutionContextSelector(SDK.targetManager, UI.context);
Bindings.blackboxManager = new Bindings.BlackboxManager(Bindings.debuggerWorkspaceBinding);
@@ -797,7 +796,8 @@ Main.Main.PauseListener = class {
*/
Main.Main.InspectedNodeRevealer = class {
constructor() {
- SDK.targetManager.addModelListener(SDK.DOMModel, SDK.DOMModel.Events.NodeInspected, this._inspectNode, this);
+ SDK.targetManager.addModelListener(
+ SDK.OverlayModel, SDK.OverlayModel.Events.InspectNodeRequested, this._inspectNode, this);
}
/**
@@ -933,7 +933,6 @@ Main.BackendSettingsSync = class {
*/
targetAdded(target) {
this._updateTarget(target);
- target.renderingAgent().setShowViewportSizeOnResize(true);
}
/**

Powered by Google App Engine
This is Rietveld 408576698