use base::Time constants in net module instead of using static constants

use base::Time constants in net module instead of using static constants

use base::Time constants in net module instead of using static constants in file

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

[prashant.hiremath, 2014-09-17 02:45:19]

prashhir@cisco.com changed reviewers:
+ davidben@chromium.org, rch@chromium.org

[prashant.hiremath, 2014-09-17 02:45:47]

Please Review

[davidben, 2014-09-17 13:32:00]

Looks good with a couple nits.

It looks like you've signed the individual CLA, but with a different email
address. Would you mind either contributing as that address or signing the CLA
again with this one? Thanks!

http://www.chromium.org/developers/contributing-code/external-contributor-che...

https://codereview.chromium.org/578703003/diff/1/net/dns/mdns_client_impl.cc
File net/dns/mdns_client_impl.cc (left):

https://codereview.chromium.org/578703003/diff/1/net/dns/mdns_client_impl.cc#...
net/dns/mdns_client_impl.cc:36: 
Really nitpicky nit: I would leave the blank line since the one on line 27 is
still there.

https://codereview.chromium.org/578703003/diff/1/net/dns/mdns_client_impl.cc
File net/dns/mdns_client_impl.cc (right):

https://codereview.chromium.org/578703003/diff/1/net/dns/mdns_client_impl.cc#...
net/dns/mdns_client_impl.cc:10: #include "base/time/default_clock.h"
Nit: This should
  #include "base/time/time.h"
since you're using a constant from there directly. (Really should have been
included already for base::Time and base::TimeDelta.)

[prashant.hiremath, 2014-09-17 15:25:42]

On 2014/09/17 13:32:00, David Benjamin wrote:
> Looks good with a couple nits.
> 
> It looks like you've signed the individual CLA, but with a different email
> address. Would you mind either contributing as that address or signing the CLA
> again with this one? Thanks!
> 
>
http://www.chromium.org/developers/contributing-code/external-contributor-che...
> 

hi David, Cisco has already signed the Corporate CLA, and my name is added in
list of authorized contributors, can you please verify once?

[davidben, 2014-09-17 16:04:40]

On 2014/09/17 15:25:42, prashant.hiremath wrote:
> hi David, Cisco has already signed the Corporate CLA, and my name is added in
> list of authorized contributors, can you please verify once?

Ah, I see it now. Sorry, I missed that there was a separate tab for corporate
signers.

[prashant.hiremath, 2014-09-17 16:16:59]

PTAL

[davidben, 2014-09-17 16:22:55]

lgtm. Thanks!

[davidben, 2014-09-17 16:24:19]

The CQ bit was checked by davidben@chromium.org

[commit-bot: I haz the power, 2014-09-17 16:25:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/578703003/20001

[commit-bot: I haz the power, 2014-09-17 17:27:35]

Committed patchset #2 (id:20001) as 9e6133128703c16c9821e4b14440a2b565991718

[commit-bot: I haz the power, 2014-09-17 17:28:09]

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