diff options
author | bungeman <bungeman@google.com> | 2016-08-02 14:40:46 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-08-02 14:40:46 -0700 |
commit | 38d909ec2875f79952de08f36adfaac5680d2c53 (patch) | |
tree | a11067146b42f3faa8531d6ef912f7246bbde1f2 /tests/PictureTest.cpp | |
parent | ada5a44f3bdac10a8a0c53b34f5add7aea9fdbb0 (diff) |
Move off SK_SUPPORT_LEGACY_DATA_FACTORIES.
This moves Skia code off of SK_SUPPORT_LEGACY_DATA_FACTORIES.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2206633004
Review-Url: https://codereview.chromium.org/2206633004
Diffstat (limited to 'tests/PictureTest.cpp')
-rw-r--r-- | tests/PictureTest.cpp | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/tests/PictureTest.cpp b/tests/PictureTest.cpp index 8f5ebd3f3b..749bd6e3d4 100644 --- a/tests/PictureTest.cpp +++ b/tests/PictureTest.cpp @@ -621,7 +621,7 @@ static void test_bad_bitmap() { } #endif -static SkData* serialized_picture_from_bitmap(const SkBitmap& bitmap) { +static sk_sp<SkData> serialized_picture_from_bitmap(const SkBitmap& bitmap) { SkPictureRecorder recorder; SkCanvas* canvas = recorder.beginRecording(SkIntToScalar(bitmap.width()), SkIntToScalar(bitmap.height())); @@ -632,7 +632,7 @@ static SkData* serialized_picture_from_bitmap(const SkBitmap& bitmap) { SkAutoTUnref<SkPixelSerializer> serializer( SkImageEncoder::CreatePixelSerializer()); picture->serialize(&wStream, serializer); - return wStream.copyToData(); + return sk_sp<SkData>(wStream.copyToData()); } struct ErrorContext { @@ -670,18 +670,18 @@ DEF_TEST(Picture_EncodedData, reporter) { if (!SkImageEncoder::EncodeStream(&wStream, original, SkImageEncoder::kPNG_Type, 100)) { return; } - SkAutoDataUnref data(wStream.copyToData()); + sk_sp<SkData> data(wStream.copyToData()); SkBitmap bm; - bool installSuccess = SkDEPRECATED_InstallDiscardablePixelRef(data, &bm); + bool installSuccess = SkDEPRECATED_InstallDiscardablePixelRef(data.get(), &bm); REPORTER_ASSERT(reporter, installSuccess); // Write both bitmaps to pictures, and ensure that the resulting data streams are the same. // Flattening original will follow the old path of performing an encode, while flattening bm // will use the already encoded data. - SkAutoDataUnref picture1(serialized_picture_from_bitmap(original)); - SkAutoDataUnref picture2(serialized_picture_from_bitmap(bm)); - REPORTER_ASSERT(reporter, picture1->equals(picture2)); + sk_sp<SkData> picture1(serialized_picture_from_bitmap(original)); + sk_sp<SkData> picture2(serialized_picture_from_bitmap(bm)); + REPORTER_ASSERT(reporter, picture1->equals(picture2.get())); // Now test that a parse error was generated when trying to create a new SkPicture without // providing a function to decode the bitmap. @@ -689,7 +689,7 @@ DEF_TEST(Picture_EncodedData, reporter) { context.fErrors = 0; context.fReporter = reporter; SkSetErrorCallback(assert_one_parse_error_cb, &context); - SkMemoryStream pictureStream(picture1); + SkMemoryStream pictureStream(std::move(picture1)); SkClearLastError(); sk_sp<SkPicture> pictureFromStream(SkPicture::MakeFromStream(&pictureStream, nullptr)); REPORTER_ASSERT(reporter, pictureFromStream.get() != nullptr); |