SkImageEncoder: simplify API
authorHal Canary <halcanary@google.com>
Wed, 23 Nov 2016 15:55:18 +0000 (08:55 -0700)
committerSkia Commit-Bot <skia-commit-bot@chromium.org>
Wed, 23 Nov 2016 16:40:32 +0000 (16:40 +0000)
commitdb6830162eca5b94e61d9825ec93306fc615d204
tree02be1aa930879c2b389e80beb4910c8d27027257
parent45580d3e3024c1536e8e1b2017b704805442b634
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>
67 files changed:
BUILD.gn
bench/EncoderBench.cpp
bench/GLBench.cpp
bench/nanobench.cpp
cmake/example.cpp
debugger/QT/SkDebuggerGUI.cpp
dm/DM.cpp
fuzz/fuzz.cpp
gm/encode-platform.cpp
gm/encode.cpp
gm/image.cpp
gm/image_shader.cpp
gn/android_framework_defines.gni
include/codec/SkAndroidCodec.h
include/codec/SkCodec.h
include/codec/SkEncodedFormat.h
include/core/SkEncodedImageFormat.h [new file with mode: 0644]
include/core/SkImage.h
include/core/SkImageEncoder.h
public.bzl
samplecode/SampleApp.cpp
samplecode/SampleSlides.cpp
src/android/SkBitmapRegionCodec.h
src/android/SkBitmapRegionDecoder.cpp
src/animator/SkSnapshot.cpp
src/animator/SkSnapshot.h
src/codec/SkAndroidCodec.cpp
src/codec/SkBmpCodec.h
src/codec/SkGifCodec.h
src/codec/SkIcoCodec.h
src/codec/SkJpegCodec.h
src/codec/SkPngCodec.h
src/codec/SkRawAdapterCodec.h
src/codec/SkRawCodec.h
src/codec/SkSampledCodec.cpp
src/codec/SkWbmpCodec.cpp
src/codec/SkWbmpCodec.h
src/codec/SkWebpCodec.h
src/gpu/GrSurface.cpp
src/image/SkImage.cpp
src/images/SkForceLinking.cpp
src/images/SkImageEncoder.cpp
src/images/SkImageEncoderPriv.h [new file with mode: 0644]
src/images/SkImageEncoder_Factory.cpp
src/images/SkJPEGImageEncoder.cpp
src/images/SkKTXImageEncoder.cpp
src/images/SkPNGImageEncoder.cpp
src/images/SkWEBPImageEncoder.cpp
src/ports/SkImageEncoder_CG.cpp
src/ports/SkImageEncoder_WIC.cpp
src/ports/SkImageEncoder_none.cpp
src/svg/SkSVGDevice.cpp
src/xps/SkXPSDevice.cpp
tests/BlitRowTest.cpp
tests/CodecTest.cpp
tests/ImageTest.cpp
tests/PDFDocumentTest.cpp
tests/PathOpsConicIntersectionTest.cpp
tests/PathOpsSkpClipTest.cpp
tests/SkpSkGrTest.cpp
tools/colorspaceinfo.cpp
tools/get_images_from_skps.cpp
tools/imgblur.cpp
tools/picture_utils.cpp
tools/sk_tool_utils.h
tools/skdiff/skdiff_utils.cpp
tools/skiaserve/Request.cpp