aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/codec/SkCodec_wbmp.cpp
diff options
context:
space:
mode:
authorGravatar scroggo <scroggo@google.com>2015-09-30 09:15:14 -0700
committerGravatar Commit bot <commit-bot@chromium.org>2015-09-30 09:15:14 -0700
commit3a7701c0101386ba05acdde6f911be0c2696f317 (patch)
treecf32920b67df2f2553cb22d1fe420c9bab45d4c5 /src/codec/SkCodec_wbmp.cpp
parent46c574725676b26ada63ac15e42cda309dcd5090 (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.cpp6
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;