Add empty implementation of GetSystemMemoryInfo for IOS.

Add empty implementation of GetSystemMemoryInfo for IOS.

BUG=514221
Committed: https://crrev.com/ae7209aeafdcb56f6761aeffe27f992fe2571a04
Cr-Commit-Position: refs/heads/master@{#342688}

[Georges Khalil, 2015-08-05 15:18:22]

Patchset #2 (id:20001) has been deleted

[Georges Khalil, 2015-08-05 15:19:20]

Patchset #1 (id:1) has been deleted

[Georges Khalil, 2015-08-05 15:19:24]

Patchset #1 (id:1) has been deleted

[Georges Khalil, 2015-08-05 18:26:00]

Patchset #1 (id:40001) has been deleted

[Georges Khalil, 2015-08-05 18:26:09]

Patchset #1 (id:40001) has been deleted

[Georges Khalil, 2015-08-05 18:28:20]

Patchset #1 (id:60001) has been deleted

[Georges Khalil, 2015-08-05 19:35:11]

georgesak@chromium.org changed reviewers:
+ thakis@chromium.org

[Georges Khalil, 2015-08-05 19:35:11]

thakis@, PTAL.

As per our earlier discussion in 1257813002, I added an empty implementation of
GetSystemMemoryInfo for IOS.

[Georges Khalil, 2015-08-10 18:24:20]

georgesak@chromium.org changed reviewers:
+ thestig@chromium.org
- thakis@chromium.org

[Georges Khalil, 2015-08-10 18:24:21]

I just realized I had the wrong reviewer! Nico, sorry for the noise.

-thakis@
+thestig@

[Lei Zhang, 2015-08-10 18:42:31]

lgtm

https://codereview.chromium.org/1264683004/diff/80001/base/process/process_me...
File base/process/process_metrics.h (right):

https://codereview.chromium.org/1264683004/diff/80001/base/process/process_me...
base/process/process_metrics.h:251: // GetSystemMemoryInfo. Total/free swap
memory are available on all platforms
nit: GetSystemMemoryInfo()

https://codereview.chromium.org/1264683004/diff/80001/base/process/process_me...
File base/process/process_metrics_ios.cc (right):

https://codereview.chromium.org/1264683004/diff/80001/base/process/process_me...
base/process/process_metrics_ios.cc:92: // Unimplemented. Must enable unittest
for IOS when this gets implemented.
Add NOTIMPLEMENTED() ?

[Georges Khalil, 2015-08-10 18:49:35]

Thanks, addressed comments. Landing this.

https://codereview.chromium.org/1264683004/diff/80001/base/process/process_me...
File base/process/process_metrics.h (right):

https://codereview.chromium.org/1264683004/diff/80001/base/process/process_me...
base/process/process_metrics.h:251: // GetSystemMemoryInfo. Total/free swap
memory are available on all platforms
On 2015/08/10 18:42:31, Lei Zhang wrote:
> nit: GetSystemMemoryInfo()

Done.

https://codereview.chromium.org/1264683004/diff/80001/base/process/process_me...
File base/process/process_metrics_ios.cc (right):

https://codereview.chromium.org/1264683004/diff/80001/base/process/process_me...
base/process/process_metrics_ios.cc:92: // Unimplemented. Must enable unittest
for IOS when this gets implemented.
On 2015/08/10 18:42:31, Lei Zhang wrote:
> Add NOTIMPLEMENTED() ?

Done.

[Georges Khalil, 2015-08-10 18:49:40]

The CQ bit was checked by georgesak@chromium.org

[Georges Khalil, 2015-08-10 18:49:40]

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/1264683004/#ps100001
(title: "thestig@ comments.")

[commit-bot: I haz the power, 2015-08-10 18:50:47]

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

[commit-bot: I haz the power, 2015-08-10 20:40:15]

Committed patchset #2 (id:100001)

[commit-bot: I haz the power, 2015-08-10 20:41:08]

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