Make SkStream *not* ref counted.
authorscroggo <scroggo@google.com>
Wed, 21 Jan 2015 20:09:53 +0000 (12:09 -0800)
committerCommit bot <commit-bot@chromium.org>
Wed, 21 Jan 2015 20:09:53 +0000 (12:09 -0800)
commita1193e4b0e34a7e4e1bd33e9708d7341679f8321
treee63e73eeadfd6d979e81596804e34e7af8fa0390
parentfd1ad48d4d9073b90f58e60219637046a8446267
Make SkStream *not* ref counted.

SkStream is a stateful object, so it does not make sense for it to have
multiple owners. Make SkStream inherit directly from SkNoncopyable.

Update methods which previously called SkStream::ref() (e.g.
SkImageDecoder::buildTileIndex() and SkFrontBufferedStream::Create(),
which required the existing owners to call SkStream::unref()) to take
ownership of their SkStream parameters and delete when done (including
on failure).

Switch all SkAutoTUnref<SkStream>s to SkAutoTDelete<SkStream>s. In some
cases this means heap allocating streams that were previously stack
allocated.

Respect ownership rules of SkTypeface::CreateFromStream() and
SkImageDecoder::buildTileIndex().

Update the comments for exceptional methods which do not affect the
ownership of their SkStream parameters (e.g.
SkPicture::CreateFromStream() and SkTypeface::Deserialize()) to be
explicit about ownership.

Remove test_stream_life, which tested that buildTileIndex() behaved
correctly when SkStream was a ref counted object. The test does not
make sense now that it is not.

In SkPDFStream, remove the SkMemoryStream member. Instead of using it,
create a new SkMemoryStream to pass to fDataStream (which is now an
SkAutoTDelete).

Make other pdf rasterizers behave like SkPDFDocumentToBitmap.

SkPDFDocumentToBitmap delete the SkStream, so do the same in the
following pdf rasterizers:

SkPopplerRasterizePDF
SkNativeRasterizePDF
SkNoRasterizePDF

Requires a change to Android, which currently treats SkStreams as ref
counted objects.

Review URL: https://codereview.chromium.org/849103004
72 files changed:
bench/nanobench.cpp
debugger/QT/SkDebuggerGUI.cpp
dm/DMSrcSink.cpp
dm/DMSrcSink.h
experimental/PdfViewer/inc/SkPdfRenderer.h
experimental/PdfViewer/pdfparser/native/SkPdfNativeDoc.h
gm/coloremoji.cpp
gm/dcshader.cpp
gm/dftext.cpp
include/core/SkImageDecoder.h
include/core/SkPicture.h
include/core/SkStream.h
include/core/SkTypeface.h
include/images/SkDecodingImageGenerator.h
include/utils/SkFrontBufferedStream.h
include/utils/mac/SkCGUtils.h
samplecode/SampleAnimator.cpp
src/animator/SkAnimateMaker.cpp
src/animator/SkAnimator.cpp
src/animator/SkDrawBitmap.cpp
src/core/SkFontDescriptor.h
src/core/SkFontMgr.cpp
src/core/SkPaint.cpp
src/core/SkPictureData.h
src/core/SkStream.cpp
src/core/SkTypeface.cpp
src/device/xps/SkXPSDevice.cpp
src/fonts/SkFontMgr_fontconfig.cpp
src/fonts/SkFontMgr_indirect.cpp
src/gpu/gl/GrGLPathRendering.cpp
src/images/SkDecodingImageGenerator.cpp
src/images/SkImageDecoder.cpp
src/images/SkImageDecoder_libjpeg.cpp
src/images/SkImageDecoder_libpng.cpp
src/images/SkImageDecoder_libwebp.cpp
src/images/SkJpegUtility.cpp
src/images/SkJpegUtility.h
src/images/SkMovie.cpp
src/pdf/SkPDFFont.cpp
src/pdf/SkPDFFormXObject.cpp
src/pdf/SkPDFImage.cpp
src/pdf/SkPDFImage.h
src/pdf/SkPDFShader.cpp
src/pdf/SkPDFStream.cpp
src/pdf/SkPDFStream.h
src/ports/SkFontConfigTypeface.h
src/ports/SkFontHost_FreeType.cpp
src/ports/SkFontHost_fontconfig.cpp
src/ports/SkFontHost_linux.cpp
src/ports/SkFontHost_win.cpp
src/ports/SkFontMgr_android.cpp
src/ports/SkFontMgr_fontconfig.cpp
src/ports/SkFontMgr_win_dw.cpp
src/ports/SkImageDecoder_empty.cpp
src/ports/SkTypeface_win_dw.cpp
src/sfnt/SkOTUtils.h
src/utils/SkFrontBufferedStream.cpp
src/utils/SkPDFRasterizer.cpp
src/utils/SkPDFRasterizer.h
src/utils/mac/SkStream_mac.cpp
src/utils/win/SkDWriteFontFileStream.cpp
src/utils/win/SkDWriteFontFileStream.h
src/views/animated/SkWidgetViews.cpp
tests/FontHostStreamTest.cpp
tests/FrontBufferedStreamTest.cpp
tests/ImageDecodingTest.cpp
tests/KtxTest.cpp
tests/PDFPrimitivesTest.cpp
tests/SerializationTest.cpp
tests/StreamTest.cpp
tools/dump_record.cpp
tools/lua/lua_pictures.cpp