| Index: tests/SurfaceTest.cpp
|
| diff --git a/tests/SurfaceTest.cpp b/tests/SurfaceTest.cpp
|
| index e4d0519c78aab132c73c74bf5629e20a4e2c6c42..5528f1a17025e79912e5daa4b14c9e0a84ae1783 100644
|
| --- a/tests/SurfaceTest.cpp
|
| +++ b/tests/SurfaceTest.cpp
|
| @@ -74,7 +74,7 @@ DEF_TEST(SurfaceEmpty, reporter) {
|
|
|
| }
|
| #if SK_SUPPORT_GPU
|
| -DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SurfaceEmpty_Gpu, reporter, ctxInfo) {
|
| +DEF_GPUTEST_FOR_GL_RENDERING_CONTEXTS(SurfaceEmpty_Gpu, reporter, ctxInfo) {
|
| const SkImageInfo info = SkImageInfo::Make(0, 0, kN32_SkColorType, kPremul_SkAlphaType);
|
| REPORTER_ASSERT(reporter, nullptr ==
|
| SkSurface::MakeRenderTarget(ctxInfo.fGrContext, SkBudgeted::kNo, info, 0,
|
| @@ -83,7 +83,7 @@ DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SurfaceEmpty_Gpu, reporter, ctxInfo) {
|
| #endif
|
|
|
| #if SK_SUPPORT_GPU
|
| -DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SurfaceWrappedTexture, reporter, ctxInfo) {
|
| +DEF_GPUTEST_FOR_GL_RENDERING_CONTEXTS(SurfaceWrappedTexture, reporter, ctxInfo) {
|
| GrGpu* gpu = ctxInfo.fGrContext->getGpu();
|
| if (!gpu) {
|
| return;
|
| @@ -186,7 +186,7 @@ DEF_TEST(SurfaceCanvasPeek, reporter) {
|
| }
|
| }
|
| #if SK_SUPPORT_GPU
|
| -DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SurfaceCanvasPeek_Gpu, reporter, ctxInfo) {
|
| +DEF_GPUTEST_FOR_GL_RENDERING_CONTEXTS(SurfaceCanvasPeek_Gpu, reporter, ctxInfo) {
|
| for (auto& surface_func : { &create_gpu_surface, &create_gpu_scratch_surface }) {
|
| SkImageInfo requestInfo;
|
| auto surface(surface_func(ctxInfo.fGrContext, kPremul_SkAlphaType, &requestInfo));
|
| @@ -219,7 +219,7 @@ DEF_TEST(SurfaceAccessPixels, reporter) {
|
| }
|
| }
|
| #if SK_SUPPORT_GPU
|
| -DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SurfaceAccessPixels_Gpu, reporter, ctxInfo) {
|
| +DEF_GPUTEST_FOR_GL_RENDERING_CONTEXTS(SurfaceAccessPixels_Gpu, reporter, ctxInfo) {
|
| for (auto& surface_func : { &create_gpu_surface, &create_gpu_scratch_surface }) {
|
| auto surface(surface_func(ctxInfo.fGrContext, kPremul_SkAlphaType, nullptr));
|
| test_access_pixels(reporter, surface);
|
| @@ -248,7 +248,7 @@ DEF_TEST(SurfaceSnapshotAlphaType, reporter) {
|
| }
|
| }
|
| #if SK_SUPPORT_GPU
|
| -DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SurfaceSnapshotAlphaType_Gpu, reporter, ctxInfo) {
|
| +DEF_GPUTEST_FOR_GL_RENDERING_CONTEXTS(SurfaceSnapshotAlphaType_Gpu, reporter, ctxInfo) {
|
| for (auto& surface_func : { &create_gpu_surface, &create_gpu_scratch_surface }) {
|
| for (auto& isOpaque : { true, false }) {
|
| SkAlphaType alphaType = isOpaque ? kOpaque_SkAlphaType : kPremul_SkAlphaType;
|
| @@ -300,7 +300,7 @@ DEF_TEST(SurfaceBackendHandleAccessCopyOnWrite, reporter) {
|
| }
|
| }
|
| #if SK_SUPPORT_GPU
|
| -DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SurfaceBackendHandleAccessCopyOnWrite_Gpu, reporter, ctxInfo) {
|
| +DEF_GPUTEST_FOR_GL_RENDERING_CONTEXTS(SurfaceBackendHandleAccessCopyOnWrite_Gpu, reporter, ctxInfo) {
|
| const SkSurface::BackendHandleAccess accessModes[] = {
|
| SkSurface::kFlushRead_BackendHandleAccess,
|
| SkSurface::kFlushWrite_BackendHandleAccess,
|
| @@ -398,7 +398,7 @@ DEF_TEST(UniqueImageSnapshot, reporter) {
|
| }
|
|
|
| #if SK_SUPPORT_GPU
|
| -DEF_GPUTEST_FOR_RENDERING_CONTEXTS(UniqueImageSnapshot_Gpu, reporter, ctxInfo) {
|
| +DEF_GPUTEST_FOR_GL_RENDERING_CONTEXTS(UniqueImageSnapshot_Gpu, reporter, ctxInfo) {
|
| GrContext* context = ctxInfo.fGrContext;
|
| for (auto& surface_func : { &create_gpu_surface, &create_gpu_scratch_surface }) {
|
| auto surface(surface_func(context, kOpaque_SkAlphaType, nullptr));
|
| @@ -501,7 +501,7 @@ static void test_backend_handle_unique_id(
|
| REPORTER_ASSERT(reporter, image2->uniqueID() != image3->uniqueID());
|
| }
|
| // No CPU test.
|
| -DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SurfaceBackendHandleAccessIDs_Gpu, reporter, ctxInfo) {
|
| +DEF_GPUTEST_FOR_GL_RENDERING_CONTEXTS(SurfaceBackendHandleAccessIDs_Gpu, reporter, ctxInfo) {
|
| for (auto& surface_func : { &create_gpu_surface, &create_gpu_scratch_surface }) {
|
| for (auto& test_func : { &test_backend_handle_unique_id, &test_backend_handle_gen_id }) {
|
| for (auto& handle_access_func :
|
| @@ -593,7 +593,7 @@ DEF_TEST(SurfaceCopyOnWrite, reporter) {
|
| test_copy_on_write(reporter, create_surface().get());
|
| }
|
| #if SK_SUPPORT_GPU
|
| -DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SurfaceCopyOnWrite_Gpu, reporter, ctxInfo) {
|
| +DEF_GPUTEST_FOR_GL_RENDERING_CONTEXTS(SurfaceCopyOnWrite_Gpu, reporter, ctxInfo) {
|
| for (auto& surface_func : { &create_gpu_surface, &create_gpu_scratch_surface }) {
|
| auto surface(surface_func(ctxInfo.fGrContext, kPremul_SkAlphaType, nullptr));
|
| test_copy_on_write(reporter, surface.get());
|
| @@ -615,7 +615,7 @@ DEF_TEST(SurfaceWriteableAfterSnapshotRelease, reporter) {
|
| test_writable_after_snapshot_release(reporter, create_surface().get());
|
| }
|
| #if SK_SUPPORT_GPU
|
| -DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SurfaceWriteableAfterSnapshotRelease_Gpu, reporter, ctxInfo) {
|
| +DEF_GPUTEST_FOR_GL_RENDERING_CONTEXTS(SurfaceWriteableAfterSnapshotRelease_Gpu, reporter, ctxInfo) {
|
| for (auto& surface_func : { &create_gpu_surface, &create_gpu_scratch_surface }) {
|
| auto surface(surface_func(ctxInfo.fGrContext, kPremul_SkAlphaType, nullptr));
|
| test_writable_after_snapshot_release(reporter, surface.get());
|
| @@ -656,7 +656,7 @@ static void test_crbug263329(skiatest::Reporter* reporter,
|
| REPORTER_ASSERT(reporter, as_IB(image3)->peekTexture() != as_IB(image1)->peekTexture());
|
| REPORTER_ASSERT(reporter, as_IB(image2)->peekTexture() != as_IB(image1)->peekTexture());
|
| }
|
| -DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SurfaceCRBug263329_Gpu, reporter, ctxInfo) {
|
| +DEF_GPUTEST_FOR_GL_RENDERING_CONTEXTS(SurfaceCRBug263329_Gpu, reporter, ctxInfo) {
|
| for (auto& surface_func : { &create_gpu_surface, &create_gpu_scratch_surface }) {
|
| auto surface1(surface_func(ctxInfo.fGrContext, kPremul_SkAlphaType, nullptr));
|
| auto surface2(surface_func(ctxInfo.fGrContext, kPremul_SkAlphaType, nullptr));
|
| @@ -673,7 +673,7 @@ DEF_TEST(SurfaceGetTexture, reporter) {
|
| REPORTER_ASSERT(reporter, as_IB(image)->peekTexture() == nullptr);
|
| }
|
| #if SK_SUPPORT_GPU
|
| -DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SurfacepeekTexture_Gpu, reporter, ctxInfo) {
|
| +DEF_GPUTEST_FOR_GL_RENDERING_CONTEXTS(SurfacepeekTexture_Gpu, reporter, ctxInfo) {
|
| for (auto& surface_func : { &create_gpu_surface, &create_gpu_scratch_surface }) {
|
| auto surface(surface_func(ctxInfo.fGrContext, kPremul_SkAlphaType, nullptr));
|
| sk_sp<SkImage> image(surface->makeImageSnapshot());
|
| @@ -705,7 +705,7 @@ static SkBudgeted is_budgeted(const sk_sp<SkImage> image) {
|
| return is_budgeted(image.get());
|
| }
|
|
|
| -DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SurfaceBudget, reporter, ctxInfo) {
|
| +DEF_GPUTEST_FOR_GL_RENDERING_CONTEXTS(SurfaceBudget, reporter, ctxInfo) {
|
| SkImageInfo info = SkImageInfo::MakeN32Premul(8,8);
|
| for (auto sbudgeted : { SkBudgeted::kNo, SkBudgeted::kYes }) {
|
| for (auto ibudgeted : { SkBudgeted::kNo, SkBudgeted::kYes }) {
|
| @@ -767,7 +767,7 @@ DEF_TEST(SurfaceNoCanvas, reporter) {
|
| }
|
| }
|
| #if SK_SUPPORT_GPU
|
| -DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SurfaceNoCanvas_Gpu, reporter, ctxInfo) {
|
| +DEF_GPUTEST_FOR_GL_RENDERING_CONTEXTS(SurfaceNoCanvas_Gpu, reporter, ctxInfo) {
|
| SkSurface::ContentChangeMode modes[] =
|
| { SkSurface::kDiscard_ContentChangeMode, SkSurface::kRetain_ContentChangeMode};
|
| for (auto& surface_func : { &create_gpu_surface, &create_gpu_scratch_surface }) {
|
| @@ -858,7 +858,7 @@ void test_surface_clear(skiatest::Reporter* reporter, sk_sp<SkSurface> surface,
|
| }
|
| }
|
|
|
| -DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SurfaceClear_Gpu, reporter, ctxInfo) {
|
| +DEF_GPUTEST_FOR_GL_RENDERING_CONTEXTS(SurfaceClear_Gpu, reporter, ctxInfo) {
|
| GrContext* context = ctxInfo.fGrContext;
|
| std::function<GrSurface*(SkSurface*)> grSurfaceGetters[] = {
|
| [] (SkSurface* s){ return s->getCanvas()->internal_private_accessTopLayerRenderTarget(); },
|
|
|