Bug 87461

Summary:

REGRESSION(r118395): Message doesn't show up on Gmail when preview pane is enabled

Product:

WebKit

Reporter:

Ryosuke Niwa <rniwa>

Component:

WebCore Misc.

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED DUPLICATE

Severity:

Normal

CC:

adele, andersca, ap, eric, fpizlo, hyatt, inferno, jamesr, jchaffraix, jcivelli, jsbell, ojan, robert, simon.fraser, tony, vsevik

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

Bug Blocks:

10900

Attachments:

Description	Flags
expected	none
actual	none
reduction	none

Description Ryosuke Niwa 2012-05-24 22:01:15 PDT

Reproduction steps:
1. Go to GMail.com
2. Enable "Review Pane" on Gmail Labs
3. Select "Vertical split" from the right upper corner icon with three bars

Comment 1 Ryosuke Niwa 2012-05-24 22:02:23 PDT

Created attachment 143973 [details]
expected

Comment 2 Ryosuke Niwa 2012-05-24 22:02:49 PDT

Created attachment 143974 [details]
actual

Comment 3 Filip Pizlo 2012-05-24 22:06:57 PDT

Did you really mean http://trac.webkit.org/changeset/118466?

Comment 4 Ryosuke Niwa 2012-05-24 22:07:58 PDT

(In reply to comment #3)
> Did you really mean http://trac.webkit.org/changeset/118466?

Sorry, copy & paste error. The regression range is between the two nightly builds.

Comment 5 Ryosuke Niwa 2012-05-24 22:10:12 PDT

Maybe http://trac.webkit.org/changeset/118413 or http://trac.webkit.org/changeset/118416 ?

Comment 6 Filip Pizlo 2012-05-24 22:11:37 PDT

(In reply to comment #5)
> Maybe http://trac.webkit.org/changeset/118413 or http://trac.webkit.org/changeset/118416 ?

More likely http://trac.webkit.org/changeset/118416 than http://trac.webkit.org/changeset/118413

http://trac.webkit.org/changeset/118413 doesn't seem like it would affect x86, but I could be wrong.

Comment 7 Ryosuke Niwa 2012-05-24 22:13:00 PDT

(In reply to comment #6)
> (In reply to comment #5)
> > Maybe http://trac.webkit.org/changeset/118413 or http://trac.webkit.org/changeset/118416 ?
> 
> More likely http://trac.webkit.org/changeset/118416 than http://trac.webkit.org/changeset/118413
> 
> http://trac.webkit.org/changeset/118413 doesn't seem like it would affect x86, but I could be wrong.

Ah! This isn't a JSC regression. I can see the same regression on Chromium's latest trunk build.

Comment 8 Ryosuke Niwa 2012-05-24 22:13:38 PDT

This is a DOM or rendering regression.

Comment 9 Ryosuke Niwa 2012-05-24 22:23:31 PDT

Chromium r138925: bad! -> WebKit r118406
Chromium r138884: good! -> WebKit r118384
See http://src.chromium.org/viewvc/chrome/trunk/src/DEPS?view=log

Comment 10 Ryosuke Niwa 2012-05-24 22:25:22 PDT

Regression range:
http://trac.webkit.org/log/?action=stop_on_copy&mode=stop_on_copy&rev=118406&stop_rev=118394&limit=100&verbose=on

Maybe http://trac.webkit.org/changeset/118395/ ?

Comment 11 Ryosuke Niwa 2012-05-24 22:57:55 PDT

Confirmed.

Comment 12 Ryosuke Niwa 2012-05-24 23:25:25 PDT

Created attachment 143986 [details]
reduction

Comment 13 Alexey Proskuryakov 2012-05-25 11:43:10 PDT

Yahoo mail is also broken by the same revision, bug 87508.

Comment 14 Joshua Bell 2012-05-25 13:27:54 PDT

Reverted in http://trac.webkit.org/changeset/118549

Comment 15 Alexey Proskuryakov 2012-05-25 17:30:15 PDT


*** This bug has been marked as a duplicate of bug 87526 ***