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

Unified Diff: third_party/WebKit/Source/core/html/TimeRanges.cpp

Issue 2384273007: reflow comments in core/html/*.{cpp,h},core/html/imports (Closed)
Patch Set: comments Created 4 years, 2 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: third_party/WebKit/Source/core/html/TimeRanges.cpp
diff --git a/third_party/WebKit/Source/core/html/TimeRanges.cpp b/third_party/WebKit/Source/core/html/TimeRanges.cpp
index f383d755b6c3d7365c54aae32e0888059060a7f8..6864d06acc8595a026bbd38007d2caa602d6fbb9 100644
--- a/third_party/WebKit/Source/core/html/TimeRanges.cpp
+++ b/third_party/WebKit/Source/core/html/TimeRanges.cpp
@@ -132,10 +132,11 @@ void TimeRanges::add(double start, double end) {
// For each present range check if we need to:
// - merge with the added range, in case we are overlapping or contiguous
- // - Need to insert in place, we we are completely, not overlapping and not contiguous
- // in between two ranges.
+ // - Need to insert in place, we we are completely, not overlapping and not
+ // contiguous in between two ranges.
//
- // TODO: Given that we assume that ranges are correctly ordered, this could be optimized.
+ // TODO: Given that we assume that ranges are correctly ordered, this could be
+ // optimized.
for (overlappingArcIndex = 0; overlappingArcIndex < m_ranges.size();
overlappingArcIndex++) {
@@ -150,15 +151,16 @@ void TimeRanges::add(double start, double end) {
// Check the case for which there is no more to do
if (!overlappingArcIndex) {
if (addedRange.isBeforeRange(m_ranges[0])) {
- // First index, and we are completely before that range (and not contiguous, nor overlapping).
- // We just need to be inserted here.
+ // First index, and we are completely before that range (and not
+ // contiguous, nor overlapping). We just need to be inserted here.
break;
}
} else {
if (m_ranges[overlappingArcIndex - 1].isBeforeRange(addedRange) &&
addedRange.isBeforeRange(m_ranges[overlappingArcIndex])) {
- // We are exactly after the current previous range, and before the current range, while
- // not overlapping with none of them. Insert here.
+ // We are exactly after the current previous range, and before the
+ // current range, while not overlapping with none of them. Insert
+ // here.
break;
}
}
« no previous file with comments | « third_party/WebKit/Source/core/html/RadioNodeList.cpp ('k') | third_party/WebKit/Source/core/html/imports/HTMLImport.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698