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

Unified Diff: Source/core/html/track/TextTrackRegion.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/TextTrackList.cpp ('k') | Source/core/html/track/TextTrackRegionList.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/html/track/TextTrackRegion.cpp
diff --git a/Source/core/html/track/TextTrackRegion.cpp b/Source/core/html/track/TextTrackRegion.cpp
index d5c30b36716cfa1a303c7bdfe6663617deb5b7b3..ca3f3447a4cc817d943dac4cd30e8858d335937b 100644
--- a/Source/core/html/track/TextTrackRegion.cpp
+++ b/Source/core/html/track/TextTrackRegion.cpp
@@ -271,7 +271,7 @@ void TextTrackRegion::parseSettingValue(RegionSetting setting, const String& val
switch (setting) {
case Id:
- if (value.find("-->") == notFound)
+ if (value.find("-->") == kNotFound)
m_id = value;
break;
case Width:
@@ -322,7 +322,7 @@ void TextTrackRegion::parseSetting(const String& input, unsigned* position)
String setting = WebVTTParser::collectWord(input, position);
size_t equalOffset = setting.find('=', 1);
- if (equalOffset == notFound || !equalOffset || equalOffset == setting.length() - 1)
+ if (equalOffset == kNotFound || !equalOffset || equalOffset == setting.length() - 1)
return;
RegionSetting name = getSettingFromString(setting.substring(0, equalOffset));
« no previous file with comments | « Source/core/html/track/TextTrackList.cpp ('k') | Source/core/html/track/TextTrackRegionList.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698