diff options
author | Mike Reed <reed@google.com> | 2017-12-15 20:42:14 +0000 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-12-15 20:42:21 +0000 |
commit | a091a4bd78011a266e7292a9b721750ff62423d3 (patch) | |
tree | e4cc7e14e7aa35a9b315690c469f715579acc643 /src | |
parent | f09e35eb59e43dedc329d3d15065b3e2735b7936 (diff) |
Revert "Revert "hide SkImageDeserializer""
This reverts commit 36d71787929002511edc37663b56d5909d2600cf.
Reason for revert: google3 updated
Original change's description:
> Revert "hide SkImageDeserializer"
>
> This reverts commit 834fb8ed906984739aee6b61417afddb9c6e469c.
>
> Reason for revert: broke google3
>
> Original change's description:
> > hide SkImageDeserializer
> >
> > Bug: skia:
> > Change-Id: I1cd4e8c626628a3e6426afd7d6bae628d58989c3
> > Reviewed-on: https://skia-review.googlesource.com/85743
> > Reviewed-by: Mike Reed <reed@google.com>
> > Commit-Queue: Mike Reed <reed@google.com>
>
> TBR=reed@google.com,reed@chromium.org
>
> Change-Id: I00679c8651817af1777785da6c8dac9411a2a7c2
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: skia:
> Reviewed-on: https://skia-review.googlesource.com/85880
> Reviewed-by: Mike Reed <reed@google.com>
> Commit-Queue: Mike Reed <reed@google.com>
TBR=reed@google.com,reed@chromium.org
Change-Id: Icce36dfb9570ccf79f165076c59aa4de49fb6040
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: skia:
Reviewed-on: https://skia-review.googlesource.com/86100
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Mike Reed <reed@google.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/core/SkPicture.cpp | 53 | ||||
-rw-r--r-- | src/core/SkReadBuffer.h | 2 | ||||
-rw-r--r-- | src/image/SkImage.cpp | 2 |
3 files changed, 36 insertions, 21 deletions
diff --git a/src/core/SkPicture.cpp b/src/core/SkPicture.cpp index 2761c417a6..f5085a63f6 100644 --- a/src/core/SkPicture.cpp +++ b/src/core/SkPicture.cpp @@ -143,18 +143,13 @@ sk_sp<SkPicture> SkPicture::Forwardport(const SkPictInfo& info, return r.finishRecordingAsPicture(); } +#ifdef SK_SUPPORT_LEGACY_IMAGEDESERIALIZER sk_sp<SkPicture> SkPicture::MakeFromStream(SkStream* stream, SkImageDeserializer* factory) { SkDeserialProcs procs; procs.fImageProc = ImageDeserializer_SkDeserialImageProc; procs.fImageCtx = factory; return MakeFromStream(stream, procs, nullptr); } - -sk_sp<SkPicture> SkPicture::MakeFromStream(SkStream* stream) { - SkImageDeserializer factory; - return MakeFromStream(stream, &factory); -} - sk_sp<SkPicture> SkPicture::MakeFromData(const void* data, size_t size, SkImageDeserializer* factory) { SkMemoryStream stream(data, size); @@ -168,6 +163,28 @@ sk_sp<SkPicture> SkPicture::MakeFromData(const SkData* data, SkImageDeserializer SkMemoryStream stream(data->data(), data->size()); return MakeFromStream(&stream, factory); } +#endif + +sk_sp<SkPicture> SkPicture::MakeFromStream(SkStream* stream) { + return MakeFromStream(stream, SkDeserialProcs(), nullptr); +} + +sk_sp<SkPicture> SkPicture::MakeFromData(const void* data, size_t size) { + if (!data) { + return nullptr; + } + SkMemoryStream stream(data, size); + return MakeFromStream(&stream, SkDeserialProcs(), nullptr); +} + +sk_sp<SkPicture> SkPicture::MakeFromData(const SkData* data) { + if (!data) { + return nullptr; + } + SkMemoryStream stream(data->data(), data->size()); + return MakeFromStream(&stream, SkDeserialProcs(), nullptr); +} + sk_sp<SkPicture> SkPicture::MakeFromData(const SkData* data, const SkDeserialProcs& procs) { if (!data) { @@ -365,24 +382,13 @@ bool SkPicture::PictureIOSecurityPrecautionsEnabled() { ////////////////////////////////////////////////////////////////////////////////////////////////// +#include "SkReadBuffer.h" +#include "SkWriteBuffer.h" + sk_sp<SkData> PixelSerializer_SkSerialImageProc(SkImage* img, void* ctx) { SkASSERT(ctx); return img->encodeToData(static_cast<SkPixelSerializer*>(ctx)); } - -sk_sp<SkImage> ImageDeserializer_SkDeserialImageProc(const void* data, size_t length, void* ctx) { - SkASSERT(ctx); - SkImageDeserializer* imd = static_cast<SkImageDeserializer*>(ctx); - const SkIRect* subset = nullptr; - return imd->makeFromMemory(data, length, subset); -} - -/////////////////////////////////////////////////////////////////////////////////////////////////// - -#ifdef SK_SUPPORT_LEGACY_SERIAL_BUFFER_OBJECTS -#include "SkReadBuffer.h" -#include "SkWriteBuffer.h" - void SkBinaryWriteBuffer::setPixelSerializer(sk_sp<SkPixelSerializer> ps) { fPS = ps; if (ps) { @@ -394,6 +400,13 @@ void SkBinaryWriteBuffer::setPixelSerializer(sk_sp<SkPixelSerializer> ps) { } } +#ifdef SK_SUPPORT_LEGACY_IMAGEDESERIALIZER +sk_sp<SkImage> ImageDeserializer_SkDeserialImageProc(const void* data, size_t length, void* ctx) { + SkASSERT(ctx); + SkImageDeserializer* imd = static_cast<SkImageDeserializer*>(ctx); + const SkIRect* subset = nullptr; + return imd->makeFromMemory(data, length, subset); +} void SkReadBuffer::setImageDeserializer(SkImageDeserializer* factory) { if (factory) { fProcs.fImageProc = ImageDeserializer_SkDeserialImageProc; diff --git a/src/core/SkReadBuffer.h b/src/core/SkReadBuffer.h index 4c0ca9bec2..28ea3d4d74 100644 --- a/src/core/SkReadBuffer.h +++ b/src/core/SkReadBuffer.h @@ -216,7 +216,7 @@ public: void setDeserialProcs(const SkDeserialProcs& procs); -#ifdef SK_SUPPORT_LEGACY_SERIAL_BUFFER_OBJECTS +#ifdef SK_SUPPORT_LEGACY_IMAGEDESERIALIZER void setImageDeserializer(SkImageDeserializer* factory); #endif diff --git a/src/image/SkImage.cpp b/src/image/SkImage.cpp index 01aceab773..31104d2548 100644 --- a/src/image/SkImage.cpp +++ b/src/image/SkImage.cpp @@ -405,6 +405,7 @@ sk_sp<SkImage> MakeTextureFromMipMap(GrContext*, const SkImageInfo&, const GrMip } /////////////////////////////////////////////////////////////////////////////////////////////////// +#ifdef SK_SUPPORT_LEGACY_IMAGEDESERIALIZER #include "SkImageDeserializer.h" sk_sp<SkImage> SkImageDeserializer::makeFromData(SkData* data, const SkIRect* subset) { @@ -414,6 +415,7 @@ sk_sp<SkImage> SkImageDeserializer::makeFromMemory(const void* data, size_t leng const SkIRect* subset) { return SkImage::MakeFromEncoded(SkData::MakeWithCopy(data, length), subset); } +#endif /////////////////////////////////////////////////////////////////////////////////////////////////// |