DevTools: fix pretty-print functionality for VM scripts

Source/devtools/front_end/ScriptFormatterEditorAction.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
  * @implements {WebInspector.SourceMapping}
  * @param {!WebInspector.Workspace} workspace
  * @param {!WebInspector.DebuggerModel} debuggerModel
  */
@@ skipping 55 matching lines @@
     {
         return false;
     },
 
     /**
      * @param {!WebInspector.UISourceCode} uiSourceCode
      * @return {!Array.<!WebInspector.Script>}
      */
     _scriptsForUISourceCode: function(uiSourceCode)
     {
-        var isInlineScript;
-        switch (uiSourceCode.contentType()) {
-        case WebInspector.resourceTypes.Document:
-            isInlineScript = true;
-            break;
-        case WebInspector.resourceTypes.Script:
-            isInlineScript = false;
-            break;
-        default:
-            return [];
+        if (uiSourceCode.contentType() === WebInspector.resourceTypes.Document)
+            return this._debuggerModel.scriptsForSourceURL(uiSourceCode.url);

[vsevik, 2014/03/26 09:02:34]

Please keep the inline filter.

[pfeldman, 2014/03/26 09:24:37]

But this does not add any value. Why would we?

+        if (uiSourceCode.contentType() === WebInspector.resourceTypes.Script) {
+            var rawLocation = uiSourceCode.uiLocationToRawLocation(0, 0);
+            return rawLocation ? [this._debuggerModel.scriptForId(rawLocation.scriptId)] : [];

[aandrey, 2014/03/25 19:33:54]

can return [undefined]

[pfeldman, 2014/03/25 20:01:40]

I don't think so - uiSourceCode here is using default debugging mapping. So it
exists as long as the corresponding script does.

         }
-        var scripts = this._debuggerModel.scriptsForSourceURL(uiSourceCode.url);
-
-        function filterOutIncorrectInlineType(script)
-        {
-            return script.isInlineScript() === isInlineScript;
-        }
-
-        return scripts.filter(filterOutIncorrectInlineType);
+        return [];
     },
 
     _init: function()
     {
         /** @type {!Map.<!WebInspector.Script, !WebInspector.UISourceCode>} */
         this._uiSourceCodes = new Map();
         /** @type {!StringMap.<string>} */
         this._formattedPaths = new StringMap();
         /** @type {!Map.<!WebInspector.UISourceCode, !WebInspector.FormatterScriptMapping.FormatData>} */
         this._formatData = new Map();
@@ skipping 275 matching lines @@
     },
 
     /**
      * @param {!WebInspector.UISourceCode} uiSourceCode
      */
     _discardFormattedUISourceCodeScript: function(uiSourceCode)
     {
         this._scriptMapping._discardFormattedUISourceCodeScript(uiSourceCode);
     }
 }