diff options
author | 2017-09-15 21:03:54 +0000 | |
---|---|---|
committer | 2017-09-15 21:04:07 +0000 | |
commit | 7031b247c9fe0cb8fa32129f9bc24fea2043cee2 (patch) | |
tree | 6ded4ad2071a03530ead08e4f5e1a41f4c42c563 /src/core/SkRWBuffer.cpp | |
parent | f95352322496796ce4c99df9582dbc630fe8a327 (diff) |
Revert "use unique_ptr for stream api"
This reverts commit 49f1f34438d3431f6d7e32847accd2ba96948a73.
Reason for revert: broke win-chrome
../../third_party/skia/src/ports/SkFontMgr_win_dw.cpp(89): error C2228: left of '.release' must have class/struct/union
../../third_party/skia/src/ports/SkFontMgr_win_dw.cpp(89): note: type is 'SkStreamAsset *'
../../third_party/skia/src/ports/SkFontMgr_win_dw.cpp(89): note: did you intend to use '->' instead?
Original change's description:
> use unique_ptr for stream api
>
> Bug: skia:6888
> Change-Id: I3459b4913982a7cae1c0061697c82cc65ad9a2d8
> Reviewed-on: https://skia-review.googlesource.com/26740
> Reviewed-by: Mike Klein <mtklein@chromium.org>
> Commit-Queue: Mike Reed <reed@google.com>
TBR=mtklein@chromium.org,mtklein@google.com,bungeman@google.com,reed@google.com
Change-Id: Ic1e4af557317abd06b7f6b7f5056645df7e469f0
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: skia:6888
Reviewed-on: https://skia-review.googlesource.com/47440
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Mike Reed <reed@google.com>
Diffstat (limited to 'src/core/SkRWBuffer.cpp')
-rw-r--r-- | src/core/SkRWBuffer.cpp | 14 |
1 files changed, 1 insertions, 13 deletions
diff --git a/src/core/SkRWBuffer.cpp b/src/core/SkRWBuffer.cpp index 8ec2baafac..3ac9677793 100644 --- a/src/core/SkRWBuffer.cpp +++ b/src/core/SkRWBuffer.cpp @@ -315,9 +315,7 @@ public: return fBuffer->size() == fGlobalOffset; } -#ifdef SK_SUPPORT_LEGACY_STREAM_API SkStreamAsset* duplicate() const override { return new SkROBufferStreamAsset(fBuffer); } -#endif size_t getPosition() const override { return fGlobalOffset; @@ -343,24 +341,14 @@ public: return true; } -#ifdef SK_SUPPORT_LEGACY_STREAM_API SkStreamAsset* fork() const override { SkStreamAsset* clone = this->duplicate(); clone->seek(this->getPosition()); return clone; } -#endif -private: -#ifndef SK_SUPPORT_LEGACY_STREAM_API - SkStreamAsset* onDuplicate() const override { return new SkROBufferStreamAsset(fBuffer); } - SkStreamAsset* onFork() const override { - auto clone = this->duplicate(); - clone->seek(this->getPosition()); - return clone.release(); - } -#endif +private: sk_sp<SkROBuffer> fBuffer; SkROBuffer::Iter fIter; size_t fLocalOffset; |