Handle the case when the updates are disabled for a component.

Handle the case when the updates are disabled for a component.

This change does not include the group policy changes.

BUG=624128
Committed: https://crrev.com/e84ff700af9117c3be17a74d90b04be03dc4a689
Cr-Commit-Position: refs/heads/master@{#409693}

[Sorin Jianu, 2016-08-03 23:38:17]

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

[commit-bot: I haz the power, 2016-08-03 23:38:53]

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-03 23:39:27]

Description was changed from

==========
Handle the case when the updates are disabled for a component.

This change does not include the group policy changes.

BUG=624128
==========

to

==========
Handle the case when the updates are disabled for a component.

This change does not include the group policy changes.

BUG=624128
==========

[Sorin Jianu, 2016-08-03 23:39:28]

sorin@chromium.org changed reviewers:
+ waffles@chromium.org

[waffles, 2016-08-03 23:49:02]

lgtm modulo quoting issue

https://codereview.chromium.org/2206583007/diff/1/components/update_client/up...
File components/update_client/update_checker.cc (right):

https://codereview.chromium.org/2206583007/diff/1/components/update_client/up...
components/update_client/update_checker.cc:98: base::StringAppendF(&app, "
\"updatedisabled\"=true");
I think you mean " updatedisabled=\"true\"". (Also in the tests.)

[Sorin Jianu, 2016-08-04 00:11:56]

The CQ bit was checked by sorin@chromium.org

[Sorin Jianu, 2016-08-04 00:11:57]

The patchset sent to the CQ was uploaded after l-g-t-m from waffles@chromium.org
Link to the patchset: https://codereview.chromium.org/2206583007/#ps20001
(title: ".")

[Sorin Jianu, 2016-08-04 00:12:09]

done, ty!

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

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-04 01:22:18]

Description was changed from

==========
Handle the case when the updates are disabled for a component.

This change does not include the group policy changes.

BUG=624128
==========

to

==========
Handle the case when the updates are disabled for a component.

This change does not include the group policy changes.

BUG=624128
==========

[commit-bot: I haz the power, 2016-08-04 01:22:19]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-04 01:25:06]

Description was changed from

==========
Handle the case when the updates are disabled for a component.

This change does not include the group policy changes.

BUG=624128
==========

to

==========
Handle the case when the updates are disabled for a component.

This change does not include the group policy changes.

BUG=624128

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

[commit-bot: I haz the power, 2016-08-04 01:25:07]

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