diff options
author | scroggo <scroggo@google.com> | 2015-09-30 09:15:14 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-09-30 09:15:14 -0700 |
commit | 3a7701c0101386ba05acdde6f911be0c2696f317 (patch) | |
tree | cf32920b67df2f2553cb22d1fe420c9bab45d4c5 /src/codec/SkCodec_wbmp.cpp | |
parent | 46c574725676b26ada63ac15e42cda309dcd5090 (diff) |
Call rewindIfNeeded in SkCodec
Rather than calling it in each subclass, call it once in the base
class. Call it first, since other steps may modify internal structures
which would be replaced by a call to onRewind.
BUG=skia:4284
Review URL: https://codereview.chromium.org/1381483002
Diffstat (limited to 'src/codec/SkCodec_wbmp.cpp')
-rw-r--r-- | src/codec/SkCodec_wbmp.cpp | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/src/codec/SkCodec_wbmp.cpp b/src/codec/SkCodec_wbmp.cpp index 0c85eada7c..1696dfb585 100644 --- a/src/codec/SkCodec_wbmp.cpp +++ b/src/codec/SkCodec_wbmp.cpp @@ -110,9 +110,6 @@ SkCodec::Result SkWbmpCodec::onGetPixels(const SkImageInfo& info, const Options& options, SkPMColor ctable[], int* ctableCount) { - if (!this->rewindIfNeeded()) { - return kCouldNotRewind; - } if (options.fSubset) { // Subsets are not supported. return kUnimplemented; @@ -180,9 +177,6 @@ SkCodec::Result SkWbmpCodec::onGetScanlines(void* dst, int count, size_t dstRowB SkCodec::Result SkWbmpCodec::onStartScanlineDecode(const SkImageInfo& dstInfo, const Options& options, SkPMColor inputColorTable[], int* inputColorCount) { - if (!this->rewindIfNeeded()) { - return kCouldNotRewind; - } if (options.fSubset) { // Subsets are not supported. return kUnimplemented; |