Make tab/tabstrip high density compatible

Make tab/tabstrip high density compatible
added ImageSkiaOperations and replaced Bitmap with ImageSkiaRep/ImagSkia.

BUG=122992
TEST=manual

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=145976

[pkotwicz, 2012-07-10 17:16:48]

LGTM, just nits

http://codereview.chromium.org/10704113/diff/14001/ui/gfx/image/image_skia_op...
File ui/gfx/image/image_skia_operations.cc (right):

http://codereview.chromium.org/10704113/diff/14001/ui/gfx/image/image_skia_op...
ui/gfx/image/image_skia_operations.cc:36: return  ImageSkiaRep(resized_bitmap,
ui::SCALE_FACTOR_200P);
Nit: 1 space

http://codereview.chromium.org/10704113/diff/14001/ui/gfx/image/image_skia_op...
ui/gfx/image/image_skia_operations.cc:41: // is provided, this function will
fail if the scale factors of two
Nit two-> the two images

http://codereview.chromium.org/10704113/diff/14001/ui/gfx/image/image_skia_op...
File ui/gfx/image/image_skia_operations.h (right):

http://codereview.chromium.org/10704113/diff/14001/ui/gfx/image/image_skia_op...
ui/gfx/image/image_skia_operations.h:19: // use have the kARGB_8888_Config
config and be of equal dimensions.
Nit: Get rid of 'have'

http://codereview.chromium.org/10704113/diff/14001/ui/gfx/image/image_skia_op...
ui/gfx/image/image_skia_operations.h:30: // Create a bitmap that is cropped from
another bitmap. This is special
Nit: bitmap->image

[oshima, 2012-07-10 18:25:00]

all done.

On 2012/07/10 17:16:48, pkotwicz wrote:
> LGTM, just nits
> 
>
http://codereview.chromium.org/10704113/diff/14001/ui/gfx/image/image_skia_op...
> File ui/gfx/image/image_skia_operations.cc (right):
> 
>
http://codereview.chromium.org/10704113/diff/14001/ui/gfx/image/image_skia_op...
> ui/gfx/image/image_skia_operations.cc:36: return  ImageSkiaRep(resized_bitmap,
> ui::SCALE_FACTOR_200P);
> Nit: 1 space
> 
>
http://codereview.chromium.org/10704113/diff/14001/ui/gfx/image/image_skia_op...
> ui/gfx/image/image_skia_operations.cc:41: // is provided, this function will
> fail if the scale factors of two
> Nit two-> the two images
> 
>
http://codereview.chromium.org/10704113/diff/14001/ui/gfx/image/image_skia_op...
> File ui/gfx/image/image_skia_operations.h (right):
> 
>
http://codereview.chromium.org/10704113/diff/14001/ui/gfx/image/image_skia_op...
> ui/gfx/image/image_skia_operations.h:19: // use have the kARGB_8888_Config
> config and be of equal dimensions.
> Nit: Get rid of 'have'
> 
>
http://codereview.chromium.org/10704113/diff/14001/ui/gfx/image/image_skia_op...
> ui/gfx/image/image_skia_operations.h:30: // Create a bitmap that is cropped
from
> another bitmap. This is special
> Nit: bitmap->image

[oshima, 2012-07-10 20:10:10]

ben -> OWNERS

[Ben Goodger (Google), 2012-07-10 20:58:48]

lgtm

[commit-bot: I haz the power, 2012-07-10 21:02:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/oshima@chromium.org/10704113/3018

[commit-bot: I haz the power, 2012-07-10 22:53:11]

Change committed as 145976