Re-enabling YUV GPU decoding

Re-enabling YUV GPU decoding

BUG=

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=181853

[sugoi1, 2014-09-11 19:57:40]

sugoi@chromium.org changed reviewers:
+ noel@chromium.org, senorblanco@chromium.org

[sugoi1, 2014-09-11 19:57:41]

https://codereview.chromium.org/544823003/diff/1/Source/platform/graphics/Ima...
File Source/platform/graphics/ImageFrameGenerator.cpp (right):

https://codereview.chromium.org/544823003/diff/1/Source/platform/graphics/Ima...
Source/platform/graphics/ImageFrameGenerator.cpp:72: static bool
updateYUVComponentSizes(ImageDecoder* decoder, SkISize componentSizes[3],
ImageDecoder::SizeType sizeType)
Oops, adding the "const" here was causing the isSizeAvailable() call to use the
const implementation instead of using the overloaded non const version in
JPEGImageDecoder, so it was always false.

[Stephen White, 2014-09-11 20:36:36]

LGTM

(BTW, if a tiny fix like this is blocking you and/or breaking stuff, it's a good
candidate to be landed TBR.)

[sugoi1, 2014-09-11 20:42:51]

The CQ bit was checked by sugoi@chromium.org

[commit-bot: I haz the power, 2014-09-11 20:43:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/544823003/1

[commit-bot: I haz the power, 2014-09-11 22:47:26]

Committed patchset #1 (id:1) as 181853

[Noel Gordon, 2014-09-12 01:47:06]

LGTM

https://codereview.chromium.org/544823003/diff/1/Source/platform/graphics/Ima...
File Source/platform/graphics/ImageFrameGenerator.cpp (right):

https://codereview.chromium.org/544823003/diff/1/Source/platform/graphics/Ima...
Source/platform/graphics/ImageFrameGenerator.cpp:72: static bool
updateYUVComponentSizes(ImageDecoder* decoder, SkISize componentSizes[3],
ImageDecoder::SizeType sizeType)
On 2014/09/11 19:57:41, sugoi1 wrote:
> Oops, adding the "const" here was causing the isSizeAvailable() call to use
the
> const implementation instead of using the overloaded non const version in
> JPEGImageDecoder, so it was always false.

Ack.