Allowing YUV data to be retrieved from the JPEG Decoder.

Source/platform/image-decoders/ImageDecoder.h

 /*
  * Copyright (C) 2006 Apple Computer, Inc.  All rights reserved.
  * Copyright (C) Research In Motion Limited 2009-2010. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 31 matching lines @@
 #if USE(QCMSLIB)
 #include "qcms.h"
 #if OS(MACOSX)
 #include <ApplicationServices/ApplicationServices.h>
 #include "wtf/RetainPtr.h"
 #endif
 #endif
 
 namespace blink {
 
+// DecodingBuffers can be used to decode color components into provided buffers instead of using an ImageFrame.
+class DecodingBuffers {
+public:
+    DecodingBuffers()
+    {
+        for (int i = 0; i < 3; ++i) {

[Peter Kasting, 2014/07/23 22:17:37]

Nit: For these methods that require more than a line or so to implement, I'd
prefer to place the implementation in the .cpp file unless there's a performance
reason not to.

[sugoi1, 2014/07/24 15:48:30]

Done.

+            m_planes[i] = 0;
+            m_rowBytes[i] = 0;
+        }
+    }
+
+    void set(void* planes[3], size_t rowBytes[3])
+    {
+        for (int i = 0; i < 3; ++i) {
+            m_planes[i] = planes[i];
+            m_rowBytes[i] = rowBytes[i];
+        }
+    }
+
+    void* getPlane(int i) const
+    {
+        return ((i >= 0) && i < 3) ? m_planes[i] : 0;

[Stephen White, 2014/07/23 21:45:53]

Should this be an assert instead?

[sugoi1, 2014/07/24 15:48:30]

Done.

+    }
+
+    size_t getRowBytes(int i) const
+    {
+        return ((i >= 0) && i < 3) ? m_rowBytes[i] : 0;

[Stephen White, 2014/07/23 21:45:53]

Same here.

[sugoi1, 2014/07/24 15:48:30]

Done.

+    }
+private:

[Peter Kasting, 2014/07/23 22:17:37]

Nit: Blank line above this

[sugoi1, 2014/07/24 15:48:30]

Done.

+    void* m_planes[3];
+    size_t m_rowBytes[3];
+};
+
 // ImageDecoder is a base for all format-specific decoders
 // (e.g. JPEGImageDecoder). This base manages the ImageFrame cache.
 //
 class PLATFORM_EXPORT ImageDecoder {
     WTF_MAKE_NONCOPYABLE(ImageDecoder); WTF_MAKE_FAST_ALLOCATED;
 public:
     static const size_t noDecodedImageByteLimit = blink::Platform::noDecodedImageByteLimit;
 
     ImageDecoder(ImageSource::AlphaOption alphaOption, ImageSource::GammaAndColorProfileOption gammaAndColorProfileOption, size_t maxDecodedBytes)
         : m_premultiplyAlpha(alphaOption == ImageSource::AlphaPremultiplied)
@@ skipping 31 matching lines @@
     // decode call out and use it here.
     virtual bool isSizeAvailable()
     {
         return !m_failed && m_sizeAvailable;
     }
 
     virtual IntSize size() const { return m_size; }
 
     // Decoders which downsample images should override this method to
     // return the actual decoded size.
-    virtual IntSize decodedSize() const { return size(); }
+    virtual IntSize decodedSize(int component = 0) const { return size(); }
 
     // This will only differ from size() for ICO (where each frame is a
     // different icon) or other formats where different frames are different
     // sizes. This does NOT differ from size() for GIF or WebP, since
     // decoding GIF or WebP composites any smaller frames against previous
     // frames to create full-size frames.
     virtual IntSize frameSizeAtIndex(size_t) const
     {
         return size();
     }
@@ skipping 131 matching lines @@
     {
         // FIXME: this doesn't work for images with multiple frames.
         if (m_frameBufferCache.isEmpty()) {
             m_frameBufferCache.resize(1);
             m_frameBufferCache[0].setRequiredPreviousFrameIndex(
                 findRequiredPreviousFrame(0, false));
         }
         m_frameBufferCache[0].setMemoryAllocator(allocator);
     }
 
+    virtual void setHardwareDecoding(bool) { }
+    virtual bool hardwareDecoding() const { return false; }
+    virtual bool doHardwareDecoding() { return false; }
+    virtual void setDecodingBuffers(OwnPtr<DecodingBuffers>&) { }
+
 protected:
     // Calculates the most recent frame whose image data may be needed in
     // order to decode frame |frameIndex|, based on frame disposal methods
     // and |frameRectIsOpaque|, where |frameRectIsOpaque| signifies whether
     // the rectangle of frame at |frameIndex| is known to be opaque.
     // If no previous frame's data is required, returns WTF::kNotFound.
     //
     // This function requires that the previous frame's
     // |m_requiredPreviousFrameIndex| member has been set correctly. The
     // easiest way to ensure this is for subclasses to call this method and
@@ skipping 34 matching lines @@
 
     IntSize m_size;
     bool m_sizeAvailable;
     bool m_isAllDataReceived;
     bool m_failed;
 };
 
 } // namespace blink
 
 #endif