DevTools: enable by default disableAgentsWhenProfile experiment

Source/devtools/front_end/sdk/DebuggerModel.js

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 35 matching lines @@
     this._scripts = {};
     /** @type {!StringMap.<!Array.<!WebInspector.Script>>} */
     this._scriptsBySourceURL = new StringMap();
 
     /** @type {!WebInspector.Object} */
     this._breakpointResolvedEventTarget = new WebInspector.Object();
 
     this._isPausing = false;
     WebInspector.settings.pauseOnExceptionEnabled.addChangeListener(this._pauseOnExceptionStateChanged, this);
     WebInspector.settings.pauseOnCaughtException.addChangeListener(this._pauseOnExceptionStateChanged, this);
-
     WebInspector.settings.enableAsyncStackTraces.addChangeListener(this._asyncStackTracesStateChanged, this);
-    WebInspector.profilingLock().addEventListener(WebInspector.Lock.Events.StateChanged, this._profilingStateChanged, this);
+    WebInspector.settings.skipStackFramesPattern.addChangeListener(this._applySkipStackFrameSettings, this);
+    WebInspector.settings.skipContentScripts.addChangeListener(this._applySkipStackFrameSettings, this);
 
     this.enableDebugger();
 
-    WebInspector.settings.skipStackFramesPattern.addChangeListener(this._applySkipStackFrameSettings, this);
-    WebInspector.settings.skipContentScripts.addChangeListener(this._applySkipStackFrameSettings, this);
     this._applySkipStackFrameSettings();
 }
 
 /** @typedef {{location: ?WebInspector.DebuggerModel.Location, sourceURL: ?string, functionName: string, scopeChain: (Array.<!DebuggerAgent.Scope>|null)}} */
 WebInspector.DebuggerModel.FunctionDetails;
 
 /**
  * Keep these in sync with WebCore::ScriptDebugServer
  *
  * @enum {string}
@@ skipping 47 matching lines @@
     },
 
     disableDebugger: function()
     {
         if (!this._debuggerEnabled)
             return;
 
         this._agent.disable();
         this._debuggerEnabled = false;
         this._isPausing = false;
+        this._asyncStackTracesStateChanged();
         this.dispatchEventToListeners(WebInspector.DebuggerModel.Events.DebuggerWasDisabled);
     },
 
     /**
      * @param {boolean} skip
      * @param {boolean=} untilReload
      */
     skipAllPauses: function(skip, untilReload)
     {
         if (this._skipAllPausesTimeout) {
@@ skipping 21 matching lines @@
         if (!WebInspector.settings.pauseOnExceptionEnabled.get()) {
             state = WebInspector.DebuggerModel.PauseOnExceptionsState.DontPauseOnExceptions;
         } else if (WebInspector.settings.pauseOnCaughtException.get()) {
             state = WebInspector.DebuggerModel.PauseOnExceptionsState.PauseOnAllExceptions;
         } else {
             state = WebInspector.DebuggerModel.PauseOnExceptionsState.PauseOnUncaughtExceptions;
         }
         this._agent.setPauseOnExceptions(state);
     },
 
-    _profilingStateChanged: function()
+    suspendModel: function()
     {
-        if (Runtime.experiments.isEnabled("disableAgentsWhenProfile")) {
-            if (WebInspector.profilingLock().isAcquired())
-                this.disableDebugger();
-            else
-                this.enableDebugger();
-        }
-        this._asyncStackTracesStateChanged();
+        this.disableDebugger();
+    },
+
+    resumeModel: function()
+    {
+        this.enableDebugger();
     },
 
     _asyncStackTracesStateChanged: function()
     {
         const maxAsyncStackChainDepth = 4;
-        var enabled = WebInspector.settings.enableAsyncStackTraces.get() && !WebInspector.profilingLock().isAcquired();
+        var enabled = WebInspector.settings.enableAsyncStackTraces.get() && !WebInspector.targetManager.areTargetsSuspended();
         this._agent.setAsyncCallStackDepth(enabled ? maxAsyncStackChainDepth : 0);
     },
 
     stepInto: function()
     {
         /**
          * @this {WebInspector.DebuggerModel}
          */
         function callback()
         {
@@ skipping 905 matching lines @@
         return this.target().runtimeModel.createRemoteObject(/** @type {!RuntimeAgent.RemoteObject} */(this.auxData));
     },
 
     __proto__: WebInspector.SDKObject.prototype
 }
 
 /**
  * @type {!WebInspector.DebuggerModel}
  */
 WebInspector.debuggerModel;