Modify YUV codecs to match Skia's API change

third_party/WebKit/Source/platform/image-decoders/ImageDecoder.h

Index: third_party/WebKit/Source/platform/image-decoders/ImageDecoder.h
diff --git a/third_party/WebKit/Source/platform/image-decoders/ImageDecoder.h b/third_party/WebKit/Source/platform/image-decoders/ImageDecoder.h
index 2ebd7797caf99844abc34018d95e42f885f8b453..66b155eca53aaeae7d45bda7a96ec27dd9fd9b03 100644
--- a/third_party/WebKit/Source/platform/image-decoders/ImageDecoder.h
+++ b/third_party/WebKit/Source/platform/image-decoders/ImageDecoder.h
@@ -55,7 +55,7 @@ class PLATFORM_EXPORT ImagePlanes final {
     WTF_MAKE_NONCOPYABLE(ImagePlanes);
 public:
     ImagePlanes();
-    ImagePlanes(void* planes[3], size_t rowBytes[3]);
+    ImagePlanes(void* planes[3], const size_t rowBytes[3]);
 
     void* plane(int);
     size_t rowBytes(int) const;
@@ -71,8 +71,6 @@ private:
 class PLATFORM_EXPORT ImageDecoder {
     WTF_MAKE_NONCOPYABLE(ImageDecoder); USING_FAST_MALLOC(ImageDecoder);
 public:
-    enum SizeType { ActualSize, SizeForMemoryAllocation };
-
     static const size_t noDecodedImageByteLimit = Platform::noDecodedImageByteLimit;
 
     enum AlphaOption {
@@ -138,7 +136,8 @@ public:
 
     // Decoders which support YUV decoding can override this to
     // give potentially different sizes per component.
-    virtual IntSize decodedYUVSize(int component, SizeType) const { return decodedSize(); }
+    virtual IntSize decodedYUVSize(int component) const { return decodedSize(); }

[scroggo_chromium, 2016/02/24 13:09:43]

This predates your CL, but it seems like this base version should never be
called - it should only be called if we're using JPEGImageDecoder, which
overrides it. I would be tempted to implement this as

{
  ASSERT(false);
  return IntSize();
}

[msarett, 2016/02/24 16:36:11]

Yes I agree.  Done.

+    virtual size_t decodedYUVWidthBytes(int component) const { return decodedSize().width(); }

[scroggo_chromium, 2016/02/24 13:09:43]

Maybe describe what this means?

Also, similar to above, I think this should never be called. So I would
implement this as

{
  ASSERT(false);
  return 0;
}

[msarett, 2016/02/24 16:36:11]

Done.

 
     // This will only differ from size() for ICO (where each frame is a
     // different icon) or other formats where different frames are different