replace NtpTime->Clock with Clock->NtpTime dependency

replace NtpTime->Clock with Clock->NtpTime dependency

BUG=None

Review-Url: https://codereview.webrtc.org/2393723004
Cr-Commit-Position: refs/heads/master@{#16519}
Committed: https://chromium.googlesource.com/external/webrtc/+/3795376ba1a0292d594859ae7826e2ca4d5f1a2c

[danilchap, 2017-02-09 16:02:07]

Patchset #2 (id:20001) has been deleted

[danilchap, 2017-02-09 16:02:13]

Patchset #2 (id:40001) has been deleted

[danilchap, 2017-02-09 16:06:34]

Patchset #2 (id:60001) has been deleted

[danilchap, 2017-02-09 16:12:30]

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

[danilchap, 2017-02-09 16:14:59]

The CQ bit was checked by danilchap@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-09 16:15:09]

Dry run: 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-09 17:29:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-09 17:29:47]

Dry run: This issue passed the CQ dry run.

[perkj_webrtc, 2017-02-09 17:55:22]

lgtm % with a request.

https://codereview.webrtc.org/2393723004/diff/80001/webrtc/system_wrappers/in...
File webrtc/system_wrappers/include/clock.h (right):

https://codereview.webrtc.org/2393723004/diff/80001/webrtc/system_wrappers/in...
webrtc/system_wrappers/include/clock.h:47: NtpTime CurrentNtpTime() const;
please make virtual and add a todo to make pure virtual and remove
CurrentNtp(uint32_t& seconds, uint32_t& fractions).

[danilchap, 2017-02-09 18:17:21]

The CQ bit was checked by danilchap@webrtc.org

[danilchap, 2017-02-09 18:17:21]

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

[commit-bot: I haz the power, 2017-02-09 18:17:31]

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

[danilchap, 2017-02-09 18:17:32]

https://codereview.webrtc.org/2393723004/diff/80001/webrtc/system_wrappers/in...
File webrtc/system_wrappers/include/clock.h (right):

https://codereview.webrtc.org/2393723004/diff/80001/webrtc/system_wrappers/in...
webrtc/system_wrappers/include/clock.h:47: NtpTime CurrentNtpTime() const;
On 2017/02/09 17:55:22, perkj_webrtc wrote:
> please make virtual and add a todo to make pure virtual and remove
> CurrentNtp(uint32_t& seconds, uint32_t& fractions).
> 

Done.

[commit-bot: I haz the power, 2017-02-09 19:15:26]

CQ is committing da patch.

Bot data: {"patchset_id": 120001, "attempt_start_ts": 1486664241055250,
"parent_rev": "1e1c84db106f5940af32e053362efea32cdaf669", "commit_rev":
"3795376ba1a0292d594859ae7826e2ca4d5f1a2c"}

[commit-bot: I haz the power, 2017-02-09 19:15:29]

Description was changed from

==========
replace NtpTime->Clock with Clock->NtpTime dependency

BUG=None
==========

to

==========
replace NtpTime->Clock with Clock->NtpTime dependency

BUG=None

Review-Url: https://codereview.webrtc.org/2393723004
Cr-Commit-Position: refs/heads/master@{#16519}
Committed:
https://chromium.googlesource.com/external/webrtc/+/3795376ba1a0292d594859ae7...
==========

[commit-bot: I haz the power, 2017-02-09 19:15:30]

Committed patchset #4 (id:120001) as
https://chromium.googlesource.com/external/webrtc/+/3795376ba1a0292d594859ae7...