Bug 157280 – Web Inspector: Unexpected "Script Element #" resources in Debugger Sidebar

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[Joseph Pecoraro]

* SUMMARY
Unexpected "Script Element #" resources in Debugger Sidebar

* STEPS TO REPRODUCE
1. Inspect apple.com
2. Reload
3. Show Debugger tab
  => Unexpected "Script Element 1" and 2 in the sidebar

[Joseph Pecoraro]

Created attachment 277934 [details]
[PATCH] Proposed Fix

[Timothy Hatcher]

Comment on attachment 277934 [details]
[PATCH] Proposed Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=277934&action=review

> Source/WebInspectorUI/UserInterface/Views/DebuggerSidebarPanel.js:456
> +        if (script.dynamicallyAddedScriptElement)
> +            return null;

Does this need to allow them with a sourceURL?

[Joseph Pecoraro]

(In reply to comment #2)
> Comment on attachment 277934 [details]
> [PATCH] Proposed Fix
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=277934&action=review
> 
> > Source/WebInspectorUI/UserInterface/Views/DebuggerSidebarPanel.js:456
> > +        if (script.dynamicallyAddedScriptElement)
> > +            return null;
> 
> Does this need to allow them with a sourceURL?

Yes, it probably should!! Good catch.

[Joseph Pecoraro]

Created attachment 277949 [details]
[PATCH] For Landing

[WebKit Commit Bot]

Comment on attachment 277949 [details]
[PATCH] For Landing

Clearing flags on attachment: 277949

Committed r200353: <http://trac.webkit.org/changeset/200353>