reflow comments in platform/{mediastream,scroll}

reflow comments in platform/{mediastream,scroll}

BUG=563793
Committed: https://crrev.com/119f976164031b951476a52270160b926d804490
Cr-Commit-Position: refs/heads/master@{#422591}

[Nico, 2016-10-03 20:53:29]

The CQ bit was checked by thakis@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-03 20:54:03]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Nico, 2016-10-03 20:55:09]

thakis@chromium.org changed reviewers:
+ pdr@chromium.org

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

LGTM

https://codereview.chromium.org/2387053003/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/scroll/ScrollbarTheme.cpp (left):

https://codereview.chromium.org/2387053003/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/scroll/ScrollbarTheme.cpp:265: 0;  // Once
the thumb is below the track length, it just goes away (to make more room for
the track).
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:43]

thanks!

https://codereview.chromium.org/2387053003/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/scroll/ScrollbarTheme.cpp (left):

https://codereview.chromium.org/2387053003/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/scroll/ScrollbarTheme.cpp:265: 0;  // Once
the thumb is below the track length, it just goes away (to make more room for
the track).
On 2016/10/03 21:30:50, pdr. wrote:
> 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.

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.

[Nico, 2016-10-03 21:34:45]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2016-10-03 21:35:40]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-03 23:20:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-03 23:24:35]

Description was changed from

==========
reflow comments in platform/{mediastream,scroll}

BUG=563793
==========

to

==========
reflow comments in platform/{mediastream,scroll}

BUG=563793

Committed: https://crrev.com/119f976164031b951476a52270160b926d804490
Cr-Commit-Position: refs/heads/master@{#422591}
==========

[commit-bot: I haz the power, 2016-10-03 23:24:36]

Patchset 2 (id:??) landed as
https://crrev.com/119f976164031b951476a52270160b926d804490
Cr-Commit-Position: refs/heads/master@{#422591}