use base::Time constants in mojo instead of defining locally

use base::Time constants in mojo instead of defining locally

BUG=412254
Committed: https://crrev.com/75b0eb8eb60d9d8d736a89c3e63c3163089ee8a8
Cr-Commit-Position: refs/heads/master@{#299088}

[prashant.hiremath, 2014-09-30 08:15:52]

prashhir@cisco.com changed reviewers:
+ davemoore@chromium.org

[prashant.hiremath, 2014-09-30 08:15:53]

Please Review

[prashant.hiremath, 2014-10-09 15:04:18]

Gentle ping.

[darin (slow to review), 2014-10-09 17:12:48]

darin@chromium.org changed reviewers:
+ darin@chromium.org

[darin (slow to review), 2014-10-09 17:12:50]

https://codereview.chromium.org/614893002/diff/1/mojo/apps/js/bindings/monoto...
File mojo/apps/js/bindings/monotonic_clock.cc (right):

https://codereview.chromium.org/614893002/diff/1/mojo/apps/js/bindings/monoto...
mojo/apps/js/bindings/monotonic_clock.cc:21: return
static_cast<double>(mojo::GetTimeTicksNow()) /
base::Time::kMicrosecondsPerSecond;
nit: need to reformat for 80 character line limit

[prashant.hiremath, 2014-10-09 17:44:40]

https://codereview.chromium.org/614893002/diff/1/mojo/apps/js/bindings/monoto...
File mojo/apps/js/bindings/monotonic_clock.cc (right):

https://codereview.chromium.org/614893002/diff/1/mojo/apps/js/bindings/monoto...
mojo/apps/js/bindings/monotonic_clock.cc:21: return
static_cast<double>(mojo::GetTimeTicksNow()) /
base::Time::kMicrosecondsPerSecond;
On 2014/10/09 17:12:50, darin wrote:
> nit: need to reformat for 80 character line limit
I,m confused between moving second part i.e base::Time::kMicrosecondsPerSecond
to next line or using namespace base::Time; and keeping 2nd part as previous.
Which one would you think better?

[prashant.hiremath, 2014-10-10 02:29:12]

PTAL

[qsr, 2014-10-10 09:02:11]

qsr@chromium.org changed reviewers:
+ qsr@chromium.org

[qsr, 2014-10-10 09:02:11]

https://codereview.chromium.org/614893002/diff/50001/mojo/apps/js/bindings/mo...
File mojo/apps/js/bindings/monotonic_clock.cc (right):

https://codereview.chromium.org/614893002/diff/50001/mojo/apps/js/bindings/mo...
mojo/apps/js/bindings/monotonic_clock.cc:22: base::Time::kMicrosecondsPerSecond;
continuation should indent by 4 characters.

[prashant.hiremath, 2014-10-10 09:22:37]

PTAL

[qsr, 2014-10-10 09:29:33]

https://codereview.chromium.org/614893002/diff/110001/mojo/apps/js/bindings/m...
File mojo/apps/js/bindings/monotonic_clock.cc (right):

https://codereview.chromium.org/614893002/diff/110001/mojo/apps/js/bindings/m...
mojo/apps/js/bindings/monotonic_clock.cc:22: base::Time::kMicrosecondsPerSecond;
Sorry, what I meant was 4 characters more than the previous line.

[prashant.hiremath, 2014-10-10 09:37:50]

indented as per review comment

[qsr, 2014-10-10 10:11:07]

lgtm

[prashant.hiremath, 2014-10-10 10:28:11]

The CQ bit was checked by prashhir@cisco.com

[commit-bot: I haz the power, 2014-10-10 10:29:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/614893002/150001

[commit-bot: I haz the power, 2014-10-10 11:07:30]

Committed patchset #4 (id:150001)

[commit-bot: I haz the power, 2014-10-10 11:08:12]

Patchset 4 (id:??) landed as
https://crrev.com/75b0eb8eb60d9d8d736a89c3e63c3163089ee8a8
Cr-Commit-Position: refs/heads/master@{#299088}