Move gif image decoder to SkCodec

third_party/WebKit/Source/platform/BUILD.gn

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import("//build/config/features.gni")
 import("//build/config/ui.gni")
 import("//testing/libfuzzer/fuzzer_test.gni")
 import("//testing/test.gni")
 import("//third_party/WebKit/public/public_features.gni")
 import("//third_party/WebKit/Source/build/scripts/scripts.gni")
@@ skipping 334 matching lines @@
     "Theme.h",
     "ThemeTypes.h",
     "Timer.cpp",
     "Timer.h",
     "UUID.cpp",
     "UUID.h",
     "UserGestureIndicator.cpp",
     "UserGestureIndicator.h",
     "WaitableEvent.cpp",
     "WaitableEvent.h",
     "WebFrameScheduler.h",

[scroggo_chromium, 2016/12/16 15:03:48]

nit: I know you didn't change this line, but you rebased and made changes in the
same upload, which makes it harder for me to see the changes between revisions.
If you have to rebase, please try to do an upload that just shows the rebase, so
I can focus on the actual changes that you made.

It's even better if you can avoid rebasing when you can, which saves work for
both of us.

[cblume, 2016/12/16 17:43:41]

Makes sense to rebase in a separate patch.
I'm a little fearful to avoid rebasing because it would make landing the final
patch harder.

     "WebIconSizesParser.cpp",
     "WebScheduler.cpp",
     "WebTaskRunner.cpp",
     "WebTaskRunner.h",
     "WebTextInputInfo.cpp",
     "WebThread.cpp",
     "WebThreadSupportingGC.cpp",
     "WebThreadSupportingGC.h",
     "WebURLLoader.cpp",
     "Widget.cpp",
@@ skipping 688 matching lines @@
     "image-decoders/ImageFrame.cpp",
     "image-decoders/ImageFrame.h",
     "image-decoders/SegmentReader.cpp",
     "image-decoders/SegmentReader.h",
     "image-decoders/bmp/BMPImageDecoder.cpp",
     "image-decoders/bmp/BMPImageDecoder.h",
     "image-decoders/bmp/BMPImageReader.cpp",
     "image-decoders/bmp/BMPImageReader.h",
     "image-decoders/gif/GIFImageDecoder.cpp",
     "image-decoders/gif/GIFImageDecoder.h",
-    "image-decoders/gif/GIFImageReader.cpp",
-    "image-decoders/gif/GIFImageReader.h",
     "image-decoders/ico/ICOImageDecoder.cpp",
     "image-decoders/ico/ICOImageDecoder.h",
     "image-decoders/jpeg/JPEGImageDecoder.cpp",
     "image-decoders/jpeg/JPEGImageDecoder.h",
     "image-decoders/png/PNGImageDecoder.cpp",
     "image-decoders/png/PNGImageDecoder.h",
     "image-decoders/png/PNGImageReader.cpp",
     "image-decoders/png/PNGImageReader.h",
     "image-decoders/webp/WEBPImageDecoder.cpp",
     "image-decoders/webp/WEBPImageDecoder.h",
@@ skipping 994 matching lines @@
     "//third_party/WebKit/Source:inside_blink",
   ]
 
   deps = [
     ":test_support",
     "//cc/surfaces:surface_id",
     "//testing/gmock",
     "//testing/gtest",
   ]
 }