Bug 77424

Summary: [REGRESSION] Web Inspector: Can't open Web Inspector
Product: WebKit Reporter: Philippe Wittenbergh <phiw2>
Component: Web Inspector (Deprecated)Assignee: Timothy Hatcher <timothy>
Status: RESOLVED FIXED    
Severity: Normal CC: apavlov, ap, bweinstein, craigyk, joel, joepeck, jon, keishi, knavia, loislo, pfeldman, pmuellr, rik, timothy, webkit.review.bot, yurys
Priority: P1    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Proposed Change none

Philippe Wittenbergh
Reported 2012-01-31 04:24:42 PST
Neither Cmd-option-I, nor the menu item, nor a right click > Inspect element work. (the Menu item just blinks but nothing happens) Regressed between r106204 and r106264
Attachments
Proposed Change (2.33 KB, patch)
2012-02-01 13:10 PST, Timothy Hatcher 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Philippe Wittenbergh
Comment 1 2012-01-31 04:25:35 PST
This is on OS X Lion 10.7.2, btw
Jon
Comment 2 2012-01-31 15:38:19 PST
I'm seeing this as well with the r106354 nightly and ToT.
Ryosuke Niwa
Comment 3 2012-01-31 15:38:51 PST
All regressions are P1.
Timothy Hatcher
Comment 4 2012-01-31 19:43:24 PST
This only happens in Release builds because the concatenated JavaScript file is failing with a syntax error. file:///Users/Timothy/Work/Safari-TOT.git/OpenSource/WebKitBuild/Release/WebCore.framework/Resources/inspector/inspector.js:61427: JS ERROR: SyntaxError: Unexpected token ',' The parse error might happen in Debug builds to, but with less severe results because only a single file would fail and it might not be a critical one and the Inspector would continue to load.
Timothy Hatcher
Comment 5 2012-02-01 10:09:22 PST
*** Bug 77559 has been marked as a duplicate of this bug. ***
Timothy Hatcher
Comment 6 2012-02-01 10:35:56 PST
*** Bug 77561 has been marked as a duplicate of this bug. ***
Timothy Hatcher
Comment 7 2012-02-01 12:37:32 PST
*** Bug 77570 has been marked as a duplicate of this bug. ***
Timothy Hatcher
Comment 8 2012-02-01 13:03:16 PST
In debug it is this: file:///Users/Timothy/Work/Safari-TOT.git/OpenSource/WebKitBuild/Release/WebCore.framework/Resources/inspector/DetailedHeapshotView.js:966: JS ERROR: SyntaxError: Unexpected token ',' This is a file Safari does not use, but the error is causing problems in release. I have a fix.
Timothy Hatcher
Comment 9 2012-02-01 13:10:59 PST
Created attachment 124999 [details] Proposed Change
WebKit Review Bot
Comment 10 2012-02-01 15:08:30 PST
Comment on attachment 124999 [details] Proposed Change Clearing flags on attachment: 124999 Committed r106501: <http://trac.webkit.org/changeset/106501>
WebKit Review Bot
Comment 11 2012-02-01 15:08:35 PST
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.