Remove old lazy image decoding heuristic logic

This led to removing a lot of transfer function behavior code. There is
more that could be done, and we need to add in decoding to dst color
space, but this CL is almost entirely mechanical.

Change-Id: I91b2169f95aadcfaacdd2b9821bb1a01ce53f9a6
Reviewed-on: https://skia-review.googlesource.com/140349
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
diff --git a/src/codec/SkCodecImageGenerator.cpp b/src/codec/SkCodecImageGenerator.cpp
index 286320c..6727dcf 100644
--- a/src/codec/SkCodecImageGenerator.cpp
+++ b/src/codec/SkCodecImageGenerator.cpp
@@ -40,12 +40,12 @@
 }
 
 bool SkCodecImageGenerator::onGetPixels(const SkImageInfo& requestInfo, void* requestPixels,
-                                        size_t requestRowBytes, const Options& opts) {
+                                        size_t requestRowBytes, const Options&) {
     SkPixmap dst(requestInfo, requestPixels, requestRowBytes);
 
-    auto decode = [this, &opts](const SkPixmap& pm) {
+    auto decode = [this](const SkPixmap& pm) {
         SkCodec::Options codecOpts;
-        codecOpts.fPremulBehavior = opts.fBehavior;
+        codecOpts.fPremulBehavior = SkTransferFunctionBehavior::kIgnore;
         SkCodec::Result result = fCodec->getPixels(pm, &codecOpts);
         switch (result) {
             case SkCodec::kSuccess: