SkPngCodec clean-ups

SkPngCodec clean-ups

Use png_read_row() instead of png_read_rows(1).
All png_read_rows() does is call png_read_row() in
a loop.  This comes from Leon's comment in:
https://codereview.chromium.org/1671003004/

Also there is a bit of refactoring.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1675843002

Committed: https://skia.googlesource.com/skia/+/60dcd3cb85797b555e47f3e66de81728a2eca40f

[msarett, 2016-02-05 22:28:03]

Description was changed from

==========
SkPngCodec clean-ups

Use png_read_row() instead of png_read_rows(1).
All png_read_rows() does is call png_read_row() in
a loop.

Also there is a bit of refactoring.

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

to

==========
SkPngCodec clean-ups

Use png_read_row() instead of png_read_rows(1).
All png_read_rows() does is call png_read_row() in
a loop.

Also there is a bit of refactoring.

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[msarett, 2016-02-05 22:29:31]

Patchset #1 (id:1) has been deleted

[msarett, 2016-02-05 22:30:11]

Description was changed from

==========
SkPngCodec clean-ups

Use png_read_row() instead of png_read_rows(1).
All png_read_rows() does is call png_read_row() in
a loop.

Also there is a bit of refactoring.

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
SkPngCodec clean-ups

Use png_read_row() instead of png_read_rows(1).
All png_read_rows() does is call png_read_row() in
a loop.  This comes from Leon's comment in:
https://codereview.chromium.org/1671003004/

Also there is a bit of refactoring.

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[msarett, 2016-02-05 22:30:37]

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

[scroggo, 2016-02-05 22:36:42]

lgtm

[msarett, 2016-02-05 22:37:52]

The CQ bit was checked by msarett@google.com

[commit-bot: I haz the power, 2016-02-05 22:38:04]

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

[commit-bot: I haz the power, 2016-02-05 22:38:05]

This CL has an open dependency (Issue 1673663002 Patch 200001). Please resolve
the dependency and try again.

[msarett, 2016-02-05 22:41:17]

The CQ bit was checked by msarett@google.com

[msarett, 2016-02-05 22:41:17]

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

[commit-bot: I haz the power, 2016-02-05 22:41:24]

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

[commit-bot: I haz the power, 2016-02-05 23:13:16]

Description was changed from

==========
SkPngCodec clean-ups

Use png_read_row() instead of png_read_rows(1).
All png_read_rows() does is call png_read_row() in
a loop.  This comes from Leon's comment in:
https://codereview.chromium.org/1671003004/

Also there is a bit of refactoring.

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
SkPngCodec clean-ups

Use png_read_row() instead of png_read_rows(1).
All png_read_rows() does is call png_read_row() in
a loop.  This comes from Leon's comment in:
https://codereview.chromium.org/1671003004/

Also there is a bit of refactoring.

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/60dcd3cb85797b555e47f3e66de81728a2eca40f
==========

[commit-bot: I haz the power, 2016-02-05 23:13:17]

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