PNGImageDecoder: Clean up rowAvailable() some more
authornoel.gordon@gmail.com <noel.gordon@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 4 May 2012 01:45:21 +0000 (01:45 +0000)
committernoel.gordon@gmail.com <noel.gordon@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 4 May 2012 01:45:21 +0000 (01:45 +0000)
commitc23d73bb17587b893c10c4ea488b8dc2d2a74aac
treebfc90b8ec5131aca48bbb36d24ec8b5ab26e532e
parent9ed5fa1d547e0c8db4d7ffcdd4f04d0746a76cb1
PNGImageDecoder: Clean up rowAvailable() some more
https://bugs.webkit.org/show_bug.cgi?id=85464

Reviewed by Eric Seidel.

No new tests. Covered by existing tests: fast/images/png-extra-row-crash.html in
particular.

* platform/image-decoders/png/PNGImageDecoder.cpp:
(WebCore::PNGImageDecoder::rowAvailable): Use colorChannels consistently. Split
the useful libpng comments in two, then place the early-out code and conditions
inbetween. The png variable is only used in one place so move it there.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@116059 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Source/WebCore/ChangeLog
Source/WebCore/platform/image-decoders/png/PNGImageDecoder.cpp