Bug 83787 - REGRESSION (r113910-113934): GMail displays a misplaced and dysfunctional Inbox Settings menu
Summary: REGRESSION (r113910-113934): GMail displays a misplaced and dysfunctional Inb...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac (Intel) OS X 10.7
: P1 Critical
Assignee: Nobody
URL:
Keywords: InRadar
Depends on: 81588
Blocks:
  Show dependency treegraph
 
Reported: 2012-04-12 08:30 PDT by R. Kevin Hill
Modified: 2012-04-15 00:06 PDT (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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).