Add support to Google Update wrappers for the "automatic updates only" group policy setting.

Add support to Google Update wrappers for the "automatic updates only" group policy setting.

BUG=251972
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=207766

[Ryan Myers (chromium), 2013-06-20 01:47:30]

Hey folks -- the exact text of the string's still in UI review, but I'd
appreciate a look at the code changes.

grt: PTAL @ chrome\installer\util\google_update_settings.* and
chrome\browser\ui\webui\help\version_updater_win.cc
pkasting: PTAL @ chrome\browser\google\google_update_win.*

Thanks!

[Peter Kasting, 2013-06-20 17:55:19]

LGTM

https://codereview.chromium.org/17261014/diff/1/chrome/browser/google/google_...
File chrome/browser/google/google_update_win.cc (right):

https://codereview.chromium.org/17261014/diff/1/chrome/browser/google/google_...
chrome/browser/google/google_update_win.cc:63: if
(GoogleUpdateSettings::GetAppUpdatePolicy(app_guid, NULL) ==
Nit: Cache the result of this in a temp variable so the function only has to
call it once.

[Ryan Myers (chromium), 2013-06-20 19:57:11]

Thanks!

https://codereview.chromium.org/17261014/diff/1/chrome/browser/google/google_...
File chrome/browser/google/google_update_win.cc (right):

https://codereview.chromium.org/17261014/diff/1/chrome/browser/google/google_...
chrome/browser/google/google_update_win.cc:63: if
(GoogleUpdateSettings::GetAppUpdatePolicy(app_guid, NULL) ==
On 2013/06/20 17:55:19, Peter Kasting wrote:
> Nit: Cache the result of this in a temp variable so the function only has to
> call it once.

Done.

[grt (UTC plus 2), 2013-06-20 22:58:44]

lgtm

[commit-bot: I haz the power, 2013-06-20 23:01:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ryanmyers@chromium.org/17261014/9001

[commit-bot: I haz the power, 2013-06-21 09:41:21]

Change committed as 207766