Add a DM flag to limit the types of Codec decodes

Add a DM flag to limit the types of Codec decodes

When the flag "simpleCodec" is set to true, we will only decode
to the canvas colortype and to a scale of 1. No SkAndroidCodec or
BRDCodec tests are run. This makes it easier to find the simple
case when running locally.

Clean up some SkImageDecoder references along the way.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1735743003

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

[scroggo_chromium, 2016-02-25 17:04:40]

Description was changed from

==========
Add a DM flag to limit the types of Codec decodes

When the flag "simpleCodec" is set to true, we will only decode
to the canvas colortype and to a scale of 1. No SkAndroidCodec or
BRDCodec tests are run. This makes it easier to find the simple
case when running locally.

Clean up some SkImageDecoder references along the way.
==========

to

==========
Add a DM flag to limit the types of Codec decodes

When the flag "simpleCodec" is set to true, we will only decode
to the canvas colortype and to a scale of 1. No SkAndroidCodec or
BRDCodec tests are run. This makes it easier to find the simple
case when running locally.

Clean up some SkImageDecoder references along the way.
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[scroggo, 2016-02-25 17:08:36]

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

[scroggo, 2016-02-25 17:08:36]

https://codereview.chromium.org/1735743003/diff/1/dm/DM.cpp
File dm/DM.cpp (right):

https://codereview.chromium.org/1735743003/diff/1/dm/DM.cpp#newcode67
dm/DM.cpp:67: DEFINE_bool(simpleCodec, false, "Only decode images to native
scale");
Alternatively, we can set this to true as the default, and update dm_flags.py to
set it to false. When running locally, I often just want to verify that decoding
worked as I expected for the normal case. Gold will help us sort out the rest,
but I hate searching through all those folders to find the basic one I want.

(FWIW, this CL cuts us down to three folders per config. I could cut it down to
only decoding to unpremul when simpleCodec is true, which cuts us down to one.)

[msarett, 2016-02-25 17:47:38]

lgtm

This is great.  Don't want to think about how time I've spent commenting out the
brd tests when testing...

https://codereview.chromium.org/1735743003/diff/1/dm/DM.cpp
File dm/DM.cpp (right):

https://codereview.chromium.org/1735743003/diff/1/dm/DM.cpp#newcode67
dm/DM.cpp:67: DEFINE_bool(simpleCodec, false, "Only decode images to native
scale");
On 2016/02/25 17:08:36, scroggo wrote:
> Alternatively, we can set this to true as the default, and update dm_flags.py
to
> set it to false. When running locally, I often just want to verify that
decoding
> worked as I expected for the normal case. Gold will help us sort out the rest,
> but I hate searching through all those folders to find the basic one I want.
> 
> (FWIW, this CL cuts us down to three folders per config. I could cut it down
to
> only decoding to unpremul when simpleCodec is true, which cuts us down to
one.)

Acknowledged.

[scroggo, 2016-02-25 18:47:37]

The CQ bit was checked by scroggo@google.com

[commit-bot: I haz the power, 2016-02-25 18:47:50]

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

[commit-bot: I haz the power, 2016-02-25 19:03:49]

Description was changed from

==========
Add a DM flag to limit the types of Codec decodes

When the flag "simpleCodec" is set to true, we will only decode
to the canvas colortype and to a scale of 1. No SkAndroidCodec or
BRDCodec tests are run. This makes it easier to find the simple
case when running locally.

Clean up some SkImageDecoder references along the way.
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Add a DM flag to limit the types of Codec decodes

When the flag "simpleCodec" is set to true, we will only decode
to the canvas colortype and to a scale of 1. No SkAndroidCodec or
BRDCodec tests are run. This makes it easier to find the simple
case when running locally.

Clean up some SkImageDecoder references along the way.
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

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

[commit-bot: I haz the power, 2016-02-25 19:03:50]

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