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

Unified Diff: Source/core/html/track/WebVTTParser.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
« no previous file with comments | « Source/core/html/track/TextTrackRegionList.cpp ('k') | Source/core/inspector/ContentSearchUtils.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/html/track/WebVTTParser.cpp
diff --git a/Source/core/html/track/WebVTTParser.cpp b/Source/core/html/track/WebVTTParser.cpp
index 27507c46fbb9bcbab58fccc73b18ada0c2387fa7..e249da4cfd635e9f2b6dad3ce2f666bd4341e952 100644
--- a/Source/core/html/track/WebVTTParser.cpp
+++ b/Source/core/html/track/WebVTTParser.cpp
@@ -96,7 +96,7 @@ FloatPoint WebVTTParser::parseFloatPercentageValuePair(const String& value, char
// percentages (x%,y%) implies that at least the first two characters
// are the first percentage value.
size_t delimiterOffset = value.find(delimiter, 2);
- if (delimiterOffset == notFound || delimiterOffset == value.length() - 1) {
+ if (delimiterOffset == kNotFound || delimiterOffset == value.length() - 1) {
isValidSetting = false;
return FloatPoint(0, 0);
}
@@ -280,7 +280,7 @@ WebVTTParser::ParseState WebVTTParser::collectTimingsAndSettings(const String& l
skipWhiteSpace(line, &position);
// 6-9 - If the next three characters are not "-->", abort and return failure.
- if (line.find("-->", position) == notFound)
+ if (line.find("-->", position) == kNotFound)
return BadCue;
position += 3;
if (position >= line.length())
« no previous file with comments | « Source/core/html/track/TextTrackRegionList.cpp ('k') | Source/core/inspector/ContentSearchUtils.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698