aboutsummaryrefslogtreecommitdiffhomepage
path: root/include/codec/SkScaledCodec.h
diff options
context:
space:
mode:
authorGravatar msarett <msarett@google.com>2015-10-09 11:07:34 -0700
committerGravatar Commit bot <commit-bot@chromium.org>2015-10-09 11:07:34 -0700
commite6dd004c1b8a81dc37a370570877b8b7d6dbe308 (patch)
tree16ecfdede3939dcd7f1b3db311371d974477e7c5 /include/codec/SkScaledCodec.h
parent79da63fa0bec40b560597c79c2e1231fa51aef36 (diff)
Fill incomplete images in SkCodec parent class
Rather than implementing some sort of "fill" in every SkCodec subclass for incomplete images, let's make the parent class handle this situation. This includes an API change to SkCodec.h SkCodec::getScanlines() now returns the number of lines it read successfully, rather than an SkCodec::Result enum. getScanlines() most often fails on an incomplete input, in which case it is useful to know how many lines were successfully decoded - this provides more information than kIncomplete vs kSuccess. We do lose information when the API is used improperly, as we are no longer able to return kInvalidParameter or kScanlineNotStarted. Known Issues: Does not work for incomplete fFrameIsSubset gifs. Does not work for incomplete icos. BUG=skia: Review URL: https://codereview.chromium.org/1332053002
Diffstat (limited to 'include/codec/SkScaledCodec.h')
-rw-r--r--include/codec/SkScaledCodec.h6
1 files changed, 5 insertions, 1 deletions
diff --git a/include/codec/SkScaledCodec.h b/include/codec/SkScaledCodec.h
index 61208e316a..028706bc94 100644
--- a/include/codec/SkScaledCodec.h
+++ b/include/codec/SkScaledCodec.h
@@ -34,7 +34,7 @@ protected:
SkISize onGetScaledDimensions(float desiredScale) const override;
bool onDimensionsSupported(const SkISize&) override;
- Result onGetPixels(const SkImageInfo&, void*, size_t, const Options&, SkPMColor*, int*)
+ Result onGetPixels(const SkImageInfo&, void*, size_t, const Options&, SkPMColor*, int*, int*)
override;
SkEncodedFormat onGetEncodedFormat() const override {
return fCodec->getEncodedFormat();
@@ -44,6 +44,10 @@ protected:
return fCodec->reallyHasAlpha();
}
+ uint32_t onGetFillValue(SkColorType colorType, SkAlphaType alphaType) const override;
+
+ SkScanlineOrder onGetScanlineOrder() const override;
+
private:
SkAutoTDelete<SkCodec> fCodec;