Lint clean video/ and add lint presubmit check.

Lint clean video/ and add lint presubmit check.

BUG=webrtc:5316
Committed: https://crrev.com/d1590b2571c4cb33416e14c92e4f2dfed42ec3d4
Cr-Commit-Position: refs/heads/master@{#10953}

[mflodman, 2015-12-08 13:28:03]

Peter,
Time for the video folder.

[mflodman, 2015-12-08 13:28:26]

mflodman@webrtc.org changed reviewers:
+ pbos@webrtc.org

[mflodman, 2015-12-08 13:28:26]

And actually adding Peter...

[pbos-webrtc, 2015-12-09 11:21:49]

lgtm with to_string removed

https://codereview.webrtc.org/1507643004/diff/20001/webrtc/video/rampup_tests.cc
File webrtc/video/rampup_tests.cc (right):

https://codereview.webrtc.org/1507643004/diff/20001/webrtc/video/rampup_tests...
webrtc/video/rampup_tests.cc:325: str += std::to_string(num_streams_);
C++11 library feature, not permitted yet.

Consider using an ostringstream which is possibly faster for this function
either way.

https://codereview.webrtc.org/1507643004/diff/20001/webrtc/video/video_qualit...
File webrtc/video/video_quality_test.cc (right):

https://codereview.webrtc.org/1507643004/diff/20001/webrtc/video/video_qualit...
webrtc/video/video_quality_test.cc:1022: title +=  " - Stream #" +
std::to_string(stream_id);
library feature, use an ostringstream

[mflodman, 2015-12-09 13:49:36]

https://codereview.webrtc.org/1507643004/diff/20001/webrtc/video/rampup_tests.cc
File webrtc/video/rampup_tests.cc (right):

https://codereview.webrtc.org/1507643004/diff/20001/webrtc/video/rampup_tests...
webrtc/video/rampup_tests.cc:325: str += std::to_string(num_streams_);
On 2015/12/09 11:21:49, pbos-webrtc wrote:
> C++11 library feature, not permitted yet.
> 
> Consider using an ostringstream which is possibly faster for this function
> either way.

Done.

https://codereview.webrtc.org/1507643004/diff/20001/webrtc/video/video_qualit...
File webrtc/video/video_quality_test.cc (right):

https://codereview.webrtc.org/1507643004/diff/20001/webrtc/video/video_qualit...
webrtc/video/video_quality_test.cc:1022: title +=  " - Stream #" +
std::to_string(stream_id);
On 2015/12/09 11:21:49, pbos-webrtc wrote:
> library feature, use an ostringstream

Done.

[mflodman, 2015-12-09 13:49:40]

The CQ bit was checked by mflodman@webrtc.org

[mflodman, 2015-12-09 13:49:41]

The patchset sent to the CQ was uploaded after l-g-t-m from pbos@webrtc.org
Link to the patchset: https://codereview.webrtc.org/1507643004/#ps40001 (title:
"ostringstream")

[commit-bot: I haz the power, 2015-12-09 13:49:46]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1507643004/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1507643004/40001

[commit-bot: I haz the power, 2015-12-09 15:08:05]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-12-09 15:08:11]

Description was changed from

==========
Lint clean video/ and add lint presubmit check.

BUG=webrtc:5316
==========

to

==========
Lint clean video/ and add lint presubmit check.

BUG=webrtc:5316

Committed: https://crrev.com/d1590b2571c4cb33416e14c92e4f2dfed42ec3d4
Cr-Commit-Position: refs/heads/master@{#10953}
==========

[commit-bot: I haz the power, 2015-12-09 15:08:12]

Patchset 3 (id:??) landed as
https://crrev.com/d1590b2571c4cb33416e14c92e4f2dfed42ec3d4
Cr-Commit-Position: refs/heads/master@{#10953}