commit 6887dcf153bf74c5d55bd7d093658946772b11c3
author Robert Phillips <robertphillips@google.com> Thu Sep 18 10:24:51 2025 -0400
committer SkCQ <skcq-be@skia-corp.google.com.iam.gserviceaccount.com> Thu Sep 18 09:34:36 2025 -0700
tree 29c1efa4e479db117f2309c9c385943ae012b9df
parent 57f6a74914105c89f41c2df686c4e1dd2dc5c981

Patch wacky_yuv_formats_imggen GM

This GM got out of whack in: https://skia-review.googlesource.com/c/skia/+/1052336.
All the raster configs would only render the topmost row and then skip all the others.

Change-Id: Ibffc617c27d0b81a66cc5696aa9a9951538d3904
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1057936
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Thomas Smith <thomsmit@google.com>

gm/wacky_yuv_formats.cpp

diff --git a/gm/wacky_yuv_formats.cpp b/gm/wacky_yuv_formats.cpp
index 78c2ca2..4d3b5eb 100644
--- a/gm/wacky_yuv_formats.cpp
+++ b/gm/wacky_yuv_formats.cpp
@@ -889,13 +889,11 @@
 #if defined(SK_GRAPHITE)
                     if (recorder) {
                         fImages[opaque][cs][format] = lazyYUV->refImage(recorder, fImageType);
-                    }
+                    } else
 #endif
-#if defined(SK_GANESH)
-                    if (dContext) {
+                    {
                         fImages[opaque][cs][format] = lazyYUV->refImage(dContext, fImageType);
                     }
-#endif
                 }
                 origin = (origin + 1) % 8;
             }
@@ -1153,14 +1151,12 @@
                 if (recorder) {
                     fImages[opaque][i++] = lazyYUV->refImage(
                             recorder, sk_gpu_test::LazyYUVImage::Type::kFromTextures);
-                }
+                } else
 #endif
-#if defined(SK_GANESH)
-                if (context) {
+                {
                     fImages[opaque][i++] = lazyYUV->refImage(
                             context, sk_gpu_test::LazyYUVImage::Type::kFromTextures);
                 }
-#endif
             }
         }