Bug 83787

Summary: REGRESSION (r113910-113934): GMail displays a misplaced and dysfunctional Inbox Settings menu
Product: WebKit Reporter: R. Kevin Hill <rkevinhill>
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Critical CC: ap, arv, fpizlo, koivisto, levin
Priority: P1 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Mac (Intel)   
OS: OS X 10.7   
Bug Depends on: 81588    
Bug Blocks:    

Description R. Kevin Hill 2012-04-12 08:30:58 PDT
Returns Inbox display to "invitation to new inbox" mode ("Try out all of the new inbox styles" etc.), none of the links of which work; "Settings" button does not work. Previous Webkits and current Safari do not do this. Not a programmer; my first bug report, guessing about the component.
Comment 1 Alexey Proskuryakov 2012-04-12 14:09:09 PDT
Steps to reproduce: just log in to GMail.

There are many JSC changes in this range, and one CSS change. Starting with JSC.
Comment 2 Alexey Proskuryakov 2012-04-12 14:09:25 PDT
<rdar://problem/11239404>
Comment 3 Antti Koivisto 2012-04-12 18:03:48 PDT
Does it occur in Chromium? That should help identifying if this is a CSS or JSC problem.
Comment 4 Filip Pizlo 2012-04-12 18:06:23 PDT
(In reply to comment #3)
> Does it occur in Chromium? That should help identifying if this is a CSS or JSC problem.

It appears to not be your patch!  I just checked that by merging your patch over r113909 and gmail still works.

On the other hand, disabling all JSC optimizations doesn't fix it, either.

So it's not clear who's at fault. :-/
Comment 6 Erik Arvidsson 2012-04-13 12:31:01 PDT
This is a bug in JSC. JSC does not support ES5 Array.prototype.toString correctly.
Comment 7 Filip Pizlo 2012-04-13 17:32:58 PDT
Verifying whether this is fixed now.
Comment 8 David Levin 2012-04-14 20:51:46 PDT
This was broken in Chromium as well.

The breaking change was rolled out here: http://trac.webkit.org/changeset/114200
Comment 9 Alexey Proskuryakov 2012-04-15 00:06:05 PDT
Marking fixed per the above comment (I also verified that it doesn't fail for me with r114211).