Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(51)

Unified Diff: src/codec/SkJpegCodec.h

Issue 1260673002: SkScaledCodec class (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Move Jpeg Swizzler to ScanlineDecoder Created 5 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: src/codec/SkJpegCodec.h
diff --git a/src/codec/SkJpegCodec.h b/src/codec/SkJpegCodec.h
index 5e2135cdffb71b66faa6a2c1cb24ae895626d31d..73e4e0f339e90f8405981b9b0e034c76935fabe6 100644
--- a/src/codec/SkJpegCodec.h
+++ b/src/codec/SkJpegCodec.h
@@ -101,6 +101,11 @@ private:
SkJpegCodec(const SkImageInfo& srcInfo, SkStream* stream, JpegDecoderMgr* decoderMgr);
/*
+ * Create the swizzler based on the encoded format
scroggo 2015/08/06 15:09:11 This should probably state that it is only used in
emmaleer 2015/08/06 18:59:52 Acknowledged.
+ */
+ Result initializeSwizzler(const SkImageInfo& requestedInfo, const Options&);
+
+ /*
* Handles rewinding the input stream if it is necessary
*/
bool handleRewind();
@@ -113,19 +118,13 @@ private:
bool setOutputColorSpace(const SkImageInfo& dst);
/*
- * Checks if we can scale to the requested dimensions and scales the dimensions
- * if possible
+ * Checks if we can natively scale to the requested dimensions and natively scales the
+ * dimensions if possible
*/
- bool scaleToDimensions(uint32_t width, uint32_t height);
-
- /*
- * Create the swizzler based on the encoded format
- */
- void initializeSwizzler(const SkImageInfo& dstInfo, void* dst, size_t dstRowBytes,
- const Options& options);
+ bool nativelyScaleToDimensions(uint32_t width, uint32_t height);
SkAutoTDelete<JpegDecoderMgr> fDecoderMgr;
-
+
friend class SkJpegScanlineDecoder;
typedef SkCodec INHERITED;

Powered by Google App Engine
This is Rietveld 408576698