Remove an obsolete TODO in fx_skia_device.cpp.

In Upsample(), all the possible bpp values have been handled. Thus the
default case in the switch statement does not need the TODO and does not
need to set anything since it is not reachable.

Change-Id: Ib43b88b9e31715b5fd8463aeb3348086f3aaccf1
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/103234
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Nigi <nigi@chromium.org>

core/fxge/skia/fx_skia_device.cpp

diff --git a/core/fxge/skia/fx_skia_device.cpp b/core/fxge/skia/fx_skia_device.cpp
index 69082ea..b287d7a 100644
--- a/core/fxge/skia/fx_skia_device.cpp
+++ b/core/fxge/skia/fx_skia_device.cpp
@@ -687,8 +687,7 @@
       pSource->DebugVerifyBitmapIsPreMultiplied();
       break;
     default:
-      NOTREACHED();  // TODO(bug_11) ensure that all cases are covered
-      colorType = SkColorType::kUnknown_SkColorType;
+      NOTREACHED();
   }
   SkImageInfo imageInfo =
       SkImageInfo::Make(width, height, colorType, alphaType);