diff options
author | robertphillips <robertphillips@google.com> | 2016-04-20 11:48:36 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-04-20 11:48:36 -0700 |
commit | 3e302275b324172c845627cbd00cee8a06571baf (patch) | |
tree | 51b7ab49a3e044145d8818051ad019432a820654 /tests/SpecialImageTest.cpp | |
parent | 9ca06c4b00bfb9bb1a7f352efd264185e5a95fbc (diff) |
Begin removing deprecated (and now, unused) ImageFilter code paths
This CL focuses on removing the Proxy objects but takes some other stuff with it.
BUG=skia:4965
TBR=reed@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1896383003
Review URL: https://codereview.chromium.org/1896383003
Diffstat (limited to 'tests/SpecialImageTest.cpp')
-rw-r--r-- | tests/SpecialImageTest.cpp | 25 |
1 files changed, 9 insertions, 16 deletions
diff --git a/tests/SpecialImageTest.cpp b/tests/SpecialImageTest.cpp index a8a79668a7..18d023f9cd 100644 --- a/tests/SpecialImageTest.cpp +++ b/tests/SpecialImageTest.cpp @@ -14,7 +14,6 @@ #include "SkSpecialSurface.h" #include "SkSurface.h" #include "Test.h" -#include "TestingSpecialImageAccess.h" #if SK_SUPPORT_GPU #include "GrContext.h" @@ -53,7 +52,7 @@ static SkBitmap create_bm() { static void test_image(const sk_sp<SkSpecialImage>& img, skiatest::Reporter* reporter, GrContext* context, bool peekTextureSucceeds, int offset, int size) { - const SkIRect subset = TestingSpecialImageAccess::Subset(img.get()); + const SkIRect subset = img->subset(); REPORTER_ASSERT(reporter, offset == subset.left()); REPORTER_ASSERT(reporter, offset == subset.top()); REPORTER_ASSERT(reporter, kSmallerSize == subset.width()); @@ -140,14 +139,13 @@ DEF_TEST(SpecialImage_Raster, reporter) { SkBitmap bm = create_bm(); sk_sp<SkSpecialImage> fullSImage(SkSpecialImage::MakeFromRaster( - nullptr, SkIRect::MakeWH(kFullSize, kFullSize), bm)); const SkIRect& subset = SkIRect::MakeXYWH(kPad, kPad, kSmallerSize, kSmallerSize); { - sk_sp<SkSpecialImage> subSImg1(SkSpecialImage::MakeFromRaster(nullptr, subset, bm)); + sk_sp<SkSpecialImage> subSImg1(SkSpecialImage::MakeFromRaster(subset, bm)); test_image(subSImg1, reporter, nullptr, false, kPad, kFullSize); } @@ -163,15 +161,13 @@ DEF_TEST(SpecialImage_Image, reporter) { sk_sp<SkImage> fullImage(SkImage::MakeFromBitmap(bm)); sk_sp<SkSpecialImage> fullSImage(SkSpecialImage::MakeFromImage( - nullptr, SkIRect::MakeWH(kFullSize, kFullSize), fullImage)); const SkIRect& subset = SkIRect::MakeXYWH(kPad, kPad, kSmallerSize, kSmallerSize); { - sk_sp<SkSpecialImage> subSImg1(SkSpecialImage::MakeFromImage(nullptr, subset, - fullImage)); + sk_sp<SkSpecialImage> subSImg1(SkSpecialImage::MakeFromImage(subset, fullImage)); test_image(subSImg1, reporter, nullptr, false, kPad, kFullSize); } @@ -193,7 +189,7 @@ DEF_TEST(SpecialImage_Pixmap, reporter) { pixmap.erase(SK_ColorRED, subset); { - sk_sp<SkSpecialImage> img(SkSpecialImage::MakeFromPixmap(nullptr, subset, pixmap, + sk_sp<SkSpecialImage> img(SkSpecialImage::MakeFromPixmap(subset, pixmap, nullptr, nullptr)); test_image(img, reporter, nullptr, false, kPad, kFullSize); } @@ -222,20 +218,19 @@ DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SpecialImage_MakeTexture, reporter, ctxInfo) { // raster sk_sp<SkSpecialImage> rasterImage(SkSpecialImage::MakeFromRaster( - nullptr, SkIRect::MakeWH(kFullSize, kFullSize), bm)); { - sk_sp<SkSpecialImage> fromRaster(rasterImage->makeTextureImage(nullptr, context)); + sk_sp<SkSpecialImage> fromRaster(rasterImage->makeTextureImage(context)); test_texture_backed(reporter, rasterImage, fromRaster); } { sk_sp<SkSpecialImage> subRasterImage(rasterImage->makeSubset(subset)); - sk_sp<SkSpecialImage> fromSubRaster(subRasterImage->makeTextureImage(nullptr, context)); + sk_sp<SkSpecialImage> fromSubRaster(subRasterImage->makeTextureImage(context)); test_texture_backed(reporter, subRasterImage, fromSubRaster); } } @@ -257,21 +252,20 @@ DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SpecialImage_MakeTexture, reporter, ctxInfo) } sk_sp<SkSpecialImage> gpuImage(SkSpecialImage::MakeFromGpu( - nullptr, SkIRect::MakeWH(kFullSize, kFullSize), kNeedNewImageUniqueID_SpecialImage, texture)); { - sk_sp<SkSpecialImage> fromGPU(gpuImage->makeTextureImage(nullptr, context)); + sk_sp<SkSpecialImage> fromGPU(gpuImage->makeTextureImage(context)); test_texture_backed(reporter, gpuImage, fromGPU); } { sk_sp<SkSpecialImage> subGPUImage(gpuImage->makeSubset(subset)); - sk_sp<SkSpecialImage> fromSubGPU(subGPUImage->makeTextureImage(nullptr, context)); + sk_sp<SkSpecialImage> fromSubGPU(subGPUImage->makeTextureImage(context)); test_texture_backed(reporter, subGPUImage, fromSubGPU); } } @@ -295,7 +289,6 @@ DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SpecialImage_Gpu, reporter, ctxInfo) { } sk_sp<SkSpecialImage> fullSImg(SkSpecialImage::MakeFromGpu( - nullptr, SkIRect::MakeWH(kFullSize, kFullSize), kNeedNewImageUniqueID_SpecialImage, texture)); @@ -304,7 +297,7 @@ DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SpecialImage_Gpu, reporter, ctxInfo) { { sk_sp<SkSpecialImage> subSImg1(SkSpecialImage::MakeFromGpu( - nullptr, subset, + subset, kNeedNewImageUniqueID_SpecialImage, texture)); test_image(subSImg1, reporter, context, true, kPad, kFullSize); |