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

Unified Diff: Source/core/platform/image-decoders/ImageDecoder.cpp

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 side-by-side diff with in-line comments
Download patch
Index: Source/core/platform/image-decoders/ImageDecoder.cpp
diff --git a/Source/core/platform/image-decoders/ImageDecoder.cpp b/Source/core/platform/image-decoders/ImageDecoder.cpp
index 657ee4cceacf26150c204528ec364343a0a8a4b5..cdb19002b9af347861c4143273eb72f06df4e788 100644
--- a/Source/core/platform/image-decoders/ImageDecoder.cpp
+++ b/Source/core/platform/image-decoders/ImageDecoder.cpp
@@ -157,13 +157,13 @@ size_t ImageDecoder::findRequiredPreviousFrame(size_t frameIndex, bool frameRect
ASSERT(frameIndex <= m_frameBufferCache.size());
if (!frameIndex) {
// The first frame doesn't rely on any previous data.
- return notFound;
+ return kNotFound;
}
const ImageFrame* currBuffer = &m_frameBufferCache[frameIndex];
if ((frameRectIsOpaque || currBuffer->alphaBlendSource() == ImageFrame::BlendAtopBgcolor)
&& currBuffer->originalFrameRect().contains(IntRect(IntPoint(), size())))
- return notFound;
+ return kNotFound;
// The starting state for this frame depends on the previous frame's
// disposal method.
@@ -190,10 +190,10 @@ size_t ImageDecoder::findRequiredPreviousFrame(size_t frameIndex, bool frameRect
// this frame is a blank frame, so it can again be decoded alone.
// Otherwise, the previous frame contributes to this frame.
return (prevBuffer->originalFrameRect().contains(IntRect(IntPoint(), size()))
- || (prevBuffer->requiredPreviousFrameIndex() == notFound)) ? notFound : prevFrame;
+ || (prevBuffer->requiredPreviousFrameIndex() == kNotFound)) ? kNotFound : prevFrame;
default:
ASSERT_NOT_REACHED();
- return notFound;
+ return kNotFound;
}
}
« no previous file with comments | « Source/core/platform/image-decoders/ImageDecoder.h ('k') | Source/core/platform/image-decoders/ImageDecoderTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698