Stop calling jpeg_finish_decompress()

Stop calling jpeg_finish_decompress()

jpeg_finish_decompress() does several things:
(1) Reads to the end of the image stream.
(2) Calls term_src(), a client provided function that
indicates we are done with the memory stream.  Our current
implementation of term_src() does nothing.
(3) Calls jpeg_abort() which aborts the decode and cleans
up some memory.

I don't think we need to call this anymore.

(1) seems irrelevant.

It seems a little dangerous to get rid of (2), but it is
fine while our implementation of term_src() does nothing.

(3) We will call jpeg_destroy() on destruction of the
JpegDecoderManager.  This should free all the memory,
making it unnecessary to call jpeg_abort() beforehand.

BUG=skia:4040
Committed: https://skia.googlesource.com/skia/+/19ae315dd06b01312de8fd40b8388aae4de9f05e

[msarett, 2015-10-02 17:15:27]

This has passed CodexTest running with valgrind.

I think the bots will let us know if this isn't ok.

[msarett, 2015-10-02 17:15:40]

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

[scroggo, 2015-10-02 18:45:56]

lgtm

[msarett, 2015-10-02 20:37:13]

The CQ bit was checked by msarett@google.com

[commit-bot: I haz the power, 2015-10-02 20:37:29]

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

[commit-bot: I haz the power, 2015-10-02 20:44:16]

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