Bug 108820

Summary:

[WK2][EFL] Weird stripe at the end of the page

Product:

WebKit

Reporter:

Mikhail Pozdnyakov <mikhail.pozdnyakov>

Component:

WebKit EFL

Assignee:

Mikhail Pozdnyakov <mikhail.pozdnyakov>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

andersca, benjamin, kenneth, lucas.de.marchi, noam, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

Bug Blocks:

105187, 103105

Attachments:

Description	Flags
screen shot with 'weird stripe'	none
treatment	none

Mikhail Pozdnyakov

Reported 2013-02-04 06:11:40 PST

There is a weird stripe at the end of the page when the page is scrolled down.

Attachments
screen shot with 'weird stripe' (76.28 KB, image/png) 2013-02-04 06:12 PST, Mikhail Pozdnyakov	no flags	Details
treatment (1.76 KB, patch) 2013-02-04 06:20 PST, Mikhail Pozdnyakov	no flags	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Mikhail Pozdnyakov

Comment 1 2013-02-04 06:12:40 PST

Created attachment 186363 [details] screen shot with 'weird stripe'

Mikhail Pozdnyakov

Comment 2 2013-02-04 06:20:08 PST

Created attachment 186366 [details] treatment

Kenneth Rohde Christiansen

Comment 3 2013-02-04 06:25:43 PST

Comment on attachment 186366 [details] treatment LGTM

Noam Rosenthal

Comment 4 2013-02-04 06:33:41 PST

Comment on attachment 186366 [details] treatment This is a trivial fix; I think it's ok to r+.

WebKit Review Bot

Comment 5 2013-02-04 07:01:46 PST

Comment on attachment 186366 [details] treatment Clearing flags on attachment: 186366 Committed r141770: <http://trac.webkit.org/changeset/141770>

WebKit Review Bot

Comment 6 2013-02-04 07:01:51 PST

All reviewed patches have been landed. Closing bug.

Anders Carlsson

Comment 7 2013-02-04 14:05:26 PST

(In reply to comment #4) > (From update of attachment 186366 [details]) > This is a trivial fix; I think it's ok to r+. It's still WebKit2 only and should thus be reviewed by a WebKit2 owner. We prefer to be consistent. In the future, please don't r+ WebKit2 patches.

Noam Rosenthal

Comment 8 2013-02-04 14:06:40 PST

(In reply to comment #7) > It's still WebKit2 only and should thus be reviewed by a WebKit2 owner. We prefer to be consistent. > > In the future, please don't r+ WebKit2 patches. No problem! I was following Sam's email which said that it's ok to trivial patches.

Noam Rosenthal

Comment 9 2013-02-04 14:07:07 PST

(In reply to comment #8) > (In reply to comment #7) > > It's still WebKit2 only and should thus be reviewed by a WebKit2 owner. We prefer to be consistent. > > > > In the future, please don't r+ WebKit2 patches. > No problem! > I was following Sam's email which said that it's ok to trivial patches. (I mean he said it's ok to r+ trivial patches).

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