Implement onSkipScanlines() for bmp and wbmp

TBR=reed@google.com
BUG=skia:4270
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1691083002

Review URL: https://codereview.chromium.org/1691083002

1 files changed, 1 insertions, 12 deletions

diff --git a/include/codec/SkCodec.h b/include/codec/SkCodec.h
index edc8ec38fa..c3a5873d7f 100644
--- a/include/codec/SkCodec.h
+++ b/include/codec/SkCodec.h
@@ -657,18 +657,7 @@ private:
         return kUnimplemented;
     }
 
-    // Naive default version just calls onGetScanlines on temp memory.
-    virtual bool onSkipScanlines(int countLines) {
-        // FIXME (msarett): Make this a pure virtual and always override this.
-        SkAutoMalloc storage(fDstInfo.minRowBytes());
-
-        // Note that we pass 0 to rowBytes so we continue to use the same memory.
-        // Also note that while getScanlines checks that rowBytes is big enough,
-        // onGetScanlines bypasses that check.
-        // Calling the virtual method also means we do not double count
-        // countLines.
-        return countLines == this->onGetScanlines(storage.get(), countLines, 0);
-    }
+    virtual bool onSkipScanlines(int /*countLines*/) { return false; }
 
     virtual int onGetScanlines(void* /*dst*/, int /*countLines*/, size_t /*rowBytes*/) { return 0; }