corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1624232 - 1624232: Replaced 'else if' with 'if' after 'return' in dom/html/TextTrackManager.cpp r=emilio

Browse Source 
Differential Revision: https://phabricator.services.mozilla.com/D68114
This commit is contained in:
Sydney
2020-04-09 20:05:41 +00:00
parent 7d94decfa7
commit 3edd1ac7d6
1 changed files with 10 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
dom/html/TextTrackManager.cpp
View File

@@ -496,7 +496,8 @@ class CompareSimpleTextTrackEvents {
// TimeMarchesOn step 13.1.
if (aOne->mTime < aTwo->mTime) {
return true;
} else if (aOne->mTime > aTwo->mTime) {
}
if (aOne->mTime > aTwo->mTime) {
return false;
}
@@ -514,7 +515,8 @@ class CompareSimpleTextTrackEvents {
auto index2 = textTracks.IndexOf(t2);
if (index1 < index2) {
return true;
} else if (index1 > index2) {
}
if (index1 > index2) {
return false;
}
}
@@ -526,12 +528,14 @@ class CompareSimpleTextTrackEvents {
if (c1 != c2) {
if (c1->StartTime() < c2->StartTime()) {
return true;
} else if (c1->StartTime() > c2->StartTime()) {
}
if (c1->StartTime() > c2->StartTime()) {
return false;
}
if (c1->EndTime() < c2->EndTime()) {
return true;
} else if (c1->EndTime() > c2->EndTime()) {
}
if (c1->EndTime() > c2->EndTime()) {
return false;
}
@@ -542,7 +546,8 @@ class CompareSimpleTextTrackEvents {
auto index2 = cues.IndexOf(c2);
if (index1 < index2) {
return true;
} else if (index1 > index2) {
}
if (index1 > index2) {
return false;
}
}