Provides multiple implementations of Android's SkBitmapRegionDecoder

dm/DMSrcSink.h

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef DMSrcSink_DEFINED
 #define DMSrcSink_DEFINED
 
 #include "DMGpuSupport.h"
 #include "SkBBHFactory.h"
 #include "SkBBoxHierarchy.h"
 #include "SkBitmap.h"
+#include "SkBitmapRegionDecoder.h"
 #include "SkCanvas.h"
 #include "SkCodec.h"
 #include "SkData.h"
 #include "SkGPipe.h"
 #include "SkPicture.h"
 #include "gm.h"
 
 namespace DM {
 
 // This is just convenience.  It lets you use either return "foo" or return SkStringPrintf(...).
@@ skipping 68 matching lines @@
 
     Error draw(SkCanvas*) const override;
     SkISize size() const override;
     Name name() const override;
     void modifyGrContextOptions(GrContextOptions* options) const override;
 
 private:
     skiagm::GMRegistry::Factory fFactory;
 };
 
+class BRDSrc : public Src {

[scroggo, 2015/08/13 16:53:07]

Maybe add a description for what this is for?

[msarett, 2015/08/13 18:10:23]

Done.

+public:
+    enum Mode {
+        kNormal_Mode,

[scroggo, 2015/08/13 16:53:07]

What do these modes mean? Isn't BRD always used for doing subsets?

[msarett, 2015/08/13 18:10:23]

Maybe we don't need both of these tests or these aren't the only interesting
tests?

I think kFullImage_Mode, kDivisor_Mode are more accurate.

[scroggo, 2015/08/28 13:49:43]

The FullImage mode seems weird. Does it show us something unique, interesting,
or analogous to how clients use it? Divisor definitely seems interesting. Or
maybe these should track the uses in your various benchmarks?
Agreed.

+        kSubset_Mode,
+    };
+
+    BRDSrc(Path, SkBitmapRegionDecoder::Strategy, Mode, SkColorType, uint32_t);
+
+    Error draw(SkCanvas*) const override;
+    SkISize size() const override;
+    Name name() const override;
+    bool veto(SinkFlags) const override;
+private:
+    Path                            fPath;
+    SkBitmapRegionDecoder::Strategy fStrategy;
+    Mode                            fMode;
+    SkColorType                     fColorType;
+    uint32_t                        fSampleSize;
+};
+
 class CodecSrc : public Src {
 public:
     enum Mode {
         kNormal_Mode,
         kScanline_Mode,
         kScanline_Subset_Mode,
         kStripe_Mode, // Tests the skipping of scanlines
         kSubset_Mode, // For codecs that support subsets directly.
     };
     enum DstColorType {
@@ skipping 196 matching lines @@
 
 class ViaTwice : public Via {
 public:
     explicit ViaTwice(Sink* sink) : Via(sink) {}
     Error draw(const Src&, SkBitmap*, SkWStream*, SkString*) const override;
 };
 
 }  // namespace DM
 
 #endif//DMSrcSink_DEFINED