Allow injecting a PrefService into ChromeComponentUpdaterConfigurator.

Allow injecting a PrefService into ChromeComponentUpdaterConfigurator.

This is a mechanical change for the purpose of testing future changes
related to group policies in the component updater.

R=sky@chromium.org
TBR=waffles@chromium.org

BUG=637452
Committed: https://crrev.com/dd63a936012d94f7ad6f6e36f09e5de68262c4e1
Cr-Commit-Position: refs/heads/master@{#411935}

[Sorin Jianu, 2016-08-12 22:56:44]

Description was changed from

==========
Allow injecting a PrefService into ChromeComponentUpdaterConfigurator.

This is a mechanical change for the purpose of testing future changes
related to group policies in the component updater.

R=thakis@chromium.org
TBR=waffles@chromium.org

BUG=637452
==========

to

==========
Allow injecting a PrefService into ChromeComponentUpdaterConfigurator.

This is a mechanical change for the purpose of testing future changes
related to group policies in the component updater.

R=jam@chromium.org
TBR=waffles@chromium.org

BUG=637452
==========

[Sorin Jianu, 2016-08-12 22:56:44]

sorin@chromium.org changed reviewers:
+ jam@chromium.org
- thakis@chromium.org

[Sorin Jianu, 2016-08-12 22:57:14]

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

[commit-bot: I haz the power, 2016-08-12 22:58:01]

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

[Sorin Jianu, 2016-08-12 22:59:19]

John, please, I need an owner's approval for changes to browser_process_impl.cc

[Sorin Jianu, 2016-08-12 23:23:12]

Description was changed from

==========
Allow injecting a PrefService into ChromeComponentUpdaterConfigurator.

This is a mechanical change for the purpose of testing future changes
related to group policies in the component updater.

R=jam@chromium.org
TBR=waffles@chromium.org

BUG=637452
==========

to

==========
Allow injecting a PrefService into ChromeComponentUpdaterConfigurator.

This is a mechanical change for the purpose of testing future changes
related to group policies in the component updater.

R=sky@chromium.org
TBR=waffles@chromium.org

BUG=637452
==========

[Sorin Jianu, 2016-08-12 23:23:12]

sorin@chromium.org changed reviewers:
+ sky@chromium.org
- jam@chromium.org

[Sorin Jianu, 2016-08-12 23:24:18]

Sky, I need an owner's review for chrome_browser_impl.cc please.
It would be nice to land this soon, since it is a mechanical change that is
blocking other CLs.

Thank you!

[commit-bot: I haz the power, 2016-08-13 00:32:13]

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

[commit-bot: I haz the power, 2016-08-13 00:32:14]

Dry run: This issue passed the CQ dry run.

[sky, 2016-08-13 15:57:31]

browser_process_impl.cc LGTM, I did not look at the component_updater side.

[Sorin Jianu, 2016-08-15 02:32:34]

The CQ bit was checked by sorin@chromium.org

[commit-bot: I haz the power, 2016-08-15 02:32:46]

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-08-15 03:23:04]

Description was changed from

==========
Allow injecting a PrefService into ChromeComponentUpdaterConfigurator.

This is a mechanical change for the purpose of testing future changes
related to group policies in the component updater.

R=sky@chromium.org
TBR=waffles@chromium.org

BUG=637452
==========

to

==========
Allow injecting a PrefService into ChromeComponentUpdaterConfigurator.

This is a mechanical change for the purpose of testing future changes
related to group policies in the component updater.

R=sky@chromium.org
TBR=waffles@chromium.org

BUG=637452
==========

[commit-bot: I haz the power, 2016-08-15 03:23:05]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-15 03:26:52]

Description was changed from

==========
Allow injecting a PrefService into ChromeComponentUpdaterConfigurator.

This is a mechanical change for the purpose of testing future changes
related to group policies in the component updater.

R=sky@chromium.org
TBR=waffles@chromium.org

BUG=637452
==========

to

==========
Allow injecting a PrefService into ChromeComponentUpdaterConfigurator.

This is a mechanical change for the purpose of testing future changes
related to group policies in the component updater.

R=sky@chromium.org
TBR=waffles@chromium.org

BUG=637452

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

[commit-bot: I haz the power, 2016-08-15 03:26:53]

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

[waffles, 2016-08-16 19:57:32]

lgtm