diff options
| author |  robertphillips <robertphillips@google.com> | 2014-06-10 11:45:46 -0700 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2014-06-10 11:45:46 -0700 |
| commit | 9058d602d0303c4d601bea260929367a8be37f04 (patch) | |
| tree | 667886d747a78cea15bea7f25a32969c1612713c /src/core/SkPicturePlayback.h | |
| parent | 77ec7a6f44ccfdacc9c3422516f09720ca17f9e2 (diff) | |
Fix error revealed by Android unit test
The issue is/was that the original Picture/PictureRecorder that is being partially replayed is not guaranteed to issue any more commands before attempting to modify the existing data. Such modification is prohibited if there is a extant copy-on-write snapshot. Rather then further complicate the SkWriter32::snapshot capability for a dis-preferred use case, this CL simply copies the operation data.
R=scroggo@google.com, reed@google.com
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/316063005
Diffstat (limited to 'src/core/SkPicturePlayback.h')
| -rw-r--r-- | src/core/SkPicturePlayback.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/core/SkPicturePlayback.h b/src/core/SkPicturePlayback.h index 91429dbd07..9d4c996653 100644 --- a/src/core/SkPicturePlayback.h +++ b/src/core/SkPicturePlayback.h @@ -78,7 +78,8 @@ class SkPicturePlayback { public: SkPicturePlayback(const SkPicture* picture, const SkPicturePlayback& src, SkPictCopyInfo* deepCopyInfo = NULL); - SkPicturePlayback(const SkPicture* picture, const SkPictureRecord& record, const SkPictInfo&); + SkPicturePlayback(const SkPicture* picture, const SkPictureRecord& record, const SkPictInfo&, + bool deepCopyOps); static SkPicturePlayback* CreateFromStream(SkPicture* picture, SkStream*, const SkPictInfo&, |