commit c3a927fc455aa3080d9b0dd6fd1832a5a37774f3
author Greg Daniel <egdaniel@google.com> Wed Oct 16 12:03:50 2019 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Thu Oct 17 17:10:17 2019 +0000
tree 20eb3ed062f101df776face72ead061672de5993
parent 883c7e9f59a783fd143f6015d11e72020ed77e78

Remove GrPixelConfigIsCompressed call.

Bug: skia:6718
Change-Id: Ia6363d74f016b076fa5e19a57d223f7c47b723c0
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/248999
Reviewed-by: Jim Van Verth <jvanverth@google.com>
Commit-Queue: Greg Daniel <egdaniel@google.com>

src/gpu/GrResourceProvider.cpp

diff --git a/src/gpu/GrResourceProvider.cpp b/src/gpu/GrResourceProvider.cpp
index 9ba9505..92b71ae 100644
--- a/src/gpu/GrResourceProvider.cpp
+++ b/src/gpu/GrResourceProvider.cpp
@@ -173,14 +173,15 @@
         return nullptr;
     }
 
-    // Compressed textures are read-only so they don't support re-use for scratch.
+    // Currently we don't recycle compressed textures as scratch. Additionally all compressed
+    // textures should be created through the createCompressedTexture function.
+    SkASSERT(!this->caps()->isFormatCompressed(format));
+
     // TODO: Support GrMipMapped::kYes in scratch texture lookup here.
-    if (!GrPixelConfigIsCompressed(desc.fConfig)) {
-        sk_sp<GrTexture> tex = this->getExactScratch(
-                desc, format, renderable, renderTargetSampleCnt, budgeted, mipMapped, isProtected);
-        if (tex) {
-            return tex;
-        }
+    sk_sp<GrTexture> tex = this->getExactScratch(
+            desc, format, renderable, renderTargetSampleCnt, budgeted, mipMapped, isProtected);
+    if (tex) {
+        return tex;
     }
 
     return fGpu->createTexture(desc, format, renderable, renderTargetSampleCnt, mipMapped, budgeted,
@@ -224,10 +225,9 @@
         return nullptr;
     }
 
-    // Currently we don't recycle compressed textures as scratch.
-    if (GrPixelConfigIsCompressed(desc.fConfig)) {
-        return nullptr;
-    }
+    // Currently we don't recycle compressed textures as scratch. Additionally all compressed
+    // textures should be created through the createCompressedTexture function.
+    SkASSERT(!this->caps()->isFormatCompressed(format));
 
     if (!fCaps->validateSurfaceParams({desc.fWidth, desc.fHeight}, format, desc.fConfig, renderable,
                                       renderTargetSampleCnt, GrMipMapped::kNo)) {
@@ -256,7 +256,7 @@
                                                        GrProtected isProtected) {
     ASSERT_SINGLE_OWNER
     SkASSERT(!this->isAbandoned());
-    SkASSERT(!GrPixelConfigIsCompressed(desc.fConfig));
+    SkASSERT(!this->caps()->isFormatCompressed(format));
     SkASSERT(fCaps->validateSurfaceParams({desc.fWidth, desc.fHeight}, format, desc.fConfig,
                                           renderable, renderTargetSampleCnt, GrMipMapped::kNo));