GRC: Hook up process-level CoordinationUnits in the browser process

GRC: Hook up process-level CoordinationUnits in the browser process

It's created on demand, and currently won't be instantiated without the GlobalResourceCoordinator feature flag.

An example of full GRC usage can be seen in https://codereview.chromium.org/2710823003

This CL depends on https://codereview.chromium.org/2913253002

R=nasko@chromium.org
BUG=691886
Review-Url: https://codereview.chromium.org/2921663003
Cr-Commit-Position: refs/heads/master@{#477217}
Committed: https://chromium.googlesource.com/chromium/src/+/2ba96b06da25d3e9e905a3a6d2c22f347ecdc9fe

[oystein (OOO til 10th of July), 2017-06-01 17:07:13]

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

[commit-bot: I haz the power, 2017-06-01 17:07:46]

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-06-01 17:14:55]

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

[commit-bot: I haz the power, 2017-06-01 17:14:58]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  android_cronet on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_cron...)
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[oystein (OOO til 10th of July), 2017-06-01 21:05:43]

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

[oystein (OOO til 10th of July), 2017-06-01 21:06:10]

nasko: ptal :). Similar thing to https://codereview.chromium.org/2913253002

[commit-bot: I haz the power, 2017-06-01 21:06:10]

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-06-01 22:38:35]

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

[commit-bot: I haz the power, 2017-06-01 22:38:36]

Dry run: This issue passed the CQ dry run.

[nasko, 2017-06-03 00:26:52]

Can we add to this CL the code that uses it outside of content? The public API
policy is to always have the code using the API in the same CL that exposes it.

[oystein (OOO til 10th of July), 2017-06-05 20:08:17]

On 2017/06/03 at 00:26:52, nasko wrote:
> Can we add to this CL the code that uses it outside of content? The public API
policy is to always have the code using the API in the same CL that exposes it.

Done; ptal. Adds https://codereview.chromium.org/2913253002 as a dependency
since these both get hooked up in
ResourceCoordinatorWebContentsObserver::DidFinishNavigation.

[oystein (OOO til 10th of July), 2017-06-05 20:08:32]

Description was changed from

==========
GRC: Hook up process-level CoordinationUnits in the browser process

It's created on demand, and currently won't be instantiated without the
GlobalResourceCoordinator feature flag.

An example of full GRC usage can be seen in
https://codereview.chromium.org/2710823003

R=nasko@chromium.org
BUG=691886
==========

to

==========
GRC: Hook up process-level CoordinationUnits in the browser process

It's created on demand, and currently won't be instantiated without the
GlobalResourceCoordinator feature flag.

An example of full GRC usage can be seen in
https://codereview.chromium.org/2710823003

This CL depends on https://codereview.chromium.org/2913253002

R=nasko@chromium.org
BUG=691886
==========

[nasko, 2017-06-05 23:59:21]

LGTM

[oystein (OOO til 10th of July), 2017-06-06 04:08:54]

The CQ bit was checked by oysteine@chromium.org

[commit-bot: I haz the power, 2017-06-06 04:09:17]

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-06-06 06:43:49]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1496722134926850,
"parent_rev": "0e42c06947d8e001c30bdc0efafefedd8afa2256", "commit_rev":
"2ba96b06da25d3e9e905a3a6d2c22f347ecdc9fe"}

[commit-bot: I haz the power, 2017-06-06 06:44:01]

Description was changed from

==========
GRC: Hook up process-level CoordinationUnits in the browser process

It's created on demand, and currently won't be instantiated without the
GlobalResourceCoordinator feature flag.

An example of full GRC usage can be seen in
https://codereview.chromium.org/2710823003

This CL depends on https://codereview.chromium.org/2913253002

R=nasko@chromium.org
BUG=691886
==========

to

==========
GRC: Hook up process-level CoordinationUnits in the browser process

It's created on demand, and currently won't be instantiated without the
GlobalResourceCoordinator feature flag.

An example of full GRC usage can be seen in
https://codereview.chromium.org/2710823003

This CL depends on https://codereview.chromium.org/2913253002

R=nasko@chromium.org
BUG=691886

Review-Url: https://codereview.chromium.org/2921663003
Cr-Commit-Position: refs/heads/master@{#477217}
Committed:
https://chromium.googlesource.com/chromium/src/+/2ba96b06da25d3e9e905a3a6d2c2...
==========

[commit-bot: I haz the power, 2017-06-06 06:44:02]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/2ba96b06da25d3e9e905a3a6d2c2...