Fix uses of SkImageInfo in BRD to preserve color space info

Fix uses of SkImageInfo in BRD to preserve color space info

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2035483005

Committed: https://skia.googlesource.com/skia/+/577967c155fe3e2ce8251b053da60bd0d8fb32f4

[msarett, 2016-06-03 14:53:51]

Description was changed from

==========
Fix uses of SkImageInfo in BRD to preserve color space info

BUG=skia:
==========

to

==========
Fix uses of SkImageInfo in BRD to preserve color space info

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2035483005
==========

[msarett, 2016-06-03 14:54:46]

msarett@google.com changed reviewers:
+ brianosman@google.com, djsollen@google.com, scroggo@google.com

[scroggo, 2016-06-03 15:01:59]

lgtm

https://codereview.chromium.org/2035483005/diff/1/src/android/SkBitmapRegionC...
File src/android/SkBitmapRegionCodec.cpp (right):

https://codereview.chromium.org/2035483005/diff/1/src/android/SkBitmapRegionC...
src/android/SkBitmapRegionCodec.cpp:135: return
conversion_possible(fCodec->getInfo().makeColorType(colorType),
fCodec->getInfo());
I suppose we'll need to (eventually) update conversion_possible to determine
whether there's a conversion between the color spaces?

[msarett, 2016-06-03 15:08:32]

https://codereview.chromium.org/2035483005/diff/1/src/android/SkBitmapRegionC...
File src/android/SkBitmapRegionCodec.cpp (right):

https://codereview.chromium.org/2035483005/diff/1/src/android/SkBitmapRegionC...
src/android/SkBitmapRegionCodec.cpp:135: return
conversion_possible(fCodec->getInfo().makeColorType(colorType),
fCodec->getInfo());
On 2016/06/03 15:01:59, scroggo wrote:
> I suppose we'll need to (eventually) update conversion_possible to determine
> whether there's a conversion between the color spaces?

Not quite sure if this change will matter or not.

I'm not sure if there is ever a case where we will ever choose to not support a
colorspace conversion (or decide that we can't).

But I figured it can't hurt to update this as well while I'm here.

[msarett, 2016-06-03 15:08:43]

The CQ bit was checked by msarett@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-06-03 15:08:46]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2035483005/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2035483005/1

[commit-bot: I haz the power, 2016-06-03 15:21:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-03 15:21:58]

Dry run: This issue passed the CQ dry run.

[msarett, 2016-06-03 15:22:22]

The CQ bit was checked by msarett@google.com

[commit-bot: I haz the power, 2016-06-03 15:22:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2035483005/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2035483005/1

[commit-bot: I haz the power, 2016-06-03 15:23:43]

Description was changed from

==========
Fix uses of SkImageInfo in BRD to preserve color space info

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2035483005
==========

to

==========
Fix uses of SkImageInfo in BRD to preserve color space info

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2035483005

Committed:
https://skia.googlesource.com/skia/+/577967c155fe3e2ce8251b053da60bd0d8fb32f4
==========

[commit-bot: I haz the power, 2016-06-03 15:23:44]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/577967c155fe3e2ce8251b053da60bd0d8fb32f4