commit 7f650bdfd84aa54bf4fb94a2aef89ad931578e8b
author Matt Sarett <msarett@google.com> Sun Oct 30 21:25:34 2016 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Mon Oct 31 19:19:33 2016 +0000
tree bcb03eec2fee20a3c46c7b1bf179aebe3421f364
parent 4d598a35cd7a9b2bbfd724e7503840e2ad8ee778

Tag images as sRGB by default in SkCodec

Unmarked images should be treated as sRGB.

BUG=skia:4895

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4151

Change-Id: I5f8c308d22fd2d069cbfa89c5a5bb19ae6fde8bd
Reviewed-on: https://skia-review.googlesource.com/4151
Reviewed-by: Leon Scroggins <scroggo@google.com>
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Matt Sarett <msarett@google.com>

src/codec/SkGifCodec.cpp

diff --git a/src/codec/SkGifCodec.cpp b/src/codec/SkGifCodec.cpp
index 3c57f2d..fe0a2b6 100644
--- a/src/codec/SkGifCodec.cpp
+++ b/src/codec/SkGifCodec.cpp
@@ -98,9 +98,10 @@
     // zeroes, which is arguably premultiplied.
     const auto alphaType = reader->firstFrameHasAlpha() ? kUnpremul_SkAlphaType
                                                         : kOpaque_SkAlphaType;
-    // FIXME: GIF should default to SkColorSpace::NewNamed(SkColorSpace::kSRGB_Named).
+
     const auto imageInfo = SkImageInfo::Make(reader->screenWidth(), reader->screenHeight(),
-                                             colorType, alphaType);
+                                             colorType, alphaType,
+                                             SkColorSpace::MakeNamed(SkColorSpace::kSRGB_Named));
     return new SkGifCodec(encodedInfo, imageInfo, reader.release());
 }