Bug 108820

Summary: [WK2][EFL] Weird stripe at the end of the page
Product: WebKit Reporter: Mikhail Pozdnyakov <mikhail.pozdnyakov>
Component: WebKit EFLAssignee: Mikhail Pozdnyakov <mikhail.pozdnyakov>
Status: RESOLVED FIXED    
Severity: Normal CC: andersca, benjamin, kenneth, lucas.de.marchi, noam, webkit.review.bot
Priority: P2    
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

Description Mikhail Pozdnyakov 2013-02-04 06:11:40 PST
There is a weird stripe at the end of the page when the page is scrolled down.
Comment 1 Mikhail Pozdnyakov 2013-02-04 06:12:40 PST
Created attachment 186363 [details]
screen shot with 'weird stripe'
Comment 2 Mikhail Pozdnyakov 2013-02-04 06:20:08 PST
Created attachment 186366 [details]
treatment
Comment 3 Kenneth Rohde Christiansen 2013-02-04 06:25:43 PST
Comment on attachment 186366 [details]
treatment

LGTM
Comment 4 Noam Rosenthal 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+.
Comment 5 WebKit Review Bot 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>
Comment 6 WebKit Review Bot 2013-02-04 07:01:51 PST
All reviewed patches have been landed.  Closing bug.
Comment 7 Anders Carlsson 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.
Comment 8 Noam Rosenthal 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.
Comment 9 Noam Rosenthal 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).