GIF decoding to Index8, unit tests and misusing unit test as benchmark

third_party/WebKit/Source/platform/image-decoders/gif/GIFImageReader.h

Index: third_party/WebKit/Source/platform/image-decoders/gif/GIFImageReader.h
diff --git a/third_party/WebKit/Source/platform/image-decoders/gif/GIFImageReader.h b/third_party/WebKit/Source/platform/image-decoders/gif/GIFImageReader.h
index 18170f5ff537b9f497a02859ac11b30ca506c760..7c10d46abcf87236b57e1202efa691fd725fd440 100644
--- a/third_party/WebKit/Source/platform/image-decoders/gif/GIFImageReader.h
+++ b/third_party/WebKit/Source/platform/image-decoders/gif/GIFImageReader.h
@@ -168,6 +168,8 @@ public:
     }
     void setDefined() { m_isDefined = true; }
     bool isDefined() const { return m_isDefined; }
+    size_t getPosition() const { return m_position; }
+    size_t getTableSize() const { return m_colors; }
 
     // Build RGBA table using the data stream.
     void buildTable(blink::FastSharedBufferReader*);
@@ -292,6 +294,7 @@ public:
         , m_sentSizeToClient(false)
         , m_loopCount(cLoopCountNotSeen)
         , m_parseCompleted(false)
+        , m_backgroundIndex(-1)

[scroggo_chromium, 2016/04/29 19:48:15]

It's a little weird that you initialize this unsigned field to -1.

     {
     }
 
@@ -329,6 +332,8 @@ public:
 
     void clearDecodeState(size_t index) { m_frames[index]->clearDecodeState(); }
 
+    unsigned backgroundIndex() const { return m_backgroundIndex; }
+
 private:
     bool parseData(size_t dataPosition, size_t len, blink::GIFImageDecoder::GIFParseQuery);
     void setRemainingBytes(size_t);
@@ -358,6 +363,7 @@ private:
 
     RefPtr<blink::SharedBuffer> m_data;
     bool m_parseCompleted;
+    unsigned m_backgroundIndex;
 };
 
 #endif