Changeset 244363 in webkit

Timestamp:

Apr 16, 2019, 4:44:07 PM (7 years ago)

Author:

Joseph Pecoraro

Message:

Web Inspector: Worker imported scripts not showing up in Open Quickly dialog if inspector open after workers exist
​https://bugs.webkit.org/show_bug.cgi?id=196986

Reviewed by Devin Rousso.

• UserInterface/Views/OpenResourceDialog.js:

(WI.OpenResourceDialog.prototype._addScriptsForTarget):
We were unnecessary skipping Scripts that do not have a sourceURL but do
have a URL, which ends up being Worker populated scripts.

Location:

trunk/Source/WebInspectorUI

Files:

• ChangeLog (modified) (1 diff)
• UserInterface/Views/OpenResourceDialog.js (modified) (1 diff)

trunk/Source/WebInspectorUI/ChangeLog

@@ +1 @@
+2019-04-16  Joseph Pecoraro  <pecoraro@apple.com>
+
+        Web Inspector: Worker imported scripts not showing up in Open Quickly dialog if inspector open after workers exist
+        https://bugs.webkit.org/show_bug.cgi?id=196986
+
+        Reviewed by Devin Rousso.
+
+        * UserInterface/Views/OpenResourceDialog.js:
+        (WI.OpenResourceDialog.prototype._addScriptsForTarget):
+        We were unnecessary skipping Scripts that do not have a sourceURL but do
+        have a URL, which ends up being Worker populated scripts.
+
 2019-04-16  Joseph Pecoraro  <pecoraro@apple.com>
 

trunk/Source/WebInspectorUI/UserInterface/Views/OpenResourceDialog.js

@@ -328 +328 @@
             if (script.dynamicallyAddedScriptElement)
                 continue;
-            if (!script.sourceURL || isWebKitInternalScript(script.sourceURL) || isWebInspectorConsoleEvaluationScript(script.sourceURL))
+            if (isWebKitInternalScript(script.sourceURL) || isWebInspectorConsoleEvaluationScript(script.sourceURL))
                 continue;
             this._addResource(script, suppressFilterUpdate);