DevTools: enable by default disableAgentsWhenProfile experiment

Source/devtools/front_end/sdk/Target.js

 /*
  * Copyright 2014 The Chromium Authors. All rights reserved.
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 /**
  * @constructor
  * @extends {Protocol.Agents}
  * @param {string} name
@@ skipping 29 matching lines @@
     CanScreencast: "CanScreencast",
     HasTouchInputs: "HasTouchInputs",
     CanProfilePower: "CanProfilePower",
     CanInspectWorkers: "CanInspectWorkers",
     CanEmulate: "CanEmulate"
 }
 
 WebInspector.Target._nextId = 1;
 
 WebInspector.Target.prototype = {
+    suspend: function()
+    {
+        this.debuggerModel.suspendModel();

[yurys, 2014/10/15 14:07:34]

Why is this model listed before the experiment check?

[loislo, 2014/10/15 15:38:47]

Done.

+        if (!Runtime.experiments.isEnabled("disableAgentsWhenProfile"))
+            return;
+        this.cssModel.suspendModel();
+        this.domModel.suspendModel();
+    },
+
+    resume: function()
+    {
+        if (Runtime.experiments.isEnabled("disableAgentsWhenProfile")) {
+            this.domModel.resumeModel();
+            this.cssModel.resumeModel();
+        }
+        this.debuggerModel.resumeModel();
+    },
 
     /**
      * @return {number}
      */
     id: function()
     {
         return this._id;
     },
 
     /**
@@ skipping 205 matching lines @@
  */
 WebInspector.TargetManager = function()
 {
     WebInspector.Object.call(this);
     /** @type {!Array.<!WebInspector.Target>} */
     this._targets = [];
     /** @type {!Array.<!WebInspector.TargetManager.Observer>} */
     this._observers = [];
     /** @type {!Object.<string, !Array.<{modelClass: !Function, thisObject: (!Object|undefined), listener: function(!WebInspector.Event)}>>} */
     this._modelListeners = {};
+    /** @type {boolean} */
+    this._targetsSuspended = false;
 }
 
 WebInspector.TargetManager.Events = {
     InspectedURLChanged: "InspectedURLChanged",
     MainFrameNavigated: "MainFrameNavigated",
     Load: "Load",
-    WillReloadPage: "WillReloadPage"
+    WillReloadPage: "WillReloadPage",
+    SuspendStateChanged: "SuspendStateChanged"
 }
 
 WebInspector.TargetManager.prototype = {
+    suspendAllTargets: function()
+    {
+        console.assert(!this._targetsSuspended);
+        if (this._targetsSuspended)
+            return;
+        this._targetsSuspended = true;
+        this._targets.forEach(function(target) {

[yurys, 2014/10/15 14:07:34]

style: { on a separate line.

[loislo, 2014/10/15 15:38:47]

Done.

+            target.suspend();
+        });
+        this.dispatchEventToListeners(WebInspector.TargetManager.Events.SuspendStateChanged);
+    },
+
+    resumeAllTargets: function()
+    {
+        console.assert(this._targetsSuspended);
+        if (!this._targetsSuspended)
+            return;
+        this._targetsSuspended = false;
+        this._targets.forEach(function(target) {

[yurys, 2014/10/15 14:07:34]

ditto

[loislo, 2014/10/15 15:38:47]

Done.

+            target.resume();
+        });
+        this.dispatchEventToListeners(WebInspector.TargetManager.Events.SuspendStateChanged);
+    },
+
+    /**
+     * @return {boolean}
+     */
+    areTargetsSuspended: function()
+    {
+        return this._targetsSuspended;
+    },
+
     /**
      * @return {string}
      */
     inspectedPageURL: function()
     {
         if (!this._targets.length)
             return "";
 
         return this._targets[0].resourceTreeModel.inspectedPageURL();
     },
@@ skipping 200 matching lines @@
     /**
      * @param {!WebInspector.Target} target
      */
     targetRemoved: function(target) { },
 }
 
 /**
  * @type {!WebInspector.TargetManager}
  */
 WebInspector.targetManager = new WebInspector.TargetManager();