Replace Clock::CurrentNtp with Clock::CurrentNtpTime

Replace Clock::CurrentNtp with Clock::CurrentNtpTime

CurrentNtp return time by taking two output parameters by reference
(also breaks style guide)
CurrentNtpTime treat ntp time as single entity and returns it using NtpTime structure.
(making interface clearer)

BUG=None

Review-Url: https://codereview.webrtc.org/2733823002
Cr-Commit-Position: refs/heads/master@{#17088}
Committed: https://chromium.googlesource.com/external/webrtc/+/21dc1890f434b5f22011192e98b8460ffc9c36d2

[danilchap, 2017-03-06 10:28:26]

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

[commit-bot: I haz the power, 2017-03-06 10:28:28]

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

[danilchap, 2017-03-06 10:35:07]

Description was changed from

==========
Replace implementation of Clock::CurrentNtp with Clock::CurrentNtpTime
Deprecated Clock::CurrentNtp
Replaced remaining usage of CurrentNtp with CurrentNtpTime

CurrentNtp return time by taking two output parameters by reference
(also breaks style guide)
CurrentNtpTime treat ntp time as single entity and returns it.
(making interface clearer)

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

to

==========
Replace Clock::CurrentNtp with Clock::CurrentNtpTime

CurrentNtp return time by taking two output parameters by reference
(also breaks style guide)
CurrentNtpTime treat ntp time as single entity and returns it.
(making interface clearer)

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

[danilchap, 2017-03-06 11:36:38]

Description was changed from

==========
Replace Clock::CurrentNtp with Clock::CurrentNtpTime

CurrentNtp return time by taking two output parameters by reference
(also breaks style guide)
CurrentNtpTime treat ntp time as single entity and returns it.
(making interface clearer)

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

to

==========
Replace Clock::CurrentNtp with Clock::CurrentNtpTime

CurrentNtp return time by taking two output parameters by reference
(also breaks style guide)
CurrentNtpTime treat ntp time as single entity and returns it using NtpTime
structure.
(making interface clearer)

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

[danilchap, 2017-03-06 11:59:43]

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

[perkj_webrtc, 2017-03-06 19:37:40]

lgtm

[stefan-webrtc, 2017-03-07 10:15:46]

lgtm

[danilchap, 2017-03-07 10:20:04]

The CQ bit was checked by danilchap@webrtc.org

[commit-bot: I haz the power, 2017-03-07 10:20:08]

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-03-07 10:51:09]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1488882004217890,
"parent_rev": "92a7a1810c0dd50c50d723621cfdddc8c721d0af", "commit_rev":
"21dc1890f434b5f22011192e98b8460ffc9c36d2"}

[commit-bot: I haz the power, 2017-03-07 10:51:12]

Description was changed from

==========
Replace Clock::CurrentNtp with Clock::CurrentNtpTime

CurrentNtp return time by taking two output parameters by reference
(also breaks style guide)
CurrentNtpTime treat ntp time as single entity and returns it using NtpTime
structure.
(making interface clearer)

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

to

==========
Replace Clock::CurrentNtp with Clock::CurrentNtpTime

CurrentNtp return time by taking two output parameters by reference
(also breaks style guide)
CurrentNtpTime treat ntp time as single entity and returns it using NtpTime
structure.
(making interface clearer)

BUG=None

Review-Url: https://codereview.webrtc.org/2733823002
Cr-Commit-Position: refs/heads/master@{#17088}
Committed:
https://chromium.googlesource.com/external/webrtc/+/21dc1890f434b5f22011192e9...
==========

[commit-bot: I haz the power, 2017-03-07 10:51:12]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/webrtc/+/21dc1890f434b5f22011192e9...