commit 974c821502042c0ab233affb0f38bf2b49f5ecfe
author Brian Salomon <bsalomon@google.com> Mon Apr 05 16:24:00 2021 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Mon Apr 05 21:18:46 2021 +0000
tree 82f1d30196fd35c239e367adc7ab473b9b7eed5d
parent 0a12b85f160f662fa4539ae87f39441fe7020604

Reland "Make RGB_888x pixel operations work."

This is a reland of 8ce24b1d11f25a871fa23285cfd29373aa5d1d6a

Original change's description:
> Make RGB_888x pixel operations work.
>
> Also remove vulkan-specific conversions in GrVkGpu and rely
> on higher level SurfaceContext to convert correctly.
>
> Bug: skia:8862
> Change-Id: Ib8b0541c8c5831148b7129c4bbed3f925660116e
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/392378
> Reviewed-by: Robert Phillips <robertphillips@google.com>
> Commit-Queue: Brian Salomon <bsalomon@google.com>

Bug: skia:8862
Change-Id: I5d085e47398a4a2947228ee67f867552fa93123a
Cq-Include-Trybots: luci.skia.skia.primary:Test-Debian10-Clang-NUC5PPYH-GPU-IntelHD405-x86_64-Debug-All-Vulkan
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/392842
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>

src/gpu/GrSurfaceContext.cpp

diff --git a/src/gpu/GrSurfaceContext.cpp b/src/gpu/GrSurfaceContext.cpp
index 31b6534..f98b377 100644
--- a/src/gpu/GrSurfaceContext.cpp
+++ b/src/gpu/GrSurfaceContext.cpp
@@ -615,13 +615,13 @@
     pt.fY = flip ? dstSurface->height() - pt.fY - src[0].height() : pt.fY;
 
     if (!dContext->priv().drawingManager()->newWritePixelsTask(
-            sk_ref_sp(dstProxy),
-            SkIRect::MakePtSize(pt, src[0].dimensions()),
-            this->colorInfo().colorType(),
-            allowedColorType,
-            srcLevels.begin(),
-            numLevels,
-            prepForSampling)) {
+                sk_ref_sp(dstProxy),
+                SkIRect::MakePtSize(pt, src[0].dimensions()),
+                allowedColorType,
+                this->colorInfo().colorType(),
+                srcLevels.begin(),
+                numLevels,
+                prepForSampling)) {
         return false;
     }
     if (numLevels > 1) {