Bug 117893 – Web Inspector: Uncaught exception when pretty printing WebInspector.Script

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

[Joseph Pecoraro]

* DESCRIPTION
I hit this debugging a page. I introduced a JS issue recently when I refactored some code to use WebInspector.script.resource. I removed a line declaring a local variable and didn't fix the other lines using the local.

* STEPS TO REPRODUCE
1. Evaluation the following in the console:
// testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing testing 
(function() { return 1+1; })(); //@ sourceURL=test
2. View the Extra Script in the inspector
  => it pretty prints and produces an uncaught exception

Patch to follow.

[Radar WebKit Bug Importer]

<rdar://problem/14233755>

[Joseph Pecoraro]

Oh, the error is:
[Error] ReferenceError: Can't find variable: script (Main.js, line 3527)

[Joseph Pecoraro]

Created attachment 205216 [details]
[PATCH] Proposed Fix

Simple fix.

[WebKit Commit Bot]

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

Clearing flags on attachment: 205216

Committed r151861: <http://trac.webkit.org/changeset/151861>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.