Implementation of SkBitmapRegionDecoder using SkAndroidCodec

tools/SkBitmapRegionDecoderInterface.cpp

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkBitmapRegionCanvas.h"
+#include "SkBitmapRegionCodec.h"
 #include "SkBitmapRegionDecoderInterface.h"
 #include "SkBitmapRegionSampler.h"
+#include "SkAndroidCodec.h"
 #include "SkCodec.h"
 #include "SkCodecPriv.h"
 #include "SkImageDecoder.h"
 
 SkBitmapRegionDecoderInterface* SkBitmapRegionDecoderInterface::CreateBitmapRegionDecoder(
         SkStreamRewindable* stream, Strategy strategy) {
     SkAutoTDelete<SkStreamRewindable> streamDeleter(stream);
     switch (strategy) {
         case kOriginal_Strategy: {
             SkImageDecoder* decoder = SkImageDecoder::Factory(stream);
@@ skipping 18 matching lines @@
             switch (codec->getScanlineOrder()) {
                 case SkCodec::kTopDown_SkScanlineOrder:
                 case SkCodec::kNone_SkScanlineOrder:
                     break;
                 default:
                     SkCodecPrintf("Error: Scanline ordering not supported.\n");
                     return nullptr;
             }
             return new SkBitmapRegionCanvas(codec.detach());
         }
+        case kAndroidCodec_Strategy: {
+            SkAutoTDelete<SkAndroidCodec> codec =
+                    SkAndroidCodec::NewFromStream(streamDeleter.detach());

[scroggo, 2015/10/21 20:56:20]

Since this targets BRD, which always uses an SkMemoryStream/SkData, maybe we
should call NewFromData instead?

[msarett, 2015/10/21 22:00:00]

Yeah I think so.

SkDatas are more flexible than SkStreams, and you've already started looking at
implementing some optimizations that take advantage of using an SkData.  Let's
use NewFromData when we can.

+            if (NULL == codec) {
+                SkCodecPrintf("Error: Failed to create codec.\n");
+                return NULL;
+            }
+            return new SkBitmapRegionCodec(codec.detach());
+        }
         default:
             SkASSERT(false);
             return nullptr;
     }
 }