Bug 117893

Summary: Web Inspector: Uncaught exception when pretty printing WebInspector.Script
Product: WebKit Reporter: Joseph Pecoraro <joepeck>
Component: Web InspectorAssignee: Joseph Pecoraro <joepeck>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, graouts, joepeck, timothy, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
[PATCH] Proposed Fix none

Joseph Pecoraro
Reported 2013-06-21 14:13:38 PDT
* 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.
Attachments
[PATCH] Proposed Fix (1.77 KB, patch)
2013-06-21 14:17 PDT, Joseph Pecoraro 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2013-06-21 14:13:47 PDT
<rdar://problem/14233755>
Joseph Pecoraro
Comment 2 2013-06-21 14:15:57 PDT
Oh, the error is: [Error] ReferenceError: Can't find variable: script (Main.js, line 3527)
Joseph Pecoraro
Comment 3 2013-06-21 14:17:13 PDT
Created attachment 205216 [details] [PATCH] Proposed Fix Simple fix.
WebKit Commit Bot
Comment 4 2013-06-21 15:13:30 PDT
Comment on attachment 205216 [details] [PATCH] Proposed Fix Clearing flags on attachment: 205216 Committed r151861: <http://trac.webkit.org/changeset/151861>
WebKit Commit Bot
Comment 5 2013-06-21 15:13:32 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.