WIP: Implement APNG

third_party/WebKit/Source/platform/image-decoders/png/PNGImageDecoder.h

 /*
  * Copyright (C) 2006 Apple Computer, 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
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 21 matching lines @@
 namespace blink {
 
 class PNGImageReader;
 
 class PLATFORM_EXPORT PNGImageDecoder final : public ImageDecoder {
     WTF_MAKE_NONCOPYABLE(PNGImageDecoder);
 public:
     PNGImageDecoder(AlphaOption, GammaAndColorProfileOption, size_t maxDecodedBytes, size_t offset = 0);
     ~PNGImageDecoder() override;
 
+    enum class PNGParseQuery { PNGSizeQuery, PNGMetaDataQuery };
+
     // ImageDecoder:
     String filenameExtension() const override { return "png"; }
+    int repetitionCount() const override;
+    
 
     // Callbacks from libpng
+    void frameHeaderAvailable() {};
     void headerAvailable();
     void rowAvailable(unsigned char* row, unsigned rowIndex, int);
     void complete();
 
+    // Additional methods used for APNG
+    void setRepetitionCount(size_t);
+    // This is called by PNGImageReader when it encounters the IEND chunk
+    // while parsing PNGMetaDataQuery. Ifm_metaDataDecoded is true, the decoder

[scroggo_chromium, 2016/10/11 20:13:09]

nit: Missing a space between "If" and "m_metaDataDecoded"

+    // does not need to query for meta data again.
+    void setMetaDataDecoded() { m_metaDataDecoded = true; };
+    
 private:
     // ImageDecoder:
-    void decodeSize() override { decode(true); }
-    void decode(size_t) override { decode(false); }
+    void decodeSize() override { parse(PNGParseQuery::PNGSizeQuery); }
+    // @TODO(joostouwerling) this needs some more work.

[scroggo_chromium, 2016/10/11 20:13:09]

What more work is needed?

[joostouwerling, 2016/10/12 20:49:46]

Reminder for me to implement decode(size_t). Will omit it from patches in the
future.

+    void decode(size_t) override; 
+    size_t decodeFrameCount() override;
+    void initializeNewFrame(size_t) override;
 
-    // Decodes the image.  If |onlySize| is true, stops decoding after
-    // calculating the image size.  If decoding fails but there is no more
-    // data coming, sets the "decode failure" flag.
-    void decode(bool onlySize);
+    void parse(PNGParseQuery);
 
     std::unique_ptr<PNGImageReader> m_reader;
     const unsigned m_offset;
+    bool m_metaDataDecoded;
+    size_t m_frameCount;
+    int m_repetitionCount;
 };
 
 } // namespace blink
 
 #endif