DevTools: simplify setScriptSource and restartFrame return values

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 711 matching lines @@
             else
                 callback(this.target().runtimeModel.createRemoteObject(result), !!wasThrown, undefined, exceptionDetails);
 
             if (objectGroup === "console")
                 this.dispatchEventToListeners(WebInspector.DebuggerModel.Events.ConsoleCommandEvaluatedInSelectedCallFrame);
         }
 
         this.selectedCallFrame().evaluate(code, objectGroup, includeCommandLineAPI, doNotPauseOnExceptionsAndMuteConsole, returnByValue, generatePreview, didEvaluate.bind(this));
     },
 
-    /**
-     * Handles notification from JavaScript VM about updated stack (liveedit or frame restart action).
-     * @param {!Array.<!DebuggerAgent.CallFrame>=} newCallFrames
-     * @param {!Object=} details
-     * @param {!DebuggerAgent.StackTrace=} asyncStackTrace
-     */
-    callStackModified: function(newCallFrames, details, asyncStackTrace)
-    {
-        // FIXME: declare this property in protocol and in JavaScript.
-        if (details && details["stack_update_needs_step_in"])
-            this.stepInto();
-        else if (newCallFrames && newCallFrames.length)
-            this._pausedScript(newCallFrames, this._debuggerPausedDetails.reason, this._debuggerPausedDetails.auxData, this._debuggerPausedDetails.breakpointIds, asyncStackTrace);
-    },
-
     _applySkipStackFrameSettings: function()
     {
         this._agent.skipStackFrames(WebInspector.moduleSetting("skipStackFramesPattern").get(), WebInspector.moduleSetting("skipContentScripts").get());
     },
 
     /**
      * @param {!WebInspector.RemoteObject} remoteObject
      * @param {function(?WebInspector.DebuggerModel.FunctionDetails)} callback
      */
     functionDetails: function(remoteObject, callback)
@@ skipping 517 matching lines @@
     },
 
     /**
      * @param {function(?Protocol.Error=)=} callback
      */
     restart: function(callback)
     {
         /**
          * @param {?Protocol.Error} error
          * @param {!Array.<!DebuggerAgent.CallFrame>=} callFrames
-         * @param {!Object=} details
          * @param {!DebuggerAgent.StackTrace=} asyncStackTrace
          * @this {WebInspector.DebuggerModel.CallFrame}
          */
-        function protocolCallback(error, callFrames, details, asyncStackTrace)
+        function protocolCallback(error, callFrames, asyncStackTrace)
         {
             if (!error)
-                this.debuggerModel.callStackModified(callFrames, details, asyncStackTrace);
+                this.debuggerModel.stepInto();
             if (callback)
                 callback(error);
         }
         this._debuggerAgent.restartFrame(this._payload.callFrameId, protocolCallback.bind(this));
     },
 
     /**
      * @param {function(!Object)} callback
      */
     variableNames: function(callback)
@@ skipping 152 matching lines @@
 /**
  * @param {?WebInspector.Target} target
  * @return {?WebInspector.DebuggerModel}
  */
 WebInspector.DebuggerModel.fromTarget = function(target)
 {
     if (!target || !target.hasJSContext())
         return null;
     return /** @type {?WebInspector.DebuggerModel} */ (target.model(WebInspector.DebuggerModel));
 }