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

Unified Diff: Source/devtools/front_end/sdk/OverridesSupport.js

Issue 306803004: DevTools: hide emulation bits that are available in responsive design toolbar from the drawer. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 7 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/overrides.css ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/devtools/front_end/sdk/OverridesSupport.js
diff --git a/Source/devtools/front_end/sdk/OverridesSupport.js b/Source/devtools/front_end/sdk/OverridesSupport.js
index e9217314b2589c1a7f1d2f782ac9823d692c85ee..2b378e9891798b401bf0776c1ce7dd5a1ba58dfd 100644
--- a/Source/devtools/front_end/sdk/OverridesSupport.js
+++ b/Source/devtools/front_end/sdk/OverridesSupport.js
@@ -32,8 +32,9 @@
* @constructor
* @implements {WebInspector.TargetManager.Observer}
* @extends {WebInspector.Object}
+ * @param {boolean} responsiveDesignAvailable
*/
-WebInspector.OverridesSupport = function()
+WebInspector.OverridesSupport = function(responsiveDesignAvailable)
{
WebInspector.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.MainFrameNavigated, this._onMainFrameNavigated.bind(this), this);
this._overrideDeviceResolution = false;
@@ -41,6 +42,7 @@ WebInspector.OverridesSupport = function()
this._userAgent = "";
this._pageResizer = null;
WebInspector.targetManager.observeTargets(this);
+ this._responsiveDesignAvailable = responsiveDesignAvailable;
}
WebInspector.OverridesSupport.Events = {
@@ -399,6 +401,14 @@ WebInspector.OverridesSupport._tablets = [
WebInspector.OverridesSupport.prototype = {
/**
+ * @return {boolean}
+ */
+ responsiveDesignAvailable: function()
+ {
+ return this._responsiveDesignAvailable;
+ },
+
+ /**
* @param {?WebInspector.OverridesSupport.PageResizer} pageResizer
*/
setPageResizer: function(pageResizer)
@@ -523,9 +533,9 @@ WebInspector.OverridesSupport.prototype = {
if (this._deviceMetricsChangedListenerMuted)
return;
-
+ var responsiveDesignAvailableAndDisabled = this._responsiveDesignAvailable && !WebInspector.settings.responsiveDesignMode.get();
var overrideDeviceResolution = this.settings.overrideDeviceResolution.get();
- if (!overrideDeviceResolution && !this.settings.emulateViewport.get()) {
+ if (responsiveDesignAvailableAndDisabled || (!overrideDeviceResolution && !this.settings.emulateViewport.get())) {
PageAgent.clearDeviceMetricsOverride(apiCallback.bind(this));
if (this._pageResizer)
this._pageResizer.update(0, 0, 0);
« no previous file with comments | « Source/devtools/front_end/overrides.css ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698