diff options
| author |  Hal Canary <halcanary@google.com> | 2016-11-23 08:55:18 -0700 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2016-11-23 16:40:32 +0000 |
| commit | db6830162eca5b94e61d9825ec93306fc615d204 (patch) | |
| tree | 02be1aa930879c2b389e80beb4910c8d27027257 /tools/get_images_from_skps.cpp | |
| parent | 45580d3e3024c1536e8e1b2017b704805442b634 (diff) | |
SkImageEncoder: simplify API
(re-land 248ff02 & 2cb6cb7, with changes)
- Hide SkImageEncoder class in private header.
- SkImageEncoder::Type becomes SkEncodedImageFormat
- SkEncodedFormat becomes SkEncodedImageFormat
- SkImageEncoder static functions replaced with
single function EncodeImage()
- utility wrappers for EncodeImage() are in
sk_tool_utils.h
TODO: remove link-time registration mechanism.
TODO: clean up clients use of API and flip the flag.
TODO: implement EncodeImage() in chromeium/skia/ext
Change-Id: I47d451e50be4d5c6c130869c7fa7c2857243d9f0
Reviewed-on: https://skia-review.googlesource.com/4909
Reviewed-by: Mike Reed <reed@google.com>
Reviewed-by: Leon Scroggins <scroggo@google.com>
Reviewed-on: https://skia-review.googlesource.com/5186
Commit-Queue: Hal Canary <halcanary@google.com>
Reviewed-by: Hal Canary <halcanary@google.com>
Diffstat (limited to 'tools/get_images_from_skps.cpp')
| -rw-r--r-- | tools/get_images_from_skps.cpp | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/tools/get_images_from_skps.cpp b/tools/get_images_from_skps.cpp index 6cd35122ad..d3a2343b88 100644 --- a/tools/get_images_from_skps.cpp +++ b/tools/get_images_from_skps.cpp @@ -65,14 +65,14 @@ struct Sniffer : public SkPixelSerializer { } SkString ext; switch (codec->getEncodedFormat()) { - case SkEncodedFormat::kBMP_SkEncodedFormat: ext = "bmp"; break; - case SkEncodedFormat::kGIF_SkEncodedFormat: ext = "gif"; break; - case SkEncodedFormat::kICO_SkEncodedFormat: ext = "ico"; break; - case SkEncodedFormat::kJPEG_SkEncodedFormat: ext = "jpg"; break; - case SkEncodedFormat::kPNG_SkEncodedFormat: ext = "png"; break; - case SkEncodedFormat::kDNG_SkEncodedFormat: ext = "dng"; break; - case SkEncodedFormat::kWBMP_SkEncodedFormat: ext = "wbmp"; break; - case SkEncodedFormat::kWEBP_SkEncodedFormat: ext = "webp"; break; + case SkEncodedImageFormat::kBMP: ext = "bmp"; break; + case SkEncodedImageFormat::kGIF: ext = "gif"; break; + case SkEncodedImageFormat::kICO: ext = "ico"; break; + case SkEncodedImageFormat::kJPEG: ext = "jpg"; break; + case SkEncodedImageFormat::kPNG: ext = "png"; break; + case SkEncodedImageFormat::kDNG: ext = "dng"; break; + case SkEncodedImageFormat::kWBMP: ext = "wbmp"; break; + case SkEncodedImageFormat::kWEBP: ext = "webp"; break; default: // This should be unreachable because we cannot create a codec if we do not know // the image type. |