base: SysInfo::Uptime() returns a TimeDelta

base: SysInfo::Uptime() returns a TimeDelta

This makes more sense than returning a number of milliseconds, especially
when the function name does not reflect the return value type.

In a few places, the number of milliseconds is converted into a TimeDelta
anyway. So this patch simplifies some of the code out there.

BUG=chromium:157549
TEST=build successfully, SysInfoTest passes
TBR=jif@chromium.org

Signed-off-by: Simon Que <sque@chromium.org>;

Committed: https://crrev.com/0f78ba540eb43fb2bb3b7726eeb41d22bfee09a3
Cr-Commit-Position: refs/heads/master@{#357937}

[Simon Que, 2015-11-04 00:46:49]

sque@chromium.org changed reviewers:
+ mark@chromium.org, thakis@chromium.org

[Simon Que, 2015-11-04 00:46:49]

Notes for getting LGTMs from individual owners:

chrome/browser/ui: sky, pkasting
chrome/browser: isherman, asvitkine
content: jam, piman
components: gab, erikchen

[Nico, 2015-11-04 00:59:39]

lgtm!

[Nico, 2015-11-04 01:00:20]

On 2015/11/04 00:59:39, Nico wrote:
> lgtm!

(sky, pkasting: I looked at the chrome/ files too, so you don't need to look at
that. I don't have OWNERS powers in content or components though.)

[Mark Mentovai, 2015-11-04 01:16:00]

LGTM

[Simon Que, 2015-11-04 01:46:14]

sque@chromium.org changed reviewers:
+ blundell@chromium.org, jam@chromium.org, jochen@chromium.org

[Simon Que, 2015-11-04 01:46:14]

+jochen +jam for contents/
+jochen +blundell for components/

[blundell, 2015-11-04 10:06:07]

lgtm

[jam, 2015-11-04 18:32:00]

lgtm

[Simon Que, 2015-11-04 20:03:42]

The CQ bit was checked by sque@chromium.org

[commit-bot: I haz the power, 2015-11-04 20:05:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1417673009/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1417673009/1

[commit-bot: I haz the power, 2015-11-04 20:09:25]

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

[commit-bot: I haz the power, 2015-11-04 20:09:25]

Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Simon Que, 2015-11-04 21:36:52]

The CQ bit was checked by sque@chromium.org

[Simon Que, 2015-11-04 21:36:52]

The patchset sent to the CQ was uploaded after l-g-t-m from mark@chromium.org,
jam@chromium.org, thakis@chromium.org, blundell@chromium.org
Link to the patchset: https://codereview.chromium.org/1417673009/#ps20001
(title: "Rebased")

[commit-bot: I haz the power, 2015-11-04 21:37:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1417673009/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1417673009/20001

[Simon Que, 2015-11-04 21:51:30]

sque@chromium.org changed reviewers:
+ jif@chromium.org

[Simon Que, 2015-11-04 21:51:30]

+jif for components/open_from_clipboard/clipboard_recent_content_ios.mm

[Nico, 2015-11-04 22:09:43]

Feel free to tbr this, for mechanical changes with general approval that's
OK
On Nov 4, 2015 1:51 PM, <sque@chromium.org> wrote:

> +jif for components/open_from_clipboard/clipboard_recent_content_ios.mm
>
> https://codereview.chromium.org/1417673009/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Simon Que, 2015-11-04 22:33:57]

Description was changed from

==========
base: SysInfo::Uptime() returns a TimeDelta

This makes more sense than returning a number of milliseconds, especially
when the function name does not reflect the return value type.

In a few places, the number of milliseconds is converted into a TimeDelta
anyway. So this patch simplifies some of the code out there.

BUG=chromium:157548
TEST=build successfully, SysInfoTest passes

Signed-off-by: Simon Que <sque@chromium.org>
==========

to

==========
base: SysInfo::Uptime() returns a TimeDelta

This makes more sense than returning a number of milliseconds, especially
when the function name does not reflect the return value type.

In a few places, the number of milliseconds is converted into a TimeDelta
anyway. So this patch simplifies some of the code out there.

BUG=chromium:157548
TEST=build successfully, SysInfoTest passes
TBR=jif@chromium.org

Signed-off-by: Simon Que <sque@chromium.org>
==========

[Simon Que, 2015-11-04 22:40:52]

The CQ bit was checked by sque@chromium.org

[Simon Que, 2015-11-04 22:40:52]

The patchset sent to the CQ was uploaded after l-g-t-m from mark@chromium.org,
jam@chromium.org, thakis@chromium.org, blundell@chromium.org
Link to the patchset: https://codereview.chromium.org/1417673009/#ps60001
(title: "Update perf_provider_chromeos.cc (just updated in another CL)")

[commit-bot: I haz the power, 2015-11-04 22:42:01]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1417673009/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1417673009/60001

[commit-bot: I haz the power, 2015-11-05 00:01:13]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-11-05 00:01:58]

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

[Simon Que, 2015-11-05 00:52:00]

Description was changed from

==========
base: SysInfo::Uptime() returns a TimeDelta

This makes more sense than returning a number of milliseconds, especially
when the function name does not reflect the return value type.

In a few places, the number of milliseconds is converted into a TimeDelta
anyway. So this patch simplifies some of the code out there.

BUG=chromium:157548
TEST=build successfully, SysInfoTest passes
TBR=jif@chromium.org

Signed-off-by: Simon Que <sque@chromium.org>

Committed: https://crrev.com/0f78ba540eb43fb2bb3b7726eeb41d22bfee09a3
Cr-Commit-Position: refs/heads/master@{#357937}
==========

to

==========
base: SysInfo::Uptime() returns a TimeDelta

This makes more sense than returning a number of milliseconds, especially
when the function name does not reflect the return value type.

In a few places, the number of milliseconds is converted into a TimeDelta
anyway. So this patch simplifies some of the code out there.

BUG=chromium:157549
TEST=build successfully, SysInfoTest passes
TBR=jif@chromium.org

Signed-off-by: Simon Que <sque@chromium.org>

Committed: https://crrev.com/0f78ba540eb43fb2bb3b7726eeb41d22bfee09a3
Cr-Commit-Position: refs/heads/master@{#357937}
==========

[jif-google, 2015-11-05 09:46:42]

jif@google.com changed reviewers:
+ jif@google.com

[jif-google, 2015-11-05 09:46:42]

lgtm