Implement CUP signing in UpdateClient.

Implement CUP signing in UpdateClient.

BUG=583027
Committed: https://crrev.com/1bc5eff904e6e20191a0abc2ab7a4c75c31dcc1f
Cr-Commit-Position: refs/heads/master@{#375930}

[Sorin Jianu, 2016-02-10 23:51:45]

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

[waffles, 2016-02-11 00:06:14]

lgtm

[Sorin Jianu, 2016-02-11 19:00:21]

Description was changed from

==========
Implement CUP signing in UpdateClient.

BUG=583027
==========

to

==========
Implement CUP signing in UpdateClient.

BUG=583027
==========

[Sorin Jianu, 2016-02-11 19:00:21]

sorin@chromium.org changed reviewers:
+ asargent@chromium.org, blundell@chromium.org

[Sorin Jianu, 2016-02-11 19:01:24]

Colin, Antony, please, I need owners approval for the configurator paths under
the ios and extensions please.

Thank you!

[asargent_no_longer_on_chrome, 2016-02-11 19:36:52]

extensions changes lgtm

https://codereview.chromium.org/1685323002/diff/1/chrome/browser/extensions/u...
File chrome/browser/extensions/updater/chrome_update_client_config.cc (right):

https://codereview.chromium.org/1685323002/diff/1/chrome/browser/extensions/u...
chrome/browser/extensions/updater/chrome_update_client_config.cc:91: return
false;
Do we eventually want to use cup signing for extensions updates?

[Sorin Jianu, 2016-02-12 00:43:09]

Thank you Antony!

https://codereview.chromium.org/1685323002/diff/1/chrome/browser/extensions/u...
File chrome/browser/extensions/updater/chrome_update_client_config.cc (right):

https://codereview.chromium.org/1685323002/diff/1/chrome/browser/extensions/u...
chrome/browser/extensions/updater/chrome_update_client_config.cc:91: return
false;
On 2016/02/11 19:36:52, Antony Sargent wrote:
> Do we eventually want to use cup signing for extensions updates?

Yes. We want to converge these updaters. I will go back to work on this problem.

[blundell, 2016-02-12 08:43:46]

blundell@chromium.org changed reviewers:
+ droger@chromium.org
- blundell@chromium.org

[blundell, 2016-02-12 08:43:47]

blundell -> droger

I'm not an //ios OWNER

[droger, 2016-02-12 08:55:48]

ios LGTM

[Sorin Jianu, 2016-02-17 17:10:42]

The CQ bit was checked by sorin@chromium.org

[commit-bot: I haz the power, 2016-02-17 17:12:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1685323002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1685323002/1

[Sorin Jianu, 2016-02-17 17:28:38]

The CQ bit was unchecked by sorin@chromium.org

[Sorin Jianu, 2016-02-17 17:54:27]

The CQ bit was checked by sorin@chromium.org

[Sorin Jianu, 2016-02-17 17:54:28]

The patchset sent to the CQ was uploaded after l-g-t-m from
waffles@chromium.org, droger@chromium.org, asargent@chromium.org
Link to the patchset: https://codereview.chromium.org/1685323002/#ps20001
(title: "rebase and remove duplicated code comment.")

[commit-bot: I haz the power, 2016-02-17 17:55:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1685323002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1685323002/20001

[commit-bot: I haz the power, 2016-02-17 18:45:31]

Description was changed from

==========
Implement CUP signing in UpdateClient.

BUG=583027
==========

to

==========
Implement CUP signing in UpdateClient.

BUG=583027
==========

[commit-bot: I haz the power, 2016-02-17 18:45:32]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-17 18:47:01]

Description was changed from

==========
Implement CUP signing in UpdateClient.

BUG=583027
==========

to

==========
Implement CUP signing in UpdateClient.

BUG=583027

Committed: https://crrev.com/1bc5eff904e6e20191a0abc2ab7a4c75c31dcc1f
Cr-Commit-Position: refs/heads/master@{#375930}
==========

[commit-bot: I haz the power, 2016-02-17 18:47:02]

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