Bug 78007 - Chromium: Fix Mac rubber band overhang drawing on composited pages and re-enable tests.
Summary: Chromium: Fix Mac rubber band overhang drawing on composited pages and re-ena...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: asvitkine
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-02-07 11:30 PST by asvitkine
Modified: 2012-02-08 11:38 PST (History)
3 users (show)

See Also:


Attachments
Patch (3.82 KB, patch)
2012-02-07 11:36 PST, asvitkine
no flags Details | Formatted Diff | Diff
Patch (3.82 KB, patch)
2012-02-07 11:40 PST, asvitkine
no flags Details | Formatted Diff | Diff
Patch (675.87 KB, patch)
2012-02-07 11:45 PST, asvitkine
no flags Details | Formatted Diff | Diff
Patch (675.79 KB, patch)
2012-02-07 13:16 PST, asvitkine
no flags Details | Formatted Diff | Diff
Patch for landing (678.39 KB, patch)
2012-02-08 06:51 PST, asvitkine
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description asvitkine 2012-02-07 11:30:53 PST
Chromium: Fix Mac rubber band overhang drawing on composited pages and re-enable tests.
Comment 1 asvitkine 2012-02-07 11:36:24 PST
Created attachment 125884 [details]
Patch
Comment 2 asvitkine 2012-02-07 11:40:37 PST
Created attachment 125887 [details]
Patch
Comment 3 asvitkine 2012-02-07 11:45:26 PST
Created attachment 125889 [details]
Patch
Comment 4 asvitkine 2012-02-07 11:46:35 PST
(The patch includes new baselines because of subtle changes to gradient pixels between the time the original baselines were added and the current WebKit.)
Comment 5 James Robinson 2012-02-07 13:00:59 PST
Comment on attachment 125889 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=125889&action=review

Why is there blue for these:

LayoutTests/platform/chromium/rubberbanding/custom-scrollbars-ne-expected.png
LayoutTests/platform/chromium/rubberbanding/custom-scrollbars-nw-expected.png
LayoutTests/platform/chromium/rubberbanding/custom-scrollbars-se-expected.png
LayoutTests/platform/chromium/rubberbanding/custom-scrollbars-sw-expected.png

? They don't look right

> Source/WebKit/chromium/src/NonCompositedContentHost.cpp:49
> +    m_graphicsLayer->platformLayer()->setBackgroundCoversViewport(false);

just #ifdef out the call with (true), it's false by default
Comment 6 asvitkine 2012-02-07 13:11:04 PST
Comment on attachment 125889 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=125889&action=review

>> Source/WebKit/chromium/src/NonCompositedContentHost.cpp:49
>> +    m_graphicsLayer->platformLayer()->setBackgroundCoversViewport(false);
> 
> just #ifdef out the call with (true), it's false by default

Done.
Comment 7 asvitkine 2012-02-07 13:16:59 PST
Created attachment 125910 [details]
Patch
Comment 8 asvitkine 2012-02-07 13:18:22 PST
(In reply to comment #5)
> (From update of attachment 125889 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=125889&action=review
> 
> Why is there blue for these:
> 
> LayoutTests/platform/chromium/rubberbanding/custom-scrollbars-ne-expected.png
> LayoutTests/platform/chromium/rubberbanding/custom-scrollbars-nw-expected.png
> LayoutTests/platform/chromium/rubberbanding/custom-scrollbars-se-expected.png
> LayoutTests/platform/chromium/rubberbanding/custom-scrollbars-sw-expected.png
> 
> ? They don't look right

The tests set:

        ::-webkit-scrollbar-track-piece,
        ::-webkit-scrollbar-corner {
            background-color: blue;
        }
Comment 9 James Robinson 2012-02-07 14:13:47 PST
(In reply to comment #8)
> (In reply to comment #5)
> > (From update of attachment 125889 [details] [details])
> > View in context: https://bugs.webkit.org/attachment.cgi?id=125889&action=review
> > 
> > Why is there blue for these:
> > 
> > LayoutTests/platform/chromium/rubberbanding/custom-scrollbars-ne-expected.png
> > LayoutTests/platform/chromium/rubberbanding/custom-scrollbars-nw-expected.png
> > LayoutTests/platform/chromium/rubberbanding/custom-scrollbars-se-expected.png
> > LayoutTests/platform/chromium/rubberbanding/custom-scrollbars-sw-expected.png
> > 
> > ? They don't look right
> 
> The tests set:
> 
>         ::-webkit-scrollbar-track-piece,
>         ::-webkit-scrollbar-corner {
>             background-color: blue;
>         }

Oh! My mistake, I was confusing that with the debug blue we use.  Might be better if the tests used green or something for scrollbars.

Anyway, this patch is fine, then. Thanks!
Comment 10 WebKit Review Bot 2012-02-07 15:48:56 PST
Comment on attachment 125910 [details]
Patch

Rejecting attachment 125910 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2

Last 500 characters of output:
ositedContentHost.cpp
patching file LayoutTests/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file LayoutTests/platform/chromium/test_expectations.txt
Hunk #1 succeeded at 517 (offset -2 lines).
Hunk #2 FAILED at 4050.
1 out of 2 hunks FAILED -- saving rejects to file LayoutTests/platform/chromium/test_expectations.txt.rej

Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--force', u'--reviewer', u'James Robi..." exit_code: 1 cwd: /mnt/git/webkit-commit-queue/

Full output: http://queues.webkit.org/results/11459133
Comment 11 asvitkine 2012-02-08 06:51:52 PST
Created attachment 126075 [details]
Patch for landing
Comment 12 WebKit Review Bot 2012-02-08 06:52:11 PST
Comment on attachment 126075 [details]
Patch for landing

Rejecting attachment 126075 [details] from commit-queue.

asvitkine@chromium.org does not have committer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/committers.py.

- If you do not have committer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have committer rights please correct the error in Tools/Scripts/webkitpy/common/config/committers.py by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your committer rights.
Comment 13 WebKit Review Bot 2012-02-08 08:58:04 PST
Comment on attachment 126075 [details]
Patch for landing

Clearing flags on attachment: 126075

Committed r107094: <http://trac.webkit.org/changeset/107094>
Comment 14 WebKit Review Bot 2012-02-08 08:58:11 PST
All reviewed patches have been landed.  Closing bug.
Comment 15 Julien Chaffraix 2012-02-08 11:33:45 PST
It looks like the baselines committed where were not matching the tests and I rebaselined them in http://trac.webkit.org/changeset/107109.

Alexei, could you double check that the new ones make sense? Thanks!
Comment 16 asvitkine 2012-02-08 11:38:21 PST
Oops, sorry about that Julien.

The rebaselines look good.

Thanks!