Add test for platform encoders, turn off platform encoders by default

Add test for platform encoders, turn off platform encoders by default

Clients that like WIC and CG can still use them.  And we can be
confident about that, since we now test WIC and CG.

Let Skia always use our own encoders by default, so we can do cool,
custom things on all platforms.

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

Committed: https://skia.googlesource.com/skia/+/36931c2b47f81db7ccf441937567c6fda72ad3e6

[msarett, 2016-08-16 20:21:31]

Description was changed from

==========
Add test for platform encoders, turn off platform encoders by default

Clients that like WIC and CG can still use them.  And we can be
confident about that, since we now test WIC and CG.

Let Skia always our own encoders by default, so we can do cool,
custom things on all platforms.

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

to

==========
Add test for platform encoders, turn off platform encoders by default

Clients that like WIC and CG can still use them.  And we can be
confident about that, since we now test WIC and CG.

Let Skia always our own encoders by default, so we can do cool,
custom things on all platforms.

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

[msarett, 2016-08-16 20:25:14]

Patchset #1 (id:1) has been deleted

[msarett, 2016-08-16 20:25:26]

Description was changed from

==========
Add test for platform encoders, turn off platform encoders by default

Clients that like WIC and CG can still use them.  And we can be
confident about that, since we now test WIC and CG.

Let Skia always our own encoders by default, so we can do cool,
custom things on all platforms.

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

to

==========
Add test for platform encoders, turn off platform encoders by default

Clients that like WIC and CG can still use them.  And we can be
confident about that, since we now test WIC and CG.

Let Skia always use our own encoders by default, so we can do cool,
custom things on all platforms.

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

[msarett, 2016-08-16 21:26:50]

msarett@google.com changed reviewers:
+ mtklein@google.com, reed@google.com

[msarett, 2016-08-16 21:26:51]

https://codereview.chromium.org/2250683003/diff/60001/src/ports/SkImageEncode...
File src/ports/SkImageEncoder_WIC.cpp (left):

https://codereview.chromium.org/2250683003/diff/60001/src/ports/SkImageEncode...
src/ports/SkImageEncoder_WIC.cpp:70: case kBMP_Type:
These don't work.  I'm guessing that they've never worked (like our WIC JPEG
encoder...).  I think I'll delete them until somebody asks for this.

[reed1, 2016-08-16 21:34:14]

lgtm

[msarett, 2016-08-16 21:37:11]

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

[commit-bot: I haz the power, 2016-08-16 21:37:14]

Dry run: 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-16 22:07:29]

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

[commit-bot: I haz the power, 2016-08-16 22:07:30]

Dry run: This issue passed the CQ dry run.

[msarett, 2016-08-16 22:10:27]

The CQ bit was checked by msarett@google.com

[commit-bot: I haz the power, 2016-08-16 22:10:29]

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-16 22:11:26]

Description was changed from

==========
Add test for platform encoders, turn off platform encoders by default

Clients that like WIC and CG can still use them.  And we can be
confident about that, since we now test WIC and CG.

Let Skia always use our own encoders by default, so we can do cool,
custom things on all platforms.

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

to

==========
Add test for platform encoders, turn off platform encoders by default

Clients that like WIC and CG can still use them.  And we can be
confident about that, since we now test WIC and CG.

Let Skia always use our own encoders by default, so we can do cool,
custom things on all platforms.

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

Committed:
https://skia.googlesource.com/skia/+/36931c2b47f81db7ccf441937567c6fda72ad3e6
==========

[commit-bot: I haz the power, 2016-08-16 22:11:27]

Committed patchset #3 (id:60001) as
https://skia.googlesource.com/skia/+/36931c2b47f81db7ccf441937567c6fda72ad3e6