commit c5243786217871fad293f99a48ba406ad84412b6
author Brian Salomon <bsalomon@google.com> Thu Apr 02 12:50:34 2020 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Fri Apr 03 15:25:34 2020 +0000
tree b2d333a8a67e68725bc1d7195c9de89911ccb255
parent 9ce08e81f1a5f6a59b877e4915dd4e7c1cbd2488

Don't require color type to make proxy copies.

Bug: skia:10078

Change-Id: I3ce0d97f8ada55403cc3f88bb16659085449ea29
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/281207
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Greg Daniel <egdaniel@google.com>
Reviewed-by: Michael Ludwig <michaelludwig@google.com>

src/image/SkImage_GpuBase.cpp

diff --git a/src/image/SkImage_GpuBase.cpp b/src/image/SkImage_GpuBase.cpp
index f9adbee..5b2a5e2 100644
--- a/src/image/SkImage_GpuBase.cpp
+++ b/src/image/SkImage_GpuBase.cpp
@@ -143,13 +143,10 @@
     const GrSurfaceProxyView* view = this->view(context);
     SkASSERT(view && view->proxy());
 
-    GrColorType grColorType = SkColorTypeToGrColorType(this->colorType());
+    auto copyView = GrSurfaceProxyView::Copy(context, *view, GrMipMapped::kNo, subset,
+                                             SkBackingFit::kExact, view->proxy()->isBudgeted());
 
-    GrSurfaceProxyView copyView = GrSurfaceProxy::Copy(
-            context, view->proxy(), view->origin(), grColorType, GrMipMapped::kNo, subset,
-            SkBackingFit::kExact, view->proxy()->isBudgeted());
-
-    if (!copyView.proxy()) {
+    if (!copyView) {
         return nullptr;
     }