Web Inspector: "loadScript" function to load scripts via xhr
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 28 Feb 2013 14:53:35 +0000 (14:53 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 28 Feb 2013 14:53:35 +0000 (14:53 +0000)
https://bugs.webkit.org/show_bug.cgi?id=110879

Patch by Andrey Lushnikov <lushnikov@chromium.org> on 2013-02-28
Reviewed by Pavel Feldman.

Create an alias for "importScript" function and use it in those cases
which assume lazy script loading.

No new tests.

* inspector/front-end/Panel.js:
(WebInspector.PanelDescriptor.prototype.panel):
* inspector/front-end/SourceFrame.js:
(WebInspector.SourceFrame):
* inspector/front-end/utilities.js:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@144305 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/inspector/front-end/Panel.js
Source/WebCore/inspector/front-end/SourceFrame.js
Source/WebCore/inspector/front-end/utilities.js

index 394b05e..e078995 100644 (file)
@@ -1,3 +1,21 @@
+2013-02-28  Andrey Lushnikov  <lushnikov@chromium.org>
+
+        Web Inspector: "loadScript" function to load scripts via xhr
+        https://bugs.webkit.org/show_bug.cgi?id=110879
+
+        Reviewed by Pavel Feldman.
+
+        Create an alias for "importScript" function and use it in those cases
+        which assume lazy script loading.
+
+        No new tests.
+
+        * inspector/front-end/Panel.js:
+        (WebInspector.PanelDescriptor.prototype.panel):
+        * inspector/front-end/SourceFrame.js:
+        (WebInspector.SourceFrame):
+        * inspector/front-end/utilities.js:
+
 2013-02-28  Alexei Filippov  <alph@chromium.org>
 
         Web Inspector: Move profiler tools into separate panels
index 1bc485c..314a12b 100644 (file)
@@ -328,7 +328,7 @@ WebInspector.PanelDescriptor.prototype = {
         if (this._panel)
             return this._panel;
         if (this._scriptName)
-            importScript(this._scriptName);
+            loadScript(this._scriptName);
         this._panel = new WebInspector[this._className];
         return this._panel;
     },
index 5313100..dd02902 100644 (file)
@@ -44,7 +44,7 @@ WebInspector.SourceFrame = function(contentProvider)
     var textEditorDelegate = new WebInspector.TextEditorDelegateForSourceFrame(this);
 
     if (WebInspector.experimentsSettings.codemirror.isEnabled()) {
-        importScript("CodeMirrorTextEditor.js");
+        loadScript("CodeMirrorTextEditor.js");
         this._textEditor = new WebInspector.CodeMirrorTextEditor(this._url, textEditorDelegate);
     } else
         this._textEditor = new WebInspector.DefaultTextEditor(this._url, textEditorDelegate);
index 84c38f7..09584e2 100644 (file)
@@ -968,6 +968,12 @@ StringPool.prototype = {
 var _importedScripts = {};
 
 /**
+ * This function behavior depends on the "debug_devtools" flag value.
+ * - In debug mode it loads scripts synchronously via xhr request.
+ * - In release mode every occurrence of "importScript" gets replaced with
+ * the script source code on the compilation phase.
+ *
+ * To load scripts lazily in release mode call "loasScript" function.
  * @param {string} scriptName
  */
 function importScript(scriptName)
@@ -981,3 +987,5 @@ function importScript(scriptName)
     var sourceURL = WebInspector.ParsedURL.completeURL(window.location.href, scriptName); 
     window.eval(xhr.responseText + "\n//@ sourceURL=" + sourceURL);
 }
+
+var loadScript = importScript;