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

Unified Diff: Source/devtools/front_end/main/AdvancedApp.js

Issue 342163002: DevTools: move emulation button into the base App.js. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Review comments addressed. Created 6 years, 6 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 | « Source/devtools/front_end/inspector.css ('k') | Source/devtools/front_end/main/App.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/devtools/front_end/main/AdvancedApp.js
diff --git a/Source/devtools/front_end/main/AdvancedApp.js b/Source/devtools/front_end/main/AdvancedApp.js
index ac0314d4137b1c1a5697442ead516b3127f2bf6f..bd306b1e3f4927403ae66fc58384887fe7e78460 100644
--- a/Source/devtools/front_end/main/AdvancedApp.js
+++ b/Source/devtools/front_end/main/AdvancedApp.js
@@ -10,26 +10,9 @@ WebInspector.AdvancedApp = function()
{
WebInspector.App.call(this);
WebInspector.dockController.addEventListener(WebInspector.DockController.Events.BeforeDockSideChanged, this._openToolboxWindow, this);
-
- this._toggleEmulationButton = new WebInspector.StatusBarButton(WebInspector.UIString("Toggle emulation enabled."), "responsive-design-status-bar-item");
- this._toggleEmulationButton.toggled = WebInspector.overridesSupport.settings.emulationEnabled.get();
- this._toggleEmulationButton.addEventListener("click", this._toggleEmulationEnabled, this);
- WebInspector.overridesSupport.settings.emulationEnabled.addChangeListener(this._emulationEnabledChanged, this);
};
WebInspector.AdvancedApp.prototype = {
- _toggleEmulationEnabled: function()
- {
- WebInspector.overridesSupport.settings.emulationEnabled.set(!this._toggleEmulationButton.toggled);
- },
-
- _emulationEnabledChanged: function()
- {
- this._toggleEmulationButton.toggled = WebInspector.overridesSupport.settings.emulationEnabled.get();
- if (!WebInspector.experimentsSettings.responsiveDesign.isEnabled() && WebInspector.overridesSupport.settings.emulationEnabled.get())
- WebInspector.inspectorView.showViewInDrawer("emulation", true);
- },
-
createRootView: function()
{
var rootView = new WebInspector.RootView();
@@ -177,26 +160,6 @@ WebInspector.AdvancedApp.prototype = {
/**
* @constructor
- * @implements {WebInspector.StatusBarButton.Provider}
- */
-WebInspector.AdvancedApp.ResponsiveDesignButtonProvider = function()
-{
-}
-
-WebInspector.AdvancedApp.ResponsiveDesignButtonProvider.prototype = {
- /**
- * @return {?WebInspector.StatusBarButton}
- */
- button: function()
- {
- if (!(WebInspector.app instanceof WebInspector.AdvancedApp))
- return null;
- return /** @type {!WebInspector.AdvancedApp} */ (WebInspector.app)._toggleEmulationButton || null;
- }
-}
-
-/**
- * @constructor
*/
WebInspector.Toolbox = function()
{
« no previous file with comments | « Source/devtools/front_end/inspector.css ('k') | Source/devtools/front_end/main/App.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698