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

Unified Diff: Source/core/html/MediaFragmentURIParser.cpp

Issue 606703002: Comply (more) with check-webkit-style in and around HTMLMediaElement (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 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/HTMLVideoElement.cpp ('k') | Source/core/html/shadow/MediaControlElements.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/html/MediaFragmentURIParser.cpp
diff --git a/Source/core/html/MediaFragmentURIParser.cpp b/Source/core/html/MediaFragmentURIParser.cpp
index 0512a1fdd51f1251b093b8ede34a558fabf7008e..8adc732b285510baea751f07bdcbe006359c1afa 100644
--- a/Source/core/html/MediaFragmentURIParser.cpp
+++ b/Source/core/html/MediaFragmentURIParser.cpp
@@ -198,7 +198,7 @@ bool MediaFragmentURIParser::parseNPTFragment(const LChar* timeString, unsigned
{
unsigned offset = 0;
if (length >= nptIdentiferLength && timeString[0] == 'n' && timeString[1] == 'p' && timeString[2] == 't' && timeString[3] == ':')
- offset += nptIdentiferLength;
+ offset += nptIdentiferLength;
if (offset == length)
return false;
@@ -206,9 +206,9 @@ bool MediaFragmentURIParser::parseNPTFragment(const LChar* timeString, unsigned
// http://www.w3.org/2008/WebVideo/Fragments/WD-media-fragments-spec/#naming-time
// If a single number only is given, this corresponds to the begin time except if it is preceded
// by a comma that would in this case indicate the end time.
- if (timeString[offset] == ',')
+ if (timeString[offset] == ',') {
startTime = 0;
- else {
+ } else {
if (!parseNPTTime(timeString, length, offset, startTime))
return false;
}
@@ -235,8 +235,8 @@ bool MediaFragmentURIParser::parseNPTFragment(const LChar* timeString, unsigned
bool MediaFragmentURIParser::parseNPTTime(const LChar* timeString, unsigned length, unsigned& offset, double& time)
{
- enum Mode { minutes, hours };
- Mode mode = minutes;
+ enum Mode { Minutes, Hours };
+ Mode mode = Minutes;
if (offset >= length || !isASCIIDigit(timeString[offset]))
return false;
@@ -280,7 +280,7 @@ bool MediaFragmentURIParser::parseNPTTime(const LChar* timeString, unsigned leng
if (digits1.length() < 2)
return false;
if (digits1.length() > 2)
- mode = hours;
+ mode = Hours;
// Collect the next sequence of 0-9 after ':'
if (offset >= length || timeString[offset++] != ':')
@@ -294,7 +294,7 @@ bool MediaFragmentURIParser::parseNPTTime(const LChar* timeString, unsigned leng
// Detect whether this timestamp includes hours.
int value3;
- if (mode == hours || (offset < length && timeString[offset] == ':')) {
+ if (mode == Hours || (offset < length && timeString[offset] == ':')) {
if (offset >= length || timeString[offset++] != ':')
return false;
if (offset >= length || !isASCIIDigit(timeString[offset]))
« no previous file with comments | « Source/core/html/HTMLVideoElement.cpp ('k') | Source/core/html/shadow/MediaControlElements.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698