Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(78)

Side by Side Diff: third_party/WebKit/Source/platform/image-decoders/gif/GIFImageDecoder.h

Issue 1460523002: GIF decoding to Index8, unit tests and misusing unit test as benchmark (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: cleanup. tableChanged was wrong - do proper check. Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2006 Apple Computer, Inc. All rights reserved. 2 * Copyright (C) 2006 Apple Computer, Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 1. Redistributions of source code must retain the above copyright 7 * 1. Redistributions of source code must retain the above copyright
8 * notice, this list of conditions and the following disclaimer. 8 * notice, this list of conditions and the following disclaimer.
9 * 2. Redistributions in binary form must reproduce the above copyright 9 * 2. Redistributions in binary form must reproduce the above copyright
10 * notice, this list of conditions and the following disclaimer in the 10 * notice, this list of conditions and the following disclaimer in the
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
51 int repetitionCount() const override; 51 int repetitionCount() const override;
52 bool frameIsCompleteAtIndex(size_t) const override; 52 bool frameIsCompleteAtIndex(size_t) const override;
53 float frameDurationAtIndex(size_t) const override; 53 float frameDurationAtIndex(size_t) const override;
54 size_t clearCacheExceptFrame(size_t) override; 54 size_t clearCacheExceptFrame(size_t) override;
55 // CAUTION: setFailed() deletes |m_reader|. Be careful to avoid 55 // CAUTION: setFailed() deletes |m_reader|. Be careful to avoid
56 // accessing deleted memory, especially when calling this from inside 56 // accessing deleted memory, especially when calling this from inside
57 // GIFImageReader! 57 // GIFImageReader!
58 bool setFailed() override; 58 bool setFailed() override;
59 59
60 // Callbacks from the GIF reader. 60 // Callbacks from the GIF reader.
61 bool haveDecodedRow(size_t frameIndex, GIFRow::const_iterator rowBegin, size _t width, size_t rowNumber, unsigned repeatCount, bool writeTransparentPixels); 61 bool (GIFImageDecoder::*phaveDecodedRow)(size_t frameIndex, GIFRow::const_it erator rowBegin, size_t width, size_t rowNumber, unsigned repeatCount, bool writ eTransparentPixels);
62 inline bool haveDecodedRow(size_t frameIndex, GIFRow::const_iterator rowBegi n, size_t width, size_t rowNumber, unsigned repeatCount, bool writeTransparentPi xels)
63 {
64 return (this->*phaveDecodedRow)(frameIndex, rowBegin, width, rowNumber, repeatCount, writeTransparentPixels);
65 }
62 bool frameComplete(size_t frameIndex); 66 bool frameComplete(size_t frameIndex);
63 67
68 // Callback implementations for different output.
69 bool haveDecodedRowN32(size_t frameIndex, GIFRow::const_iterator rowBegin, s ize_t width, size_t rowNumber, unsigned repeatCount, bool writeTransparentPixels );
70 bool haveDecodedRowIndex8(size_t frameIndex, GIFRow::const_iterator rowBegin , size_t width, size_t rowNumber, unsigned repeatCount, bool writeTransparentPix els);
71
72 // Callback when reader detects state requiring RGBA decoding.
73 void setForceN32Decoding(bool);
74
64 // For testing. 75 // For testing.
65 bool parseCompleted() const; 76 bool parseCompleted() const;
66 77
78 // For supporting both N32 and Index8 output.
79 bool canDecodeTo(size_t index, ImageFrame::ColorType outputType) override;
80
67 private: 81 private:
68 // ImageDecoder: 82 // ImageDecoder:
69 void clearFrameBuffer(size_t frameIndex) override; 83 void clearFrameBuffer(size_t frameIndex) override;
70 virtual void decodeSize() { parse(GIFSizeQuery); } 84 virtual void decodeSize() { parse(GIFSizeQuery); }
71 size_t decodeFrameCount() override; 85 size_t decodeFrameCount() override;
72 void initializeNewFrame(size_t) override; 86 void initializeNewFrame(size_t) override;
73 void decode(size_t) override; 87 void decode(size_t) override;
88 void decodeTo(size_t index, ImageFrame::ColorType outputColor) override;
89
74 90
75 // Parses as much as is needed to answer the query, ignoring bitmap 91 // Parses as much as is needed to answer the query, ignoring bitmap
76 // data. If parsing fails, sets the "decode failure" flag. 92 // data. If parsing fails, sets the "decode failure" flag.
77 void parse(GIFParseQuery); 93 void parse(GIFParseQuery);
78 94
79 // Called to initialize the frame buffer with the given index, based on 95 // Called to initialize the frame buffer with the given index, based on
80 // the previous frame's disposal method. Returns true on success. On 96 // the previous frame's disposal method. Returns true on success. On
81 // failure, this will mark the image as failed. 97 // failure, this will mark the image as failed.
82 bool initFrameBuffer(size_t frameIndex); 98 bool initFrameBuffer(size_t frameIndex);
99 bool initFrameBufferIndex8(size_t frameIndex);
83 100
84 bool m_currentBufferSawAlpha; 101 bool m_currentBufferSawAlpha;
85 mutable int m_repetitionCount; 102 mutable int m_repetitionCount;
86 OwnPtr<GIFImageReader> m_reader; 103 OwnPtr<GIFImageReader> m_reader;
104
105 // Holds information about which decoding output was in client request.
106 ImageFrame::ColorType m_colorMode;
107
108 // In Index8 decoding mode, some of the frames need to get decoded to N32.
109 // This flag is reset when situation allows going back to Index8 decoding in initFrameBuffer.
110 bool m_forceN32Decoding;
87 }; 111 };
88 112
89 } // namespace blink 113 } // namespace blink
90 114
91 #endif 115 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698