fuchsia: Implement SysInfo::AmountOfPhysicalMemory and NumberOfProcessors

fuchsia: Implement SysInfo::AmountOfPhysicalMemory and NumberOfProcessors

Ref:
https://fuchsia.googlesource.com/magenta/+/HEAD/docs/syscalls/system_get_physmem.md
https://fuchsia.googlesource.com/magenta/+/HEAD/docs/syscalls/system_get_num_cpus.md

BUG=706592
Review-Url: https://codereview.chromium.org/2894993002
Cr-Commit-Position: refs/heads/master@{#474179}
Committed: https://chromium.googlesource.com/chromium/src/+/aaa608f0db6a15e4fbe95e535636e3c8f28df598

[scottmg, 2017-05-23 22:02:17]

Description was changed from

==========
fuchsia: Implement SysInfo::AmountOfPhysicalMemory

BUG=706592
==========

to

==========
fuchsia: Implement SysInfo::AmountOfPhysicalMemory and NumberOfProcessors

BUG=706592
==========

[scottmg, 2017-05-24 00:05:47]

Description was changed from

==========
fuchsia: Implement SysInfo::AmountOfPhysicalMemory and NumberOfProcessors

BUG=706592
==========

to

==========
fuchsia: Implement SysInfo::AmountOfPhysicalMemory and NumberOfProcessors

See
https://fuchsia.googlesource.com/magenta/+/HEAD/docs/syscalls/system_get_num_...

and 

https://fuchsia.googlesource.com/magenta/+/HEAD/docs/syscalls/system_get_phys...

BUG=706592
==========

[scottmg, 2017-05-24 00:05:47]

scottmg@chromium.org changed reviewers:
+ thestig@chromium.org

[scottmg, 2017-05-24 00:06:15]

The CQ bit was checked by scottmg@chromium.org to run a CQ dry run

[scottmg, 2017-05-24 00:06:28]

Description was changed from

==========
fuchsia: Implement SysInfo::AmountOfPhysicalMemory and NumberOfProcessors

See
https://fuchsia.googlesource.com/magenta/+/HEAD/docs/syscalls/system_get_num_...

and 

https://fuchsia.googlesource.com/magenta/+/HEAD/docs/syscalls/system_get_phys...

BUG=706592
==========

to

==========
fuchsia: Implement SysInfo::AmountOfPhysicalMemory and NumberOfProcessors

Ref:
https://fuchsia.googlesource.com/magenta/+/HEAD/docs/syscalls/system_get_num_...
https://fuchsia.googlesource.com/magenta/+/HEAD/docs/syscalls/system_get_phys...

BUG=706592
==========

[scottmg, 2017-05-24 00:06:40]

Description was changed from

==========
fuchsia: Implement SysInfo::AmountOfPhysicalMemory and NumberOfProcessors

Ref:
https://fuchsia.googlesource.com/magenta/+/HEAD/docs/syscalls/system_get_num_...
https://fuchsia.googlesource.com/magenta/+/HEAD/docs/syscalls/system_get_phys...

BUG=706592
==========

to

==========
fuchsia: Implement SysInfo::AmountOfPhysicalMemory and NumberOfProcessors

Ref:
https://fuchsia.googlesource.com/magenta/+/HEAD/docs/syscalls/system_get_phys...
https://fuchsia.googlesource.com/magenta/+/HEAD/docs/syscalls/system_get_num_...

BUG=706592
==========

[commit-bot: I haz the power, 2017-05-24 00:08:47]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[scottmg, 2017-05-24 00:11:08]

The CQ bit was checked by scottmg@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-24 00:12:15]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Lei Zhang, 2017-05-24 00:19:08]

lgtm

Have you looked into enabling (parts of) base/sys_info_unittest.cc ?

[scottmg, 2017-05-24 00:26:23]

On 2017/05/24 00:19:08, Lei Zhang wrote:
> lgtm
> 
> Have you looked into enabling (parts of) base/sys_info_unittest.cc ?

I did just now. The one #ifdefd test that looks applicable here (secondary
validation via SysInfoTest.AmountOfAvailablePhysicalMemory) uses
base/process/process_metrics.h but we don't have that yet, so the basic sanity
tests will have to do for now.

[scottmg, 2017-05-24 00:30:06]

The CQ bit was checked by scottmg@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-24 00:31:03]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[scottmg, 2017-05-24 00:33:01]

The CQ bit was unchecked by scottmg@chromium.org

[scottmg, 2017-05-24 00:33:04]

The CQ bit was checked by scottmg@chromium.org

[scottmg, 2017-05-24 00:33:04]

The patchset sent to the CQ was uploaded after l-g-t-m from thestig@chromium.org
Link to the patchset: https://codereview.chromium.org/2894993002/#ps140001
(title: "rebase to origin/master")

[commit-bot: I haz the power, 2017-05-24 00:33:37]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[scottmg, 2017-05-24 03:53:26]

The CQ bit was checked by scottmg@chromium.org

[scottmg, 2017-05-24 03:53:26]

The patchset sent to the CQ was uploaded after l-g-t-m from thestig@chromium.org
Link to the patchset: https://codereview.chromium.org/2894993002/#ps160001
(title: "rebase")

[commit-bot: I haz the power, 2017-05-24 03:54:16]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-24 05:56:12]

CQ is committing da patch.

Bot data: {"patchset_id": 160001, "attempt_start_ts": 1495598006258900,
"parent_rev": "ae952008457a2f703d91e24b301efb59718fce87", "commit_rev":
"aaa608f0db6a15e4fbe95e535636e3c8f28df598"}

[commit-bot: I haz the power, 2017-05-24 05:56:23]

Description was changed from

==========
fuchsia: Implement SysInfo::AmountOfPhysicalMemory and NumberOfProcessors

Ref:
https://fuchsia.googlesource.com/magenta/+/HEAD/docs/syscalls/system_get_phys...
https://fuchsia.googlesource.com/magenta/+/HEAD/docs/syscalls/system_get_num_...

BUG=706592
==========

to

==========
fuchsia: Implement SysInfo::AmountOfPhysicalMemory and NumberOfProcessors

Ref:
https://fuchsia.googlesource.com/magenta/+/HEAD/docs/syscalls/system_get_phys...
https://fuchsia.googlesource.com/magenta/+/HEAD/docs/syscalls/system_get_num_...

BUG=706592

Review-Url: https://codereview.chromium.org/2894993002
Cr-Commit-Position: refs/heads/master@{#474179}
Committed:
https://chromium.googlesource.com/chromium/src/+/aaa608f0db6a15e4fbe95e535636...
==========

[commit-bot: I haz the power, 2017-05-24 05:56:24]

Committed patchset #9 (id:160001) as
https://chromium.googlesource.com/chromium/src/+/aaa608f0db6a15e4fbe95e535636...