tuned skiaserve's png output for better performance

tuned skiaserve's png output for better performance
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1704573002

Committed: https://skia.googlesource.com/skia/+/546d66544bcfac4bc9e77626ba549c4b3a79a654

[ethannicholas, 2016-02-16 17:38:26]

Description was changed from

==========
tuned skiaserve's png output for better performance
==========

to

==========
tuned skiaserve's png output for better performance
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[ethannicholas, 2016-02-16 17:41:36]

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

[commit-bot: I haz the power, 2016-02-16 17:41:40]

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

[ethannicholas, 2016-02-16 17:43:23]

ethannicholas@google.com changed reviewers:
+ jcgregorio@google.com, joshualitt@google.com

[joshualitt, 2016-02-16 17:45:08]

On 2016/02/16 17:43:24, ethannicholas wrote:

lgtm + nit

[joshualitt, 2016-02-16 17:45:23]

nit

https://codereview.chromium.org/1704573002/diff/1/tools/skiaserve/skiaserve.cpp
File tools/skiaserve/skiaserve.cpp (right):

https://codereview.chromium.org/1704573002/diff/1/tools/skiaserve/skiaserve.c...
tools/skiaserve/skiaserve.cpp:113: for (png_size_t y = 0; y < height; ++y)
I like to wrap one line for loops, but I don't have a strong preference.

[ethannicholas, 2016-02-16 17:47:41]

The CQ bit was checked by ethannicholas@google.com

[ethannicholas, 2016-02-16 17:47:41]

The patchset sent to the CQ was uploaded after l-g-t-m from
joshualitt@google.com
Link to the patchset: https://codereview.chromium.org/1704573002/#ps20001
(title: "formatting")

[commit-bot: I haz the power, 2016-02-16 17:47:45]

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

[mtklein, 2016-02-16 17:48:51]

mtklein@google.com changed reviewers:
+ mtklein@google.com

[mtklein, 2016-02-16 17:48:52]

https://codereview.chromium.org/1704573002/diff/20001/tools/skiaserve/skiaser...
File tools/skiaserve/skiaserve.cpp (right):

https://codereview.chromium.org/1704573002/diff/20001/tools/skiaserve/skiaser...
tools/skiaserve/skiaserve.cpp:116: png_set_filter(png, 0, PNG_NO_FILTERS);
If you're interested in trying allowing any filters back into your set, I'd try
them in this order: UP, SUB, AVG, PAETH.  UP is likely super, super fast.

[commit-bot: I haz the power, 2016-02-16 17:49:01]

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

[commit-bot: I haz the power, 2016-02-16 17:49:02]

Try jobs failed on following builders:
  Build-Win-MSVC-x86-Debug-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Win-MSVC-x86-D...)

[ethannicholas, 2016-02-16 18:58:22]

The CQ bit was checked by ethannicholas@google.com

[commit-bot: I haz the power, 2016-02-16 18:58:30]

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

[commit-bot: I haz the power, 2016-02-16 19:03:06]

Description was changed from

==========
tuned skiaserve's png output for better performance
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
tuned skiaserve's png output for better performance
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/546d66544bcfac4bc9e77626ba549c4b3a79a654
==========

[commit-bot: I haz the power, 2016-02-16 19:03:07]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/546d66544bcfac4bc9e77626ba549c4b3a79a654