Clean up unused or overly complex modes in SkSwizzler

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1582083005

Review URL: https://codereview.chromium.org/1582083005

1 files changed, 8 insertions, 6 deletions

diff --git a/src/codec/SkSwizzler.h b/src/codec/SkSwizzler.h
index e75ab43cfe..edc7dcbe2a 100644
--- a/src/codec/SkSwizzler.h
+++ b/src/codec/SkSwizzler.h
@@ -20,7 +20,7 @@ public:
      */
     enum SrcConfig {
         kUnknown,  // Invalid type.
-        kBit,      // A single bit to distinguish between white and black
+        kBit,      // A single bit to distinguish between white and black.
         kGray,
         kIndex1,
         kIndex2,
@@ -28,12 +28,13 @@ public:
         kIndex,
         kRGB,
         kBGR,
-        kRGBX,
-        kBGRX,
+        kBGRX,     // The alpha channel can be anything, but the image is opaque.
         kRGBA,
         kBGRA,
-        kRGB_565,
         kCMYK,
+        kNoOp8,    // kNoOp modes are used exclusively for sampling, subsetting, and
+        kNoOp16,   // copying.  The pixels themselves do not need to be modified.
+        kNoOp32,
     };
 
     /*
@@ -52,17 +53,18 @@ public:
                 return 4;
             case kGray:
             case kIndex:
+            case kNoOp8:
                 return 8;
-            case kRGB_565:
+            case kNoOp16:
                 return 16;
             case kRGB:
             case kBGR:
                 return 24;
-            case kRGBX:
             case kRGBA:
             case kBGRX:
             case kBGRA:
             case kCMYK:
+            case kNoOp32:
                 return 32;
             default:
                 SkASSERT(false);