remove discardablepixelref

BUG=skia:

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=3883

Change-Id: Ica284be78563a4ccd6e0cb07404064fb5511ba57
Reviewed-on: https://skia-review.googlesource.com/3883
Commit-Queue: Mike Reed <reed@google.com>
Reviewed-by: Florin Malita <fmalita@chromium.org>

4 files changed, 3 insertions, 317 deletions

diff --git a/tests/CachedDecodingPixelRefTest.cpp b/tests/CachedDecodingPixelRefTest.cpp
index b6253c73a2..b65765f96c 100644
--- a/tests/CachedDecodingPixelRefTest.cpp
+++ b/tests/CachedDecodingPixelRefTest.cpp
@@ -11,149 +11,13 @@
 #include "SkDiscardableMemoryPool.h"
 #include "SkImage.h"
 #include "SkImageEncoder.h"
-#include "SkImageGeneratorPriv.h"
+#include "SkImageGenerator.h"
 #include "SkResourceCache.h"
 #include "SkStream.h"
 #include "SkUtils.h"
 
 #include "Test.h"
 
-/**
- * Fill this bitmap with some color.
- */
-static void make_test_image(SkBitmap* bm) {
-    const int W = 50, H = 50;
-    bm->allocN32Pixels(W, H);
-    bm->eraseColor(SK_ColorBLACK);
-    SkCanvas canvas(*bm);
-    SkPaint paint;
-    paint.setColor(SK_ColorBLUE);
-    canvas.drawRectCoords(0, 0, SkIntToScalar(W/2),
-                          SkIntToScalar(H/2), paint);
-    paint.setColor(SK_ColorWHITE);
-    canvas.drawRectCoords(SkIntToScalar(W/2), SkIntToScalar(H/2),
-                          SkIntToScalar(W), SkIntToScalar(H), paint);
-}
-
-/**
- * encode this bitmap into some data via SkImageEncoder
- */
-static sk_sp<SkData> create_data_from_bitmap(const SkBitmap& bm, SkImageEncoder::Type type) {
-    SkDynamicMemoryWStream stream;
-    if (SkImageEncoder::EncodeStream(&stream, bm, type, 100)) {
-        return stream.detachAsData();
-    }
-    return nullptr;
-}
-
-////////////////////////////////////////////////////////////////////////////////
-
-static void compare_bitmaps(skiatest::Reporter* reporter,
-                            const SkBitmap& b1, const SkBitmap& b2,
-                            bool pixelPerfect = true) {
-    REPORTER_ASSERT(reporter, b1.empty() == b2.empty());
-    REPORTER_ASSERT(reporter, b1.width() == b2.width());
-    REPORTER_ASSERT(reporter, b1.height() == b2.height());
-    REPORTER_ASSERT(reporter, b1.isNull() == b2.isNull());
-    SkAutoLockPixels autoLockPixels1(b1);
-    SkAutoLockPixels autoLockPixels2(b2);
-    REPORTER_ASSERT(reporter, b1.isNull() == b2.isNull());
-    if (b1.isNull() || b1.empty()) {
-        return;
-    }
-    REPORTER_ASSERT(reporter, b1.getPixels());
-    REPORTER_ASSERT(reporter, b2.getPixels());
-    if ((!(b1.getPixels())) || (!(b2.getPixels()))) {
-        return;
-    }
-    if ((b1.width() != b2.width()) ||
-        (b1.height() != b2.height())) {
-        return;
-    }
-    if (!pixelPerfect) {
-        return;
-    }
-
-    int pixelErrors = 0;
-    for (int y = 0; y < b2.height(); ++y) {
-        for (int x = 0; x < b2.width(); ++x) {
-            if (b1.getColor(x, y) != b2.getColor(x, y)) {
-                ++pixelErrors;
-            }
-        }
-    }
-    REPORTER_ASSERT(reporter, 0 == pixelErrors);
-}
-
-typedef bool (*InstallEncoded)(SkData* encoded, SkBitmap* dst);
-
-/**
-   This function tests three differently encoded images against the
-   original bitmap */
-static void test_three_encodings(skiatest::Reporter* reporter,
-                                 InstallEncoded install) {
-    SkBitmap original;
-    make_test_image(&original);
-    REPORTER_ASSERT(reporter, !original.empty());
-    REPORTER_ASSERT(reporter, !original.isNull());
-    if (original.empty() || original.isNull()) {
-        return;
-    }
-    static const SkImageEncoder::Type types[] = {
-        SkImageEncoder::kPNG_Type,
-        SkImageEncoder::kJPEG_Type,
-        SkImageEncoder::kWEBP_Type
-    };
-    for (size_t i = 0; i < SK_ARRAY_COUNT(types); i++) {
-        SkImageEncoder::Type type = types[i];
-        sk_sp<SkData> encoded = create_data_from_bitmap(original, type);
-        REPORTER_ASSERT(reporter, encoded.get() != nullptr);
-        if (nullptr == encoded.get()) {
-            continue;
-        }
-        SkBitmap lazy;
-        bool installSuccess = install(encoded.get(), &lazy);
-        REPORTER_ASSERT(reporter, installSuccess);
-        if (!installSuccess) {
-            continue;
-        }
-        REPORTER_ASSERT(reporter, nullptr == lazy.getPixels());
-        {
-            SkAutoLockPixels autoLockPixels(lazy);  // now pixels are good.
-            REPORTER_ASSERT(reporter, lazy.getPixels());
-            if (nullptr == lazy.getPixels()) {
-                continue;
-            }
-        }
-        // pixels should be gone!
-        REPORTER_ASSERT(reporter, nullptr == lazy.getPixels());
-        {
-            SkAutoLockPixels autoLockPixels(lazy);  // now pixels are good.
-            REPORTER_ASSERT(reporter, lazy.getPixels());
-            if (nullptr == lazy.getPixels()) {
-                continue;
-            }
-        }
-        bool comparePixels = (SkImageEncoder::kPNG_Type == type);
-        compare_bitmaps(reporter, original, lazy, comparePixels);
-    }
-}
-
-////////////////////////////////////////////////////////////////////////////////
-static bool install_skDiscardablePixelRef(SkData* encoded, SkBitmap* dst) {
-    // Use system-default discardable memory.
-    return SkDEPRECATED_InstallDiscardablePixelRef(encoded, dst);
-}
-
-////////////////////////////////////////////////////////////////////////////////
-/**
- *  This checks to see that SkDiscardablePixelRef works as advertised with a
- *  SkDecodingImageGenerator.
- */
-DEF_TEST(DecodingImageGenerator, reporter) {
-    test_three_encodings(reporter, install_skDiscardablePixelRef);
-}
-
 class TestImageGenerator : public SkImageGenerator {
 public:
     enum TestType {
@@ -227,82 +91,6 @@ private:
     typedef SkImageGenerator INHERITED;
 };
 
-static void check_test_image_generator_bitmap(skiatest::Reporter* reporter,
-                                              const SkBitmap& bm) {
-    REPORTER_ASSERT(reporter, TestImageGenerator::Width() == bm.width());
-    REPORTER_ASSERT(reporter, TestImageGenerator::Height() == bm.height());
-    SkAutoLockPixels autoLockPixels(bm);
-    REPORTER_ASSERT(reporter, bm.getPixels());
-    if (nullptr == bm.getPixels()) {
-        return;
-    }
-    int errors = 0;
-    for (int y = 0; y < bm.height(); ++y) {
-        for (int x = 0; x < bm.width(); ++x) {
-            if (TestImageGenerator::Color() != bm.getColor(x, y)) {
-                ++errors;
-            }
-        }
-    }
-    REPORTER_ASSERT(reporter, 0 == errors);
-}
-
-static void check_pixelref(TestImageGenerator::TestType type,
-                           skiatest::Reporter* reporter,
-                           SkDiscardableMemory::Factory* factory,
-                           SkColorType colorType) {
-    SkAutoTDelete<SkImageGenerator> gen(new TestImageGenerator(type, reporter, colorType));
-    REPORTER_ASSERT(reporter, gen.get() != nullptr);
-    SkBitmap lazy;
-    bool success = SkDEPRECATED_InstallDiscardablePixelRef(gen.release(), nullptr, &lazy, factory);
-
-    REPORTER_ASSERT(reporter, success);
-    if (TestImageGenerator::kSucceedGetPixels_TestType == type) {
-        check_test_image_generator_bitmap(reporter, lazy);
-    } else if (TestImageGenerator::kFailGetPixels_TestType == type) {
-        SkAutoLockPixels autoLockPixels(lazy);
-        REPORTER_ASSERT(reporter, nullptr == lazy.getPixels());
-    }
-}
-
-/**
- *  This tests the basic functionality of SkDiscardablePixelRef with a
- *  basic SkImageGenerator implementation and several
- *  SkDiscardableMemory::Factory choices.
- */
-DEF_TEST(DiscardableAndCachingPixelRef, reporter) {
-    const SkColorType testColorTypes[] = {
-        kN32_SkColorType,
-        kIndex_8_SkColorType,
-        kRGB_565_SkColorType
-    };
-    for (const SkColorType testColorType : testColorTypes) {
-        check_pixelref(TestImageGenerator::kFailGetPixels_TestType, reporter, nullptr,
-                       testColorType);
-        check_pixelref(TestImageGenerator::kSucceedGetPixels_TestType, reporter, nullptr,
-                       testColorType);
-
-        SkAutoTUnref<SkDiscardableMemoryPool> pool(
-            SkDiscardableMemoryPool::Create(1, nullptr));
-        REPORTER_ASSERT(reporter, 0 == pool->getRAMUsed());
-        check_pixelref(TestImageGenerator::kFailGetPixels_TestType, reporter, pool,
-                       testColorType);
-        REPORTER_ASSERT(reporter, 0 == pool->getRAMUsed());
-        check_pixelref(TestImageGenerator::kSucceedGetPixels_TestType, reporter, pool,
-                       testColorType);
-        REPORTER_ASSERT(reporter, 0 == pool->getRAMUsed());
-
-        SkDiscardableMemoryPool* globalPool = SkGetGlobalDiscardableMemoryPool();
-        // Only acts differently from nullptr on a platform that has a
-        // default discardable memory implementation that differs from the
-        // global DM pool.
-        check_pixelref(TestImageGenerator::kFailGetPixels_TestType, reporter, globalPool,
-                       testColorType);
-        check_pixelref(TestImageGenerator::kSucceedGetPixels_TestType, reporter, globalPool,
-                       testColorType);
-    }
-}
-
 ////////////////////////////////////////////////////////////////////////////////
 
 DEF_TEST(Image_NewFromGenerator, r) {
diff --git a/tests/DrawBitmapRectTest.cpp b/tests/DrawBitmapRectTest.cpp
index 23fa2448a0..2fbe1908c1 100644
--- a/tests/DrawBitmapRectTest.cpp
+++ b/tests/DrawBitmapRectTest.cpp
@@ -9,7 +9,7 @@
 #include "SkCanvas.h"
 #include "SkData.h"
 #include "SkDiscardableMemoryPool.h"
-#include "SkImageGeneratorPriv.h"
+#include "SkImageGenerator.h"
 #include "SkMatrixUtils.h"
 #include "SkPaint.h"
 #include "SkPath.h"
diff --git a/tests/PDFJpegEmbedTest.cpp b/tests/PDFJpegEmbedTest.cpp
index 2e75c3998a..17dbac8cfa 100644
--- a/tests/PDFJpegEmbedTest.cpp
+++ b/tests/PDFJpegEmbedTest.cpp
@@ -15,14 +15,6 @@
 #include "Resources.h"
 #include "Test.h"
 
-// Returned bitmap is lazy.  Only lazy bitmaps hold onto the original data.
-static SkBitmap bitmap_from_data(SkData* data) {
-    SkASSERT(data);
-    SkBitmap bm;
-    SkDEPRECATED_InstallDiscardablePixelRef(data, &bm);
-    return bm;
-}
-
 static bool is_subset_of(SkData* smaller, SkData* larger) {
     SkASSERT(smaller && larger);
     if (smaller->size() > larger->size()) {
@@ -70,29 +62,6 @@ DEF_TEST(SkPDF_JpegEmbedTest, r) {
 
     canvas->clear(SK_ColorLTGRAY);
 
-    SkBitmap bm1(bitmap_from_data(mandrillData.get()));
-    canvas->drawBitmap(bm1, 65.0, 0.0, nullptr);
-    SkBitmap bm2(bitmap_from_data(cmykData.get()));
-    canvas->drawBitmap(bm2, 0.0, 512.0, nullptr);
-
-    canvas->flush();
-    document->endPage();
-    document->close();
-    sk_sp<SkData> pdfData(pdf.detachAsData());
-    SkASSERT(pdfData);
-
-    REPORTER_ASSERT(r, is_subset_of(mandrillData.get(), pdfData.get()));
-
-    // This JPEG uses a nonstandard colorspace - it can not be
-    // embedded into the PDF directly.
-    REPORTER_ASSERT(r, !is_subset_of(cmykData.get(), pdfData.get()));
-    ////////////////////////////////////////////////////////////////////////////
-    pdf.reset();
-    document = SkDocument::MakePDF(&pdf);
-    canvas = document->beginPage(642, 1028);
-
-    canvas->clear(SK_ColorLTGRAY);
-
     sk_sp<SkImage> im1(SkImage::MakeFromEncoded(mandrillData));
     canvas->drawImage(im1.get(), 65.0, 0.0, nullptr);
     sk_sp<SkImage> im2(SkImage::MakeFromEncoded(cmykData));
@@ -101,7 +70,7 @@ DEF_TEST(SkPDF_JpegEmbedTest, r) {
     canvas->flush();
     document->endPage();
     document->close();
-    pdfData = pdf.detachAsData();
+    sk_sp<SkData> pdfData = pdf.detachAsData();
     SkASSERT(pdfData);
 
     REPORTER_ASSERT(r, is_subset_of(mandrillData.get(), pdfData.get()));
diff --git a/tests/PictureTest.cpp b/tests/PictureTest.cpp
index 3aca69f4e1..8c7312ef07 100644
--- a/tests/PictureTest.cpp
+++ b/tests/PictureTest.cpp
@@ -620,77 +620,6 @@ static void test_bad_bitmap() {
 }
 #endif
 
-static sk_sp<SkData> serialized_picture_from_bitmap(const SkBitmap& bitmap) {
-    SkPictureRecorder recorder;
-    SkCanvas* canvas = recorder.beginRecording(SkIntToScalar(bitmap.width()),
-                                               SkIntToScalar(bitmap.height()));
-    canvas->drawBitmap(bitmap, 0, 0);
-    sk_sp<SkPicture> picture(recorder.finishRecordingAsPicture());
-
-    SkDynamicMemoryWStream wStream;
-    SkAutoTUnref<SkPixelSerializer> serializer(
-            SkImageEncoder::CreatePixelSerializer());
-    picture->serialize(&wStream, serializer);
-    return wStream.detachAsData();
-}
-
-static void md5(const SkBitmap& bm, SkMD5::Digest* digest) {
-    SkAutoLockPixels autoLockPixels(bm);
-    SkASSERT(bm.getPixels());
-    SkMD5 md5;
-    size_t rowLen = bm.info().bytesPerPixel() * bm.width();
-    for (int y = 0; y < bm.height(); ++y) {
-        md5.write(bm.getAddr(0, y), rowLen);
-    }
-    md5.finish(*digest);
-}
-
-DEF_TEST(Picture_EncodedData, reporter) {
-    // Create a bitmap that will be encoded.
-    SkBitmap original;
-    make_bm(&original, 100, 100, SK_ColorBLUE, true);
-    SkDynamicMemoryWStream wStream;
-    if (!SkImageEncoder::EncodeStream(&wStream, original, SkImageEncoder::kPNG_Type, 100)) {
-        return;
-    }
-    sk_sp<SkData> data = wStream.detachAsData();
-
-    SkBitmap 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.
-    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.
-    SkMemoryStream pictureStream(std::move(picture1));
-    sk_sp<SkPicture> pictureFromStream(SkPicture::MakeFromStream(&pictureStream));
-    REPORTER_ASSERT(reporter, pictureFromStream.get() != nullptr);
-
-    // Test that using the version of CreateFromStream that just takes a stream also decodes the
-    // bitmap. Drawing this picture should look exactly like the original bitmap.
-    SkMD5::Digest referenceDigest;
-    md5(original, &referenceDigest);
-
-    SkBitmap dst;
-    dst.allocPixels(original.info());
-    dst.eraseColor(SK_ColorRED);
-    SkCanvas canvas(dst);
-
-    pictureStream.rewind();
-    pictureFromStream = SkPicture::MakeFromStream(&pictureStream);
-    canvas.drawPicture(pictureFromStream.get());
-
-    SkMD5::Digest digest2;
-    md5(dst, &digest2);
-    REPORTER_ASSERT(reporter, referenceDigest == digest2);
-}
-
 static void test_clip_bound_opt(skiatest::Reporter* reporter) {
     // Test for crbug.com/229011
     SkRect rect1 = SkRect::MakeXYWH(SkIntToScalar(4), SkIntToScalar(4),