Bug 35253 - Web Inspector: Regression: r55027+: Inspector broken
Summary: Web Inspector: Regression: r55027+: Inspector broken
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac (PowerPC) OS X 10.5
: P2 Normal
Assignee: Oliver Hunt
URL:
Keywords: InRadar, Regression
: 35237 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-02-22 11:55 PST by Kevin M. Dean
Modified: 2010-02-23 20:28 PST (History)
2 users (show)

See Also:


Attachments
Screenshot (18.20 KB, image/png)
2010-02-22 12:19 PST, Kevin M. Dean
no flags Details
Patch (7.27 KB, patch)
2010-02-23 20:23 PST, Oliver Hunt
mjs: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kevin M. Dean 2010-02-22 11:55:12 PST
PPC, 10.5.8, when using r55027+. If I try to open the Web Inspector with Command-Option-I or Inspect Element, the first time nothing happens and the second time the pane opens and is completely blank.
Comment 1 Kevin M. Dean 2010-02-22 12:19:01 PST
Created attachment 49231 [details]
Screenshot

Screenshot
Comment 2 Alexey Proskuryakov 2010-02-23 11:36:27 PST
*** Bug 35237 has been marked as a duplicate of this bug. ***
Comment 3 Alexey Proskuryakov 2010-02-23 11:37:19 PST
Per bug 35253, Yahoo Mail got broken on PowerPC in the same timeframe.
Comment 4 Alexey Proskuryakov 2010-02-23 11:37:47 PST
<rdar://problem/7680864>
Comment 5 Oliver Hunt 2010-02-23 14:06:24 PST
This is probably fallout from my getter caching change.  Looking into it now.
Comment 6 Oliver Hunt 2010-02-23 14:48:44 PST
Okay the problem is that something is going wrong when performing the call to a cached getter from the prototype chain.

Unsure of exactly what is going wrong, by code inspection it would appear to be correct, but we seem to be picking up the wrong this object :-/
Comment 7 Oliver Hunt 2010-02-23 20:23:53 PST
Created attachment 49354 [details]
Patch
Comment 8 Maciej Stachowiak 2010-02-23 20:25:51 PST
Comment on attachment 49354 [details]
Patch

r=me
Comment 9 Oliver Hunt 2010-02-23 20:28:09 PST
Committed r55181