Ensure that SkColorTable->fCount is set properly after decodes

include/core/SkColorTable.h

Index: include/core/SkColorTable.h
diff --git a/include/core/SkColorTable.h b/include/core/SkColorTable.h
index 39553badfc4a70efd56c3551b6a08d422e444cf5..07dfd675b2c37b716ef3524005f4312f3ac607b3 100644
--- a/include/core/SkColorTable.h
+++ b/include/core/SkColorTable.h
@@ -69,6 +69,7 @@ private:
     void init(const SkPMColor* colors, int count);
 
     friend class SkImageGenerator;
+    friend class SkBitmapRegionCodec;
     // Only call if no other thread or cache has seen this table.
     void dangerous_overwriteColors(const SkPMColor newColors[], int count) {
         if (count < 0 || count > fCount) {

[reed1, 2016/05/17 13:48:38]

I wonder if we can add a SkASSERT(this->unique()); so we don't ever try to
change the ctable if there is more than one owner...

[msarett, 2016/05/17 15:29:34]

I tried.  I released the local ref early in SkBitmapRegionCodec, so that this
would work there.  But it breaks somewhere for image generator unfortunately.