Add MemoryCoordinator::GetGlobalMemoryState()

Add MemoryCoordinator::GetGlobalMemoryState()

MemoryCoordinator::GetCurrentMemoryState() is not always the same as
the global memory state because the browser process won't be suspended
(at least for now). On the other hand, we want to have a way to record
the global memory state to tweak memory coordinator's parameters.
Add GetGlobalMemoryState() method which returns the raw value of the
global state. Keep GetCurrentMemoryState() for clients in the browser
process.

BUG=662772
Committed: https://crrev.com/5e2c875dca161e2b897d4cf454633c424b52f55b
Cr-Commit-Position: refs/heads/master@{#435873}

[bashi, 2016-12-02 05:44:49]

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

[commit-bot: I haz the power, 2016-12-02 05:45:12]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[bashi, 2016-12-02 05:47:27]

bashi@chromium.org changed reviewers:
+ chrisha@chromium.org, haraken@chromium.org

[bashi, 2016-12-02 05:47:28]

PTAL

I noticed that we don't record SUSPENDED state correctly :(

[haraken, 2016-12-02 05:55:35]

LGTM

Maybe it's worth renaming CurrentMemoryState to LocalMemoryState for clarity.

[bashi, 2016-12-02 05:59:55]

On 2016/12/02 05:55:35, haraken wrote:
> LGTM
> 
> Maybe it's worth renaming CurrentMemoryState to LocalMemoryState for clarity.

Yeah, but let me submit this CL first. I attempted the renaming but didn't do
that because I couldn't came up with a better name for
SetCurrentMemoryStateForTesting().

[bashi, 2016-12-02 06:00:41]

The CQ bit was unchecked by bashi@chromium.org

[bashi, 2016-12-02 06:00:42]

The CQ bit was checked by bashi@chromium.org

[commit-bot: I haz the power, 2016-12-02 06:00:47]

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, 2016-12-02 07:05:44]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480658442864760, "parent_rev":
"168f907a6ea6e3b69c873c57c68d222b3a410ad8", "commit_rev":
"c49585309894d4d7aca5b40c0727799b3d006cb4"}

[commit-bot: I haz the power, 2016-12-02 07:06:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-02 07:08:21]

Description was changed from

==========
Add MemoryCoordinator::GetGlobalMemoryState()

MemoryCoordinator::GetCurrentMemoryState() is not always the same as
the global memory state because the browser process won't be suspended
(at least for now). On the other hand, we want to have a way to record
the global memory state to tweak memory coordinator's parameters.
Add GetGlobalMemoryState() method which returns the raw value of the
global state. Keep GetCurrentMemoryState() for clients in the browser
process.

BUG=662772
==========

to

==========
Add MemoryCoordinator::GetGlobalMemoryState()

MemoryCoordinator::GetCurrentMemoryState() is not always the same as
the global memory state because the browser process won't be suspended
(at least for now). On the other hand, we want to have a way to record
the global memory state to tweak memory coordinator's parameters.
Add GetGlobalMemoryState() method which returns the raw value of the
global state. Keep GetCurrentMemoryState() for clients in the browser
process.

BUG=662772

Committed: https://crrev.com/5e2c875dca161e2b897d4cf454633c424b52f55b
Cr-Commit-Position: refs/heads/master@{#435873}
==========

[commit-bot: I haz the power, 2016-12-02 07:08:23]

Patchset 1 (id:??) landed as
https://crrev.com/5e2c875dca161e2b897d4cf454633c424b52f55b
Cr-Commit-Position: refs/heads/master@{#435873}