Fixed cpu time unittest to be less flaky

Fixed cpu time unittest to be less flaky

BUG=webrtc:7095
Review-Url: https://codereview.webrtc.org/2725553002
Cr-Commit-Position: refs/heads/master@{#16897}
Committed: https://chromium.googlesource.com/external/webrtc/+/78f2d21e74f09301aa70e7055ed9c4f373386098

[ilnik, 2017-02-28 09:32:38]

ilnik@webrtc.org changed reviewers:
+ deadbeef@webrtc.org, kjellander@webrtc.org

[Taylor Brandstetter, 2017-02-28 09:46:39]

lgtm

https://codereview.webrtc.org/2725553002/diff/20001/webrtc/base/cpu_time_unit...
File webrtc/base/cpu_time_unittest.cc (right):

https://codereview.webrtc.org/2725553002/diff/20001/webrtc/base/cpu_time_unit...
webrtc/base/cpu_time_unittest.cc:71: webrtc::SleepMs(kProcessingTimeMillisecs);
nit: Checking the time before and after sleeping could be in its own test, since
it seems independent from the rest of the test.

[ilnik, 2017-02-28 09:53:47]

https://codereview.webrtc.org/2725553002/diff/20001/webrtc/base/cpu_time_unit...
File webrtc/base/cpu_time_unittest.cc (right):

https://codereview.webrtc.org/2725553002/diff/20001/webrtc/base/cpu_time_unit...
webrtc/base/cpu_time_unittest.cc:71: webrtc::SleepMs(kProcessingTimeMillisecs);
On 2017/02/28 09:46:39, Taylor Brandstetter wrote:
> nit: Checking the time before and after sleeping could be in its own test,
since
> it seems independent from the rest of the test.

Done.

[ilnik, 2017-02-28 09:54:45]

The CQ bit was checked by ilnik@webrtc.org

[ilnik, 2017-02-28 09:54:45]

The patchset sent to the CQ was uploaded after l-g-t-m from deadbeef@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2725553002/#ps40001 (title:
"Implemented Deadbeef@ comments")

[commit-bot: I haz the power, 2017-02-28 09:54:47]

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

[commit-bot: I haz the power, 2017-02-28 10:24:10]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1488275685126190,
"parent_rev": "50235b77d06965d75f0adaf874d8cf04618edf63", "commit_rev":
"78f2d21e74f09301aa70e7055ed9c4f373386098"}

[commit-bot: I haz the power, 2017-02-28 10:24:12]

Description was changed from

==========
Fixed cpu time unittest to be less flaky

BUG=webrtc:7095
==========

to

==========
Fixed cpu time unittest to be less flaky

BUG=webrtc:7095

Review-Url: https://codereview.webrtc.org/2725553002
Cr-Commit-Position: refs/heads/master@{#16897}
Committed:
https://chromium.googlesource.com/external/webrtc/+/78f2d21e74f09301aa70e7055...
==========

[commit-bot: I haz the power, 2017-02-28 10:24:13]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/external/webrtc/+/78f2d21e74f09301aa70e7055...