Make all the codecs default profiles based on gDefaultProfileIsSRGB.

src/core/SkBitmap.cpp

Index: src/core/SkBitmap.cpp
diff --git a/src/core/SkBitmap.cpp b/src/core/SkBitmap.cpp
index 3bb763b4f43cf65f7117dbcdb79cbe8f444b79bb..f9da29bc275b8219c4de0cf4c6bdf3b2918416de 100644
--- a/src/core/SkBitmap.cpp
+++ b/src/core/SkBitmap.cpp
@@ -747,7 +747,9 @@ bool SkBitmap::extractSubset(SkBitmap* result, const SkIRect& subset) const {
         SkPixelRef* pixelRef = fPixelRef->deepCopy(this->colorType(), this->profileType(), &subset);
         if (pixelRef != nullptr) {
             SkBitmap dst;
-            dst.setInfo(this->info().makeWH(subset.width(), subset.height()));
+            dst.setInfo(SkImageInfo::Make(subset.width(), subset.height(),

[Brian Osman, 2016/04/21 13:33:37]

Sorry, I just went back and looked at this change more carefully. Does this do
anything different? (The rest of the review makes sense - just wasn't sure why
this file was modified like this).

[herb_g, 2016/04/21 15:09:54]

No This is a regression probably from one of the may rebases I did. I will put
it back.

+                                          this->colorType(), this->alphaType(),
+                                          this->profileType()));
             dst.setIsVolatile(this->isVolatile());
             dst.setPixelRef(pixelRef)->unref();
             SkDEBUGCODE(dst.validate());
@@ -762,7 +764,9 @@ bool SkBitmap::extractSubset(SkBitmap* result, const SkIRect& subset) const {
     SkASSERT(static_cast<unsigned>(r.fTop) < static_cast<unsigned>(this->height()));
 
     SkBitmap dst;
-    dst.setInfo(this->info().makeWH(r.width(), r.height()), this->rowBytes());
+    dst.setInfo(SkImageInfo::Make(r.width(), r.height(),
+                                  this->colorType(), this->alphaType(), this->profileType()),
+                this->rowBytes());
     dst.setIsVolatile(this->isVolatile());
 
     if (fPixelRef) {