diff options
author | vandebo@chromium.org <vandebo@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2013-01-04 18:11:21 +0000 |
---|---|---|
committer | vandebo@chromium.org <vandebo@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2013-01-04 18:11:21 +0000 |
commit | 6eb549e8ca3d88d7536859fd5aa3343fc3011f2f (patch) | |
tree | d97411a5a61648e62f2e23e6139982681b9fa6d1 /src/pdf/SkPDFStream.cpp | |
parent | e8a76ae8edc4f90456f9d8f90e56bf97f2657f3a (diff) |
Revert "Remove SkRefPtr" - r7021
samplecode/ still needs to be updated.
Review URL: https://codereview.appspot.com/7032048
git-svn-id: http://skia.googlecode.com/svn/trunk@7022 2bbb7eff-a529-9590-31e7-b0007b416f81
Diffstat (limited to 'src/pdf/SkPDFStream.cpp')
-rw-r--r-- | src/pdf/SkPDFStream.cpp | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/src/pdf/SkPDFStream.cpp b/src/pdf/SkPDFStream.cpp index 38e874ea1b..d113a0b355 100644 --- a/src/pdf/SkPDFStream.cpp +++ b/src/pdf/SkPDFStream.cpp @@ -21,20 +21,19 @@ static bool skip_compression(SkPDFCatalog* catalog) { SkPDFStream::SkPDFStream(SkStream* stream) : fState(kUnused_State), fData(stream) { - SkSafeRef(stream); } SkPDFStream::SkPDFStream(SkData* data) : fState(kUnused_State) { SkMemoryStream* stream = new SkMemoryStream; stream->setData(data); - fData.reset(stream); // Transfer ownership. + fData = stream; + fData->unref(); // SkRefPtr and new both took a reference. } SkPDFStream::SkPDFStream(const SkPDFStream& pdfStream) : SkPDFDict(), fState(kUnused_State), - fData(pdfStream.fData.get()) { - fData.get()->ref(); + fData(pdfStream.fData) { bool removeLength = true; // Don't uncompress an already compressed stream, but we could. if (pdfStream.fState == kCompressed_State) { @@ -85,8 +84,7 @@ size_t SkPDFStream::getOutputSize(SkPDFCatalog* catalog, bool indirect) { SkPDFStream::SkPDFStream() : fState(kUnused_State) {} void SkPDFStream::setData(SkStream* stream) { - fData.reset(stream); - SkSafeRef(stream); + fData = stream; } bool SkPDFStream::populate(SkPDFCatalog* catalog) { @@ -98,7 +96,8 @@ bool SkPDFStream::populate(SkPDFCatalog* catalog) { if (compressedData.getOffset() < fData->getLength()) { SkMemoryStream* stream = new SkMemoryStream; stream->setData(compressedData.copyToData())->unref(); - fData.reset(stream); // Transfer ownership. + fData = stream; + fData->unref(); // SkRefPtr and new both took a reference. insertName("Filter", "FlateDecode"); } fState = kCompressed_State; @@ -109,7 +108,8 @@ bool SkPDFStream::populate(SkPDFCatalog* catalog) { } else if (fState == kNoCompression_State && !skip_compression(catalog) && SkFlate::HaveFlate()) { if (!fSubstitute.get()) { - fSubstitute.reset(new SkPDFStream(*this)); + fSubstitute = new SkPDFStream(*this); + fSubstitute->unref(); // SkRefPtr and new both took a reference. catalog->setSubstitute(this, fSubstitute.get()); } return false; |