Support horizontal/vertical only images in NineImagePainter

Support horizontal/vertical only images in NineImagePainter.

This will be used in https://codereview.chromium.org/798163003/

BUG=432500
Committed: https://crrev.com/e98d8d68c1cde7ae05386befdd797360c7900b76
Cr-Commit-Position: refs/heads/master@{#308440}

[oshima, 2014-12-13 02:14:47]

oshima@chromium.org changed reviewers:
+ sadrul@chromium.org

[sadrul, 2014-12-15 17:21:19]

LGTM

https://codereview.chromium.org/806433002/diff/1/ui/gfx/nine_image_painter.cc
File ui/gfx/nine_image_painter.cc (right):

https://codereview.chromium.org/806433002/diff/1/ui/gfx/nine_image_painter.cc...
ui/gfx/nine_image_painter.cc:190: }
It would be nice if we didn't need to check isNull() for every call. (e.g. move
the check inside Fill, and/or have a new function DrawImage that checks isNull()
before calling into canvas->DragImageIntInPixel(), or move the check in there
etc.)

[oshima, 2014-12-15 19:11:58]

Patchset #3 (id:40001) has been deleted

[oshima, 2014-12-15 19:12:02]

Patchset #3 (id:60001) has been deleted

[oshima, 2014-12-15 19:15:23]

Patchset #2 (id:20001) has been deleted

[oshima, 2014-12-15 19:15:27]

Patchset #2 (id:80001) has been deleted

[oshima, 2014-12-15 19:16:28]

Uploaded new patch. PTAL

[oshima, 2014-12-15 19:25:25]

https://codereview.chromium.org/806433002/diff/120001/ui/base/nine_image_pain...
File ui/base/nine_image_painter_factory.h (right):

https://codereview.chromium.org/806433002/diff/120001/ui/base/nine_image_pain...
ui/base/nine_image_painter_factory.h:44: const int kImageGridNum = 9;
I added this to be used to define array of the image grid.
context:

https://codereview.chromium.org/798163003/diff/20001/ash/system/tray/tray_bac...

[oshima, 2014-12-15 21:26:38]

https://codereview.chromium.org/806433002/diff/120001/ui/base/nine_image_pain...
File ui/base/nine_image_painter_factory.h (right):

https://codereview.chromium.org/806433002/diff/120001/ui/base/nine_image_pain...
ui/base/nine_image_painter_factory.h:44: const int kImageGridNum = 9;
On 2014/12/15 19:25:25, oshima wrote:
> I added this to be used to define array of the image grid.
> context:
> 
>
https://codereview.chromium.org/798163003/diff/20001/ash/system/tray/tray_bac...

you didn't seem to like it, so removed. PTAL

[sadrul, 2014-12-15 21:29:52]

LGTM!

[oshima, 2014-12-15 21:30:15]

The CQ bit was checked by oshima@chromium.org

[commit-bot: I haz the power, 2014-12-15 21:31:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/806433002/140001

[commit-bot: I haz the power, 2014-12-15 22:22:03]

Committed patchset #4 (id:140001)

[commit-bot: I haz the power, 2014-12-15 22:23:14]

Patchset 4 (id:??) landed as
https://crrev.com/e98d8d68c1cde7ae05386befdd797360c7900b76
Cr-Commit-Position: refs/heads/master@{#308440}