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

Side by Side Diff: third_party/WebKit/Source/platform/image-decoders/png/PNGImageDecoder.h

Issue 2523943002: Explicitly specify target color space to ImageDecoder at creation (Closed)
Patch Set: Rebase Created 4 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 21 matching lines...) Expand all
32 namespace blink { 32 namespace blink {
33 33
34 class PNGImageReader; 34 class PNGImageReader;
35 35
36 class PLATFORM_EXPORT PNGImageDecoder final : public ImageDecoder { 36 class PLATFORM_EXPORT PNGImageDecoder final : public ImageDecoder {
37 WTF_MAKE_NONCOPYABLE(PNGImageDecoder); 37 WTF_MAKE_NONCOPYABLE(PNGImageDecoder);
38 38
39 public: 39 public:
40 PNGImageDecoder(AlphaOption, 40 PNGImageDecoder(AlphaOption,
41 ColorSpaceOption, 41 ColorSpaceOption,
42 sk_sp<SkColorSpace>,
42 size_t maxDecodedBytes, 43 size_t maxDecodedBytes,
43 size_t offset = 0); 44 size_t offset = 0);
44 ~PNGImageDecoder() override; 45 ~PNGImageDecoder() override;
45 46
46 // ImageDecoder: 47 // ImageDecoder:
47 String filenameExtension() const override { return "png"; } 48 String filenameExtension() const override { return "png"; }
48 49
49 // Callbacks from libpng 50 // Callbacks from libpng
50 void headerAvailable(); 51 void headerAvailable();
51 void rowAvailable(unsigned char* row, unsigned rowIndex, int); 52 void rowAvailable(unsigned char* row, unsigned rowIndex, int);
52 void complete(); 53 void complete();
53 54
54 private: 55 private:
55 // ImageDecoder: 56 // ImageDecoder:
56 void decodeSize() override { decode(true); } 57 void decodeSize() override { decode(true); }
57 void decode(size_t) override { decode(false); } 58 void decode(size_t) override { decode(false); }
58 59
59 // Decodes the image. If |onlySize| is true, stops decoding after 60 // Decodes the image. If |onlySize| is true, stops decoding after
60 // calculating the image size. If decoding fails but there is no more 61 // calculating the image size. If decoding fails but there is no more
61 // data coming, sets the "decode failure" flag. 62 // data coming, sets the "decode failure" flag.
62 void decode(bool onlySize); 63 void decode(bool onlySize);
63 64
64 std::unique_ptr<PNGImageReader> m_reader; 65 std::unique_ptr<PNGImageReader> m_reader;
65 const unsigned m_offset; 66 const unsigned m_offset;
66 }; 67 };
67 68
68 } // namespace blink 69 } // namespace blink
69 70
70 #endif 71 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698