diff options
author | Leon Scroggins III <scroggo@google.com> | 2017-10-30 09:07:53 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-11-03 15:29:34 +0000 |
commit | 0cbc10f94e2b350761e64b225c97390269f41479 (patch) | |
tree | a3dd274a32583b289af7fb930ce44a6eec001599 /src/core/SkPixmapPriv.h | |
parent | 60aaeb2b551d5e3e90baacac315e899e26f758f1 (diff) |
Make SkImageGeneratorCG apply encoded origin
SkImageGeneratorCG:
- Detect the origin and apply it to the output
- Deprecate NewFromEncodedCG and add MakeFromEncodedCG
SkCodecImageGenerator:
- Move code elsewhere for sharing
- Apply origin for incomplete decodes
SkPixmap.cpp/SkPixmapPriv.h:
- Now has the shared code for generators to apply origin
DMSrcSink.cpp:
- Call MakeFromEncodedCG
SkCGUtils.h:
- Add a version of SkCopyPixelsFromCGImage that takes an SkPixmap
Bug: skia:7138
Bug: skia:3834
Change-Id: Ic6dbc76360c6a84913b67373582f328d3946d637
Reviewed-on: https://skia-review.googlesource.com/63740
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Leon Scroggins <scroggo@google.com>
Diffstat (limited to 'src/core/SkPixmapPriv.h')
-rw-r--r-- | src/core/SkPixmapPriv.h | 36 |
1 files changed, 36 insertions, 0 deletions
diff --git a/src/core/SkPixmapPriv.h b/src/core/SkPixmapPriv.h index 1d677fdc46..9941862676 100644 --- a/src/core/SkPixmapPriv.h +++ b/src/core/SkPixmapPriv.h @@ -10,6 +10,7 @@ #include "SkPixmap.h" #include "SkEncodedOrigin.h" +#include "SkAutoPixmapStorage.h" class SkPixmapPriv { public: @@ -27,6 +28,41 @@ public: * by the flags. If the inputs are invalid, this returns false and no copy is made. */ static bool Orient(const SkPixmap& dst, const SkPixmap& src, OrientFlags); + + static bool ShouldSwapWidthHeight(SkEncodedOrigin o); + static SkImageInfo SwapWidthHeight(const SkImageInfo& info); + + /** + * Decode an image and then copy into dst, applying origin. + * + * @param dst SkPixmap to write the final image, after + * applying the origin. + * @param origin SkEncodedOrigin to apply to the raw pixels. + * @param decode Function for decoding into a pixmap without + * applying the origin. + */ + static bool Orient(const SkPixmap& dst, SkEncodedOrigin origin, + std::function<bool(const SkPixmap&)> decode) { + SkAutoPixmapStorage storage; + const SkPixmap* tmp = &dst; + if (origin != kTopLeft_SkEncodedOrigin) { + auto info = dst.info(); + if (ShouldSwapWidthHeight(origin)) { + info = SwapWidthHeight(info); + } + if (!storage.tryAlloc(info)) { + return false; + } + tmp = &storage; + } + if (!decode(*tmp)) { + return false; + } + if (tmp != &dst) { + return Orient(dst, *tmp, OriginToOrient(origin)); + } + return true; + } }; #endif |