Initialize ChildMemoryCoordinator before blink

Initialize ChildMemoryCoordinator before blink

Some features of memory coordinator will be exposed to v8. We should
initialize ChildMemoryCoordinator before blink to make sure that v8
can access memory coordinator features from the beginning.

BUG=693430
Review-Url: https://codereview.chromium.org/2775583002
Cr-Commit-Position: refs/heads/master@{#459308}
Committed: https://chromium.googlesource.com/chromium/src/+/d43ce5d5dc0b568a1c0ec616012ebc38e9d9e8b3

[bashi, 2017-03-23 07:17:35]

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

[commit-bot: I haz the power, 2017-03-23 07:17:47]

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

[bashi, 2017-03-23 07:19:19]

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

[commit-bot: I haz the power, 2017-03-23 07:19:19]

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

[bashi, 2017-03-23 07:19:20]

PTAL

This is a part of https://codereview.chromium.org/2731913002/. I want to make
the WIP CL as small as possible.

[commit-bot: I haz the power, 2017-03-23 07:19:20]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[commit-bot: I haz the power, 2017-03-23 07:19:42]

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

[commit-bot: I haz the power, 2017-03-23 07:19:43]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[bashi, 2017-03-23 07:22:17]

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

[commit-bot: I haz the power, 2017-03-23 07:22:31]

Dry run: 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-03-23 07:31:23]

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

[commit-bot: I haz the power, 2017-03-23 07:31:24]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[bashi, 2017-03-23 07:44:24]

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

[commit-bot: I haz the power, 2017-03-23 07:44:39]

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

[bashi, 2017-03-23 07:47:52]

https://codereview.chromium.org/2775583002/diff/40001/content/renderer/render...
File content/renderer/render_thread_impl.cc (left):

https://codereview.chromium.org/2775583002/diff/40001/content/renderer/render...
content/renderer/render_thread_impl.cc:836: // TODO(bashi): Revisit how to
manage the lifetime of
Removed this TODO as this looks a common pattern to bind interface.

[haraken, 2017-03-23 08:09:20]

LGTM

[commit-bot: I haz the power, 2017-03-23 08:29:13]

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

[commit-bot: I haz the power, 2017-03-23 08:29:14]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[bashi, 2017-03-23 23:10:20]

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

[bashi, 2017-03-23 23:10:21]

+avi@ for content/ OWNERs review

[Avi (use Gerrit), 2017-03-24 00:03:47]

lgtm

[bashi, 2017-03-24 00:07:15]

The CQ bit was checked by bashi@chromium.org

[commit-bot: I haz the power, 2017-03-24 00:07: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, 2017-03-24 00:51:22]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1490314035090150,
"parent_rev": "d604de96c960021b33f929e9cdf9eca8079ad92b", "commit_rev":
"d43ce5d5dc0b568a1c0ec616012ebc38e9d9e8b3"}

[commit-bot: I haz the power, 2017-03-24 00:52:10]

Description was changed from

==========
Initialize ChildMemoryCoordinator before blink

Some features of memory coordinator will be exposed to v8. We should
initialize ChildMemoryCoordinator before blink to make sure that v8
can access memory coordinator features from the beginning.

BUG=693430
==========

to

==========
Initialize ChildMemoryCoordinator before blink

Some features of memory coordinator will be exposed to v8. We should
initialize ChildMemoryCoordinator before blink to make sure that v8
can access memory coordinator features from the beginning.

BUG=693430

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

[commit-bot: I haz the power, 2017-03-24 00:52:11]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/d43ce5d5dc0b568a1c0ec616012e...

[bashi, 2017-03-28 00:39:58]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/2777393002/ by bashi@chromium.org.

The reason for reverting is: This caused renderer crashes.

crbug.com/705021
.