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: Source/core/platform/image-decoders/gif/GIFImageReader.h

Issue 23464095: WTF::notFound looks too much like a local variable. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 7 years, 3 months 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 | Annotate | Revision Log
OLDNEW
1 /* -*- Mode: C; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 2 -*- */ 1 /* -*- Mode: C; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 2 -*- */
2 /* ***** BEGIN LICENSE BLOCK ***** 2 /* ***** BEGIN LICENSE BLOCK *****
3 * Version: MPL 1.1/GPL 2.0/LGPL 2.1 3 * Version: MPL 1.1/GPL 2.0/LGPL 2.1
4 * 4 *
5 * The contents of this file are subject to the Mozilla Public License Version 5 * The contents of this file are subject to the Mozilla Public License Version
6 * 1.1 (the "License"); you may not use this file except in compliance with 6 * 1.1 (the "License"); you may not use this file except in compliance with
7 * the License. You may obtain a copy of the License at 7 * the License. You may obtain a copy of the License at
8 * http://www.mozilla.org/MPL/ 8 * http://www.mozilla.org/MPL/
9 * 9 *
10 * Software distributed under the License is distributed on an "AS IS" basis, 10 * Software distributed under the License is distributed on an "AS IS" basis,
(...skipping 171 matching lines...) Expand 10 before | Expand all | Expand 10 after
182 // Frame output state machine. 182 // Frame output state machine.
183 struct GIFFrameContext { 183 struct GIFFrameContext {
184 WTF_MAKE_FAST_ALLOCATED; 184 WTF_MAKE_FAST_ALLOCATED;
185 public: 185 public:
186 GIFFrameContext(int id) 186 GIFFrameContext(int id)
187 : m_frameId(id) 187 : m_frameId(id)
188 , m_xOffset(0) 188 , m_xOffset(0)
189 , m_yOffset(0) 189 , m_yOffset(0)
190 , m_width(0) 190 , m_width(0)
191 , m_height(0) 191 , m_height(0)
192 , m_transparentPixel(notFound) 192 , m_transparentPixel(kNotFound)
193 , m_disposalMethod(WebCore::ImageFrame::DisposeNotSpecified) 193 , m_disposalMethod(WebCore::ImageFrame::DisposeNotSpecified)
194 , m_dataSize(0) 194 , m_dataSize(0)
195 , m_progressiveDisplay(false) 195 , m_progressiveDisplay(false)
196 , m_interlaced(false) 196 , m_interlaced(false)
197 , m_delayTime(0) 197 , m_delayTime(0)
198 , m_currentLzwBlock(0) 198 , m_currentLzwBlock(0)
199 , m_isComplete(false) 199 , m_isComplete(false)
200 , m_isHeaderDefined(false) 200 , m_isHeaderDefined(false)
201 , m_isDataSizeDefined(false) 201 , m_isDataSizeDefined(false)
202 { 202 {
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
251 void clearDecodeState() { m_lzwContext.clear(); } 251 void clearDecodeState() { m_lzwContext.clear(); }
252 const GIFColorMap& localColorMap() const { return m_localColorMap; } 252 const GIFColorMap& localColorMap() const { return m_localColorMap; }
253 GIFColorMap& localColorMap() { return m_localColorMap; } 253 GIFColorMap& localColorMap() { return m_localColorMap; }
254 254
255 private: 255 private:
256 int m_frameId; 256 int m_frameId;
257 unsigned m_xOffset; 257 unsigned m_xOffset;
258 unsigned m_yOffset; // With respect to "screen" origin. 258 unsigned m_yOffset; // With respect to "screen" origin.
259 unsigned m_width; 259 unsigned m_width;
260 unsigned m_height; 260 unsigned m_height;
261 size_t m_transparentPixel; // Index of transparent pixel. Value is notFound if there is no transparent pixel. 261 size_t m_transparentPixel; // Index of transparent pixel. Value is kNotFound if there is no transparent pixel.
262 WebCore::ImageFrame::DisposalMethod m_disposalMethod; // Restore to backgrou nd, leave in place, etc. 262 WebCore::ImageFrame::DisposalMethod m_disposalMethod; // Restore to backgrou nd, leave in place, etc.
263 int m_dataSize; 263 int m_dataSize;
264 264
265 bool m_progressiveDisplay; // If true, do Haeberli interlace hack. 265 bool m_progressiveDisplay; // If true, do Haeberli interlace hack.
266 bool m_interlaced; // True, if scanlines arrive interlaced order. 266 bool m_interlaced; // True, if scanlines arrive interlaced order.
267 267
268 unsigned m_delayTime; // Display time, in milliseconds, for this image in a multi-image GIF. 268 unsigned m_delayTime; // Display time, in milliseconds, for this image in a multi-image GIF.
269 269
270 OwnPtr<GIFLZWContext> m_lzwContext; 270 OwnPtr<GIFLZWContext> m_lzwContext;
271 Vector<GIFLZWBlock> m_lzwBlocks; // LZW blocks for this frame. 271 Vector<GIFLZWBlock> m_lzwBlocks; // LZW blocks for this frame.
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
358 GIFColorMap m_globalColorMap; 358 GIFColorMap m_globalColorMap;
359 int m_loopCount; // Netscape specific extension block to control the number of animation loops a GIF renders. 359 int m_loopCount; // Netscape specific extension block to control the number of animation loops a GIF renders.
360 360
361 Vector<OwnPtr<GIFFrameContext> > m_frames; 361 Vector<OwnPtr<GIFFrameContext> > m_frames;
362 362
363 RefPtr<WebCore::SharedBuffer> m_data; 363 RefPtr<WebCore::SharedBuffer> m_data;
364 bool m_parseCompleted; 364 bool m_parseCompleted;
365 }; 365 };
366 366
367 #endif 367 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698