Use MockRenderProcessHost in memory coordinator unittests

Use MockRenderProcessHost in memory coordinator unittests

Calling RenderProcessHost::FromID() directly prevents us to write
unittests. Add a virtual method for getting RenderProcessHost so that
testing code can use MockRenderProcessHost to write unittests.

Also, this CL installs a MemoryCoordinatorDelegate in unittests so that
unittests can have wider coverage.

BUG=675811
Committed: https://crrev.com/d37e4c892eed8e2ad96de67f347254b846531ede
Cr-Commit-Position: refs/heads/master@{#439764}

[bashi, 2016-12-20 05:39:23]

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

[commit-bot: I haz the power, 2016-12-20 05:39:38]

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-20 06:30:57]

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

[bashi, 2016-12-20 06:30:58]

PTAL

[commit-bot: I haz the power, 2016-12-20 06:46:41]

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

[commit-bot: I haz the power, 2016-12-20 06:46:42]

Dry run: This issue passed the CQ dry run.

[haraken, 2016-12-20 09:42:11]

LGTM

[bashi, 2016-12-20 10:35:42]

The CQ bit was checked by bashi@chromium.org

[commit-bot: I haz the power, 2016-12-20 10:36:00]

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-20 10:41:32]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482230142005770, "parent_rev":
"8216efa3353a8e0d0207c4da804f756eb436cfdd", "commit_rev":
"faffa1489261ed384fd4f1360fc08b85cb90018e"}

[commit-bot: I haz the power, 2016-12-20 10:41:57]

Description was changed from

==========
Use MockRenderProcessHost in memory coordinator unittests

Calling RenderProcessHost::FromID() directly prevents us to write
unittests. Add a virtual method for getting RenderProcessHost so that
testing code can use MockRenderProcessHost to write unittests.

Also, this CL installs a MemoryCoordinatorDelegate in unittests so that
unittests can have wider coverage.

BUG=675811
==========

to

==========
Use MockRenderProcessHost in memory coordinator unittests

Calling RenderProcessHost::FromID() directly prevents us to write
unittests. Add a virtual method for getting RenderProcessHost so that
testing code can use MockRenderProcessHost to write unittests.

Also, this CL installs a MemoryCoordinatorDelegate in unittests so that
unittests can have wider coverage.

BUG=675811

Review-Url: https://codereview.chromium.org/2594513002
==========

[commit-bot: I haz the power, 2016-12-20 10:41:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-20 10:45:39]

Description was changed from

==========
Use MockRenderProcessHost in memory coordinator unittests

Calling RenderProcessHost::FromID() directly prevents us to write
unittests. Add a virtual method for getting RenderProcessHost so that
testing code can use MockRenderProcessHost to write unittests.

Also, this CL installs a MemoryCoordinatorDelegate in unittests so that
unittests can have wider coverage.

BUG=675811

Review-Url: https://codereview.chromium.org/2594513002
==========

to

==========
Use MockRenderProcessHost in memory coordinator unittests

Calling RenderProcessHost::FromID() directly prevents us to write
unittests. Add a virtual method for getting RenderProcessHost so that
testing code can use MockRenderProcessHost to write unittests.

Also, this CL installs a MemoryCoordinatorDelegate in unittests so that
unittests can have wider coverage.

BUG=675811

Committed: https://crrev.com/d37e4c892eed8e2ad96de67f347254b846531ede
Cr-Commit-Position: refs/heads/master@{#439764}
==========

[commit-bot: I haz the power, 2016-12-20 10:45:40]

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