reflow comments in platform/{mediastream,scroll}

third_party/WebKit/Source/platform/scroll/ScrollbarTheme.cpp

Index: third_party/WebKit/Source/platform/scroll/ScrollbarTheme.cpp
diff --git a/third_party/WebKit/Source/platform/scroll/ScrollbarTheme.cpp b/third_party/WebKit/Source/platform/scroll/ScrollbarTheme.cpp
index d74bbd33829ac2c6cb252c60bfd93bb7b4021c10..8f1403dde5f4ece101232e74c0ef2f461c69b5ee 100644
--- a/third_party/WebKit/Source/platform/scroll/ScrollbarTheme.cpp
+++ b/third_party/WebKit/Source/platform/scroll/ScrollbarTheme.cpp
@@ -237,7 +237,8 @@ int ScrollbarTheme::thumbPosition(const ScrollbarThemeClient& scrollbar,
                                   float scrollPosition) {
   if (scrollbar.enabled()) {
     float size = scrollbar.totalSize() - scrollbar.visibleSize();
-    // Avoid doing a floating point divide by zero and return 1 when usedTotalSize == visibleSize.
+    // Avoid doing a floating point divide by zero and return 1 when
+    // usedTotalSize == visibleSize.
     if (!size)
       return 0;
     float pos = std::max(0.0f, scrollPosition) *
@@ -261,8 +262,8 @@ int ScrollbarTheme::thumbLength(const ScrollbarThemeClient& scrollbar) {
   int length = round(proportion * trackLen);
   length = std::max(length, minimumThumbLength(scrollbar));
   if (length > trackLen)
-    length =
-        0;  // Once the thumb is below the track length, it just goes away (to make more room for the track).

[pdr., 2016/10/03 21:30:50]

This looks like it was an edge-case mistake in the big reformat. Do you think it
would be worth grepping for more instances of this? I am not sure how.

[Nico, 2016/10/03 21:34:42]

I think "ReflowComments: false" triggers this. When that's set, clang-format
can't break the comment and the line end, but column limit is 80, so to minimize
the damage, clang-format makes the line with the comment on it as short as
possible. The comment reflow cleanup (that this CL is part of) will fix this
issue everywhere it happened.

+    length = 0;  // Once the thumb is below the track length, it just goes away
+                 // (to make more room for the track).
   return length;
 }
 
@@ -310,8 +311,8 @@ void ScrollbarTheme::splitTrack(const ScrollbarThemeClient& scrollbar,
                                 IntRect& beforeThumbRect,
                                 IntRect& thumbRect,
                                 IntRect& afterThumbRect) {
-  // This function won't even get called unless we're big enough to have some combination of these three rects where at least
-  // one of them is non-empty.
+  // This function won't even get called unless we're big enough to have some
+  // combination of these three rects where at least one of them is non-empty.
   IntRect trackRect =
       constrainTrackRectToTrackPieces(scrollbar, unconstrainedTrackRect);
   int thumbPos = thumbPosition(scrollbar);