Make image decoders faster by refactoring hot loops that fills the lines of ImageFrame.

Source/core/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 157 matching lines @@
         // Use fix point multiplier instead of integer division or floating point math.
         // This multipler produces exactly the same result for all values in range 0 - 255.
         static const unsigned fixPointShift = 24;
         static const unsigned fixPointMult = static_cast<unsigned>(1.0 / 255.0 * (1 << fixPointShift)) + 1;
         // Multiplies unsigned value by fixpoint value and converts back to unsigned.
         static unsigned fixPointUnsignedMultiply(unsigned fixed, unsigned v)
         {
             return  (fixed * v) >> fixPointShift;
         }
 
+        inline void setRGB(PixelData* dest, unsigned r, unsigned g, unsigned b)
+        {
+            checkAddress(dest);
+            *dest = SkPackARGB32NoCheck(255, r, g, b);
+        }
+
         inline void setRGBA(PixelData* dest, unsigned r, unsigned g, unsigned b, unsigned a)
         {
-            if (m_premultiplyAlpha && a < 255) {
-                if (!a) {
-                    *dest = 0;
-                    return;
-                }
+            if (m_premultiplyAlpha)
+                setRGBAPremultiply(dest, a, r, g, b);
+            else
+                setRGBANoPremultiply(dest, a, r, g, b);
+        }
 
+        inline void setRGBAPremultiply(PixelData* dest, unsigned r, unsigned g, unsigned b, unsigned a)
+        {
+            checkAddress(dest);
+            ASSERT(m_premultiplyAlpha);
+
+            if (!a) {
+                *dest = 0;
+                return;
+            }
+            if (a < 255) {
                 unsigned alphaMult = a * fixPointMult;
                 r = fixPointUnsignedMultiply(r, alphaMult);
                 g = fixPointUnsignedMultiply(g, alphaMult);
                 b = fixPointUnsignedMultiply(b, alphaMult);
             }
+
             // Call the "NoCheck" version since we may deliberately pass non-premultiplied
             // values, and we don't want an assert.
             *dest = SkPackARGB32NoCheck(a, r, g, b);
         }
 
+        inline void setRGBANoPremultiply(PixelData* dest, unsigned r, unsigned g, unsigned b, unsigned a)
+        {
+            checkAddress(dest);
+            ASSERT(!m_premultiplyAlpha);
+
+            *dest = SkPackARGB32NoCheck(a, r, g, b);
+        }
+
+        inline void setInvertedCMYK(PixelData* dest, unsigned c, unsigned m, unsigned y, unsigned k)
+        {
+            checkAddress(dest);
+
+            unsigned kMult = k * fixPointMult;
+            unsigned r = fixPointUnsignedMultiply(c, kMult);
+            unsigned g = fixPointUnsignedMultiply(m, kMult);
+            unsigned b = fixPointUnsignedMultiply(y, kMult);
+
+            *dest = SkPackARGB32NoCheck(255, r, g, b);
+        }
+
     private:
         int width() const
         {
             return m_bitmap->bitmap().width();
         }
 
         int height() const
         {
             return m_bitmap->bitmap().height();
         }
 
+        void checkAddress(PixelData* address)

[Peter Kasting, 2013/06/12 18:47:07]

I would just omit this.  The previous code had a setRGBA() function that wrote
to a pointer without checking and I think it's OK for the new functions to do so
too.  This just adds verbosity and debug-mode slowness, callers should be doing
something legal here.

+        {
+            ASSERT(getAddr(0, 0) <= address && address <= getAddr(0, 0) + width() * height() * m_hasAlpha ? 4 : 3);
+        }
+
         RefPtr<NativeImageSkia> m_bitmap;
         SkBitmap::Allocator* m_allocator;
         bool m_hasAlpha;
         IntRect m_originalFrameRect; // This will always just be the entire
                                      // buffer except for GIF frames whose
                                      // original rect was smaller than the
                                      // overall image size.
         FrameStatus m_status;
         unsigned m_duration;
         FrameDisposalMethod m_disposalMethod;
@@ skipping 225 matching lines @@
 
         IntSize m_size;
         bool m_sizeAvailable;
         bool m_isAllDataReceived;
         bool m_failed;
     };
 
 } // namespace WebCore
 
 #endif