Animated WebP: Optimize decoding in case of seeking

Source/core/platform/graphics/chromium/ImageFrameGeneratorTest.cpp

 /*
  * Copyright (C) 2012 Google Inc. 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 72 matching lines @@
     virtual void decoderBeingDestroyed()
     {
         ++m_decodersDestroyed;
     }
 
     virtual void frameBufferRequested()
     {
         ++m_frameBufferRequestCount;
     }
 
-    virtual ImageFrame::FrameStatus frameStatus()
+    virtual ImageFrame::Status frameStatus()
     {
-        ImageFrame::FrameStatus currentStatus = m_frameStatus;
+        ImageFrame::Status currentStatus = m_frameStatus;
         m_frameStatus = m_nextFrameStatus;
         return currentStatus;
     }
 
     virtual size_t frameCount() { return 1; }
     virtual int repetitionCount() const { return cAnimationNone; }
     virtual float frameDuration() const { return 0; }
 
 protected:
     PassOwnPtr<ScaledImageFragment> createCompleteImage(const SkISize& size)
     {
         SkBitmap bitmap;
         bitmap.setConfig(SkBitmap::kARGB_8888_Config, size.width(), size.height());
         bitmap.allocPixels();
         return ScaledImageFragment::createComplete(size, 0, bitmap);
     }
 
     void addNewData()
     {
         m_data->append("g", 1);
         m_generator->setData(m_data, false);
     }
 
-    void setFrameStatus(ImageFrame::FrameStatus status)  { m_frameStatus = m_nextFrameStatus = status; }
-    void setNextFrameStatus(ImageFrame::FrameStatus status)  { m_nextFrameStatus = status; }
+    void setFrameStatus(ImageFrame::Status status)  { m_frameStatus = m_nextFrameStatus = status; }
+    void setNextFrameStatus(ImageFrame::Status status)  { m_nextFrameStatus = status; }
 
     RefPtr<SharedBuffer> m_data;
     RefPtr<ImageFrameGenerator> m_generator;
     int m_decodersDestroyed;
     int m_frameBufferRequestCount;
-    ImageFrame::FrameStatus m_frameStatus;
-    ImageFrame::FrameStatus m_nextFrameStatus;
+    ImageFrame::Status m_frameStatus;
+    ImageFrame::Status m_nextFrameStatus;

[Peter Kasting, 2013/08/23 01:14:02]

(This can stay as m_nextFrameStatus)

[urvang (Google), 2013/08/23 02:34:31]

Ack.

 };
 
 PassOwnPtr<ImageDecoder> MockImageDecoderFactory::create()
 {
     OwnPtr<MockImageDecoder> decoder = MockImageDecoder::create(m_test);
     decoder->setSize(fullSize().width(), fullSize().height());
     decoder->setFrameHasAlpha(false);
     return decoder.release();
 }
 
@@ skipping 284 matching lines @@
     ASSERT_TRUE(tempDecoder);
     static_cast<MockImageDecoder*>(tempDecoder)->setFrameHasAlpha(false);
     ImageDecodingStore::instance()->unlockDecoder(m_generator.get(), tempDecoder);
 
     setFrameStatus(ImageFrame::FrameComplete);
     ImageDecodingStore::instance()->unlockCache(m_generator.get(), m_generator->decodeAndScale(fullSize(), 1));
     EXPECT_FALSE(m_generator->hasAlpha(1));
 }
 
 } // namespace