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

Unified Diff: Source/devtools/front_end/components/InspectElementModeController.js

Issue 1310273006: Devtools: convert pair of booleans in setInspectModeEnabled into enum parameter (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 5 years, 4 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: Source/devtools/front_end/components/InspectElementModeController.js
diff --git a/Source/devtools/front_end/components/InspectElementModeController.js b/Source/devtools/front_end/components/InspectElementModeController.js
index 87ef22f29ca9906747d771339adc647aae8ef8f3..29ea66f766bbec59b6ffd899aba1900297e67c12 100644
--- a/Source/devtools/front_end/components/InspectElementModeController.js
+++ b/Source/devtools/front_end/components/InspectElementModeController.js
@@ -58,7 +58,7 @@ WebInspector.InspectElementModeController.prototype = {
if (!this.enabled())
return;
var domModel = WebInspector.DOMModel.fromTarget(target);
- domModel.setInspectModeEnabled(true, WebInspector.moduleSetting("showUAShadowDOM").get());
+ domModel.setInspectMode(WebInspector.moduleSetting("showUAShadowDOM").get() ? DOMAgent.InspectMode.SearchForUAShadowDOM : DOMAgent.InspectMode.SearchForNormal);
},
/**
@@ -89,7 +89,7 @@ WebInspector.InspectElementModeController.prototype = {
this._toggleSearchButton.setToggled(enabled);
for (var domModel of WebInspector.DOMModel.instances())
- domModel.setInspectModeEnabled(enabled, WebInspector.moduleSetting("showUAShadowDOM").get());
+ domModel.setInspectMode(WebInspector.moduleSetting("showUAShadowDOM").get() ? DOMAgent.InspectMode.SearchForUAShadowDOM : DOMAgent.InspectMode.SearchForNormal);
dgozman 2015/08/29 01:36:40 Where is none?
sergeyv 2015/08/29 02:03:44 Done.
},
_suspendStateChanged: function()

Powered by Google App Engine
This is Rietveld 408576698