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

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

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

Status:	RESOLVED DUPLICATE of bug 87526

Alias:	None

Product:	WebKit
Classification:	Unclassified
Component:	WebCore Misc. (show other bugs)
Version:	528+ (Nightly build)
Hardware:	Unspecified Unspecified

Importance:	P1 Normal
Assignee:	Nobody

URL:
Keywords:

Depends on:
Blocks:	10900
	Show dependency tree / graph

Reported:	2012-05-24 22:01 PDT by Ryosuke Niwa
Modified:	2012-05-25 17:30 PDT (History)
CC List:	16 users (show)

See Also:

Attachments
expected (93.29 KB, image/png) 2012-05-24 22:02 PDT, Ryosuke Niwa	no flags	Details
actual (79.95 KB, image/png) 2012-05-24 22:02 PDT, Ryosuke Niwa	no flags	Details
reduction (301 bytes, text/html) 2012-05-24 23:25 PDT, Ryosuke Niwa	no flags	Details
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

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 ***