diff options
author | 2018-01-27 17:30:04 +0000 | |
---|---|---|
committer | 2018-01-27 17:30:15 +0000 | |
commit | 88d90714fad0fed2739fb8c5f90fe814c8a274ef (patch) | |
tree | 85d26395c1fbcdd21ca0e2029b022862ca09fe79 /tests/ImageTest.cpp | |
parent | fc20008819f39e1504aa572e28c56e83e14bff79 (diff) |
Revert "hide picture virtuals (no public callers)"
This reverts commit 8005bff7e631a269f0dfaae93ff9963dc0e5ff39.
Reason for revert: hwui, flutter, and headless blink in G3 all still using these.
Original change's description:
> hide picture virtuals (no public callers)
>
> This prepares the way for a clean impl of a "placeholder" picture that never unrolls
>
> Bug: skia:
> Change-Id: I3b5785c5c94432b54e9a7dc280b2a6e716592473
> Reviewed-on: https://skia-review.googlesource.com/100260
> Commit-Queue: Mike Reed <reed@google.com>
> Reviewed-by: Mike Klein <mtklein@chromium.org>
TBR=mtklein@chromium.org,mtklein@google.com,reed@google.com
Change-Id: I385789dd420588ea9a9390c8a44c6ecb96c7f358
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: skia:
Reviewed-on: https://skia-review.googlesource.com/100880
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>
Diffstat (limited to 'tests/ImageTest.cpp')
-rw-r--r-- | tests/ImageTest.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/tests/ImageTest.cpp b/tests/ImageTest.cpp index 3efb72e919..6cc0fee4d9 100644 --- a/tests/ImageTest.cpp +++ b/tests/ImageTest.cpp @@ -233,6 +233,7 @@ DEF_TEST(Image_Serialize_Encoding_Failure, reporter) { canvas->drawImage(image, 0, 0); sk_sp<SkPicture> picture(recorder.finishRecordingAsPicture()); REPORTER_ASSERT(reporter, picture); + REPORTER_ASSERT(reporter, picture->approximateOpCount() > 0); bool was_called = false; SkSerialProcs procs; @@ -249,6 +250,7 @@ DEF_TEST(Image_Serialize_Encoding_Failure, reporter) { auto deserialized = SkPicture::MakeFromData(data->data(), data->size()); REPORTER_ASSERT(reporter, deserialized); + REPORTER_ASSERT(reporter, deserialized->approximateOpCount() > 0); } // Test that a draw that only partially covers the drawing surface isn't |