Collapse most spammy output of webrtc_perf_tests with PrintResultList

Collapse most stdout spammy output of webrtc_perf_tests with PrintResultList

BUG=webrtc:5032
NOTRY=true

Committed: https://crrev.com/46b89b9a3f3f92acea250ab9e34a479040101145
Cr-Commit-Position: refs/heads/master@{#13037}

[danilchap, 2016-06-02 16:44:45]

Description was changed from

==========
Collapse most spammy output of webrtc_perf_tests with PrintResultList

BUG=webrtc:5032
NOTRY=true
==========

to

==========
Collapse most stdout spammy output of webrtc_perf_tests with PrintResultList

BUG=webrtc:5032
NOTRY=true
==========

[danilchap, 2016-06-02 16:46:19]

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

[pbos-webrtc, 2016-06-03 15:10:03]

lgtm with code moved

https://codereview.webrtc.org/2030073002/diff/1/webrtc/call/call_perf_tests.cc
File webrtc/call/call_perf_tests.cc (right):

https://codereview.webrtc.org/2030073002/diff/1/webrtc/call/call_perf_tests.c...
webrtc/call/call_perf_tests.cc:54: // Converts list of values into
comma-separated string for PrintResultList.
Pref that you move this to webrtc/test/testsupport/perf_test.{h,cc} where
PrintResultList is.

https://codereview.webrtc.org/2030073002/diff/1/webrtc/call/call_perf_tests.c...
webrtc/call/call_perf_tests.cc:65: return ss.str();
preferring break here, I think some compilers might also have an issue with not
having the return statement last.

[danilchap, 2016-06-03 15:32:13]

https://codereview.webrtc.org/2030073002/diff/1/webrtc/call/call_perf_tests.cc
File webrtc/call/call_perf_tests.cc (right):

https://codereview.webrtc.org/2030073002/diff/1/webrtc/call/call_perf_tests.c...
webrtc/call/call_perf_tests.cc:54: // Converts list of values into
comma-separated string for PrintResultList.
On 2016/06/03 15:10:03, pbos-webrtc wrote:
> Pref that you move this to webrtc/test/testsupport/perf_test.{h,cc} where
> PrintResultList is.

Moved to perf_test.h since it is a template function.

https://codereview.webrtc.org/2030073002/diff/1/webrtc/call/call_perf_tests.c...
webrtc/call/call_perf_tests.cc:65: return ss.str();
On 2016/06/03 15:10:03, pbos-webrtc wrote:
> preferring break here, I think some compilers might also have an issue with
not
> having the return statement last.

Done.

[pbos-webrtc, 2016-06-03 15:33:53]

lgtm

[danilchap, 2016-06-03 16:25:53]

The CQ bit was checked by danilchap@webrtc.org

[commit-bot: I haz the power, 2016-06-03 16:26:02]

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

[commit-bot: I haz the power, 2016-06-03 16:27:41]

Description was changed from

==========
Collapse most stdout spammy output of webrtc_perf_tests with PrintResultList

BUG=webrtc:5032
NOTRY=true
==========

to

==========
Collapse most stdout spammy output of webrtc_perf_tests with PrintResultList

BUG=webrtc:5032
NOTRY=true
==========

[commit-bot: I haz the power, 2016-06-03 16:27:42]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-03 16:27:52]

Description was changed from

==========
Collapse most stdout spammy output of webrtc_perf_tests with PrintResultList

BUG=webrtc:5032
NOTRY=true
==========

to

==========
Collapse most stdout spammy output of webrtc_perf_tests with PrintResultList

BUG=webrtc:5032
NOTRY=true

Committed: https://crrev.com/46b89b9a3f3f92acea250ab9e34a479040101145
Cr-Commit-Position: refs/heads/master@{#13037}
==========

[commit-bot: I haz the power, 2016-06-03 16:27:53]

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