Assume all TURBO_HAS_* are true.

Assume all TURBO_HAS_* are true.

Should be everyone's on libjpeg-turbo >= 1.5.0.

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

Committed: https://skia.googlesource.com/skia/+/da19f6f3b5f1375c854a9bde947c12099afc25c7

[mtklein_C, 2016-08-23 18:05:29]

Description was changed from

==========
Assume all TURBO_HAS_* are true.

Should be everyone's on libjpeg-turbo >= 1.5.0.

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

to

==========
Assume all TURBO_HAS_* are true.

Should be everyone's on libjpeg-turbo >= 1.5.0.

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

[mtklein_C, 2016-08-23 18:07:04]

The CQ bit was checked by mtklein@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-23 18:07:09]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[mtklein_C, 2016-08-23 18:08:30]

mtklein@chromium.org changed reviewers:
+ msarett@google.com

[msarett, 2016-08-23 18:13:48]

Beat me to it :)

https://codereview.chromium.org/2274643002/diff/20001/src/codec/SkJpegCodec.cpp
File src/codec/SkJpegCodec.cpp (right):

https://codereview.chromium.org/2274643002/diff/20001/src/codec/SkJpegCodec.c...
src/codec/SkJpegCodec.cpp:567: if (JCS_CMYK == colorSpace || JCS_RGB ==
colorSpace) {
Remove " || JCS_RGB == colorSpace".  We will never decode to RGB now that we
always have RGB565 support.

https://codereview.chromium.org/2274643002/diff/20001/src/codec/SkJpegCodec.c...
src/codec/SkJpegCodec.cpp:613: case JCS_RGB:
Remove this case.  Also probably makes sense to make this an "if" statement.

[mtklein_C, 2016-08-23 18:17:29]

The CQ bit was checked by mtklein@chromium.org to run a CQ dry run

[mtklein_C, 2016-08-23 18:17:35]

On 2016/08/23 at 18:13:48, msarett wrote:
> Beat me to it :)
> 
>
https://codereview.chromium.org/2274643002/diff/20001/src/codec/SkJpegCodec.cpp
> File src/codec/SkJpegCodec.cpp (right):
> 
>
https://codereview.chromium.org/2274643002/diff/20001/src/codec/SkJpegCodec.c...
> src/codec/SkJpegCodec.cpp:567: if (JCS_CMYK == colorSpace || JCS_RGB ==
colorSpace) {
> Remove " || JCS_RGB == colorSpace".  We will never decode to RGB now that we
always have RGB565 support.
> 
>
https://codereview.chromium.org/2274643002/diff/20001/src/codec/SkJpegCodec.c...
> src/codec/SkJpegCodec.cpp:613: case JCS_RGB:
> Remove this case.  Also probably makes sense to make this an "if" statement.

Done.

[commit-bot: I haz the power, 2016-08-23 18:17:39]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[msarett, 2016-08-23 18:19:07]

lgtm

[mtklein_C, 2016-08-23 18:35:49]

The CQ bit was unchecked by mtklein@chromium.org

[mtklein_C, 2016-08-23 18:35:51]

The CQ bit was checked by mtklein@chromium.org

[commit-bot: I haz the power, 2016-08-23 18:35:55]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-23 18:49:31]

Description was changed from

==========
Assume all TURBO_HAS_* are true.

Should be everyone's on libjpeg-turbo >= 1.5.0.

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

to

==========
Assume all TURBO_HAS_* are true.

Should be everyone's on libjpeg-turbo >= 1.5.0.

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

Committed:
https://skia.googlesource.com/skia/+/da19f6f3b5f1375c854a9bde947c12099afc25c7
==========

[commit-bot: I haz the power, 2016-08-23 18:49:32]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/da19f6f3b5f1375c854a9bde947c12099afc25c7