HostDiscardableSharedMemoryManager should store the default memory limit

HostDiscardableSharedMemoryManager should store the default memory limit

OnMemoryStateChange() callback is invoked on the UI thread but we can't
call GetDefaultMemoryLimit() from the UI thread. Store the default limit
on construction.

BUG=652557
Committed: https://crrev.com/738aeed44f1e6f1bdbc3bf2e34eeab91e9726888
Cr-Commit-Position: refs/heads/master@{#422700}

[bashi, 2016-10-04 03:07:55]

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

[commit-bot: I haz the power, 2016-10-04 03:08:10]

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

[bashi, 2016-10-04 03:13:10]

bashi@chromium.org changed reviewers:
+ avi@chromium.org, hajimehoshi@chromium.org

[bashi, 2016-10-04 03:13:10]

PTAL

https://codereview.chromium.org/2388403002/diff/1/content/common/host_discard...
File content/common/host_discardable_shared_memory_manager.cc (right):

https://codereview.chromium.org/2388403002/diff/1/content/common/host_discard...
content/common/host_discardable_shared_memory_manager.cc:167: :
default_memory_limit_(GetDefaultMemoryLimit()),
Though GetDefaultMemoryLimit() could be dynamically changed, I think using the
initial value as a default value makes sense because (1) the implementation is
simpler and (2) amount of free disk space isn't a primary metric.

[commit-bot: I haz the power, 2016-10-04 03:57:32]

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

[commit-bot: I haz the power, 2016-10-04 03:57:33]

Dry run: This issue passed the CQ dry run.

[Avi (use Gerrit), 2016-10-04 04:05:28]

lgtm

[bashi, 2016-10-04 04:06:07]

The CQ bit was checked by bashi@chromium.org

[commit-bot: I haz the power, 2016-10-04 04:06:25]

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-10-04 04:11:35]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-04 04:13:48]

Description was changed from

==========
HostDiscardableSharedMemoryManager should store the default memory limit

OnMemoryStateChange() callback is invoked on the UI thread but we can't
call GetDefaultMemoryLimit() from the UI thread. Store the default limit
on construction.

BUG=652557
==========

to

==========
HostDiscardableSharedMemoryManager should store the default memory limit

OnMemoryStateChange() callback is invoked on the UI thread but we can't
call GetDefaultMemoryLimit() from the UI thread. Store the default limit
on construction.

BUG=652557

Committed: https://crrev.com/738aeed44f1e6f1bdbc3bf2e34eeab91e9726888
Cr-Commit-Position: refs/heads/master@{#422700}
==========

[commit-bot: I haz the power, 2016-10-04 04:13:49]

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

[hajimehoshi, 2016-10-04 04:18:23]

lgtm