Add custom overhang pattern for rubber-banding.

Add custom overhang pattern for rubber-banding.

This will be used in WebKit for overhang painting.

BUG=93397
TEST=none
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=97855

[tony, 2011-08-22 18:28:57]

Is this only going to be used by DRT or will it be used by chromium?  Would it
be possible to just draw the pattern using CG/skia?

[sail, 2011-08-22 18:33:37]

Updated the bug with a new image. Thanks!

[Alexei Svitkine (slow), 2011-08-22 18:41:22]

On 2011/08/22 18:28:57, tony wrote:
> Is this only going to be used by DRT or will it be used by chromium? 

It will be used by Chromium. It's needed for the overscroll area when
rubber-band scrolling (only on Mac OS X Lion, at the moment).

> Would it be possible to just draw the pattern using CG/skia?

You mean programatically? Sounds like it would be hard to maintain if we decide
to change it, especially since it comes from UI leads.

> Updated the bug with a new image. Thanks!

New image uploaded.

[tony, 2011-08-22 19:11:20]

LGTM.

20k seems like a lot of size to add to Win/Linux for something that isn't used. 
Writing code to draw a pattern seems better in the long run (the pattern isn't
meant to be complex, right?).

[Alexei Svitkine (slow), 2011-08-22 19:46:01]

> 20k seems like a lot of size to add to Win/Linux for something that isn't
used. 
> Writing code to draw a pattern seems better in the long run (the pattern isn't
> meant to be complex, right?).

I assume there's no way to limit the resource to just Mac platform in the grd
file?

[tony, 2011-08-22 19:47:07]

On 2011/08/22 19:46:01, asvitkine_ wrote:
> > 20k seems like a lot of size to add to Win/Linux for something that isn't
> used. 
> > Writing code to draw a pattern seems better in the long run (the pattern
isn't
> > meant to be complex, right?).
> 
> I assume there's no way to limit the resource to just Mac platform in the grd
> file?

You can wrap it in an <if expr="is_macosx"></if>.  There are examples of this in
chrome/browser/browser_resources.grd.

[Alexei Svitkine (slow), 2011-08-22 19:52:59]

> You can wrap it in an <if expr="is_macosx"></if>.  There are examples of this
in
> chrome/browser/browser_resources.grd.

Done. Now, we only need to pay the 20k on the Mac.

[tony, 2011-08-22 20:29:49]

LGTM

[commit-bot: I haz the power, 2011-08-23 02:12:25]

The commit queue went berserk retrying too often for a
seemingly flaky test. Builder is mac, revision is 97755, job name
was 7711007-3005 (previous was lost) (retry) (previous was lost) (previous was
lost).