Add a callback parameter to UpdateClient::SendUninstallPing.

Add a callback parameter to UpdateClient::SendUninstallPing.

This is a mechanical change.

This refactoring is needed for future work and it resolves an
issue with the UpdateClientTest::SendUninstallPing, where the test
is missing the RunThreads() call at the end.

This change makes the UpdateClient API consistent, since all
non-blocking APIs now take a callback parameter.

Review-Url: https://codereview.chromium.org/2831473002
Cr-Commit-Position: refs/heads/master@{#465632}
Committed: https://chromium.googlesource.com/chromium/src/+/8037ac8c239b75a76ba3392dcbae14bbb046c080

[Sorin Jianu, 2017-04-19 02:05:26]

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

[commit-bot: I haz the power, 2017-04-19 02:05:47]

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-04-19 02:51:02]

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

[commit-bot: I haz the power, 2017-04-19 02:51:03]

Dry run: This issue passed the CQ dry run.

[Sorin Jianu, 2017-04-19 16:04:41]

sorin@chromium.org changed reviewers:
+ rdevlin.cronin@chromium.org

[Sorin Jianu, 2017-04-19 16:04:42]

Hi Devlin, PTAL!

Thank you!

[Devlin, 2017-04-19 16:16:41]

extensions lgtm

[Sorin Jianu, 2017-04-19 16:21:11]

Description was changed from

==========
Add a callback parameter to UpdateClient::SendUninstallPing.

This is a mechanical change.

This refactoring is needed for future work and it resolves an
issue with the UpdateClientTest::SendUninstallPing, where the test
is missing the RunThreads() call at the end.

This change makes the UpdateClient API consistent, since all
non-blocking APIs now take a callback parameter.
==========

to

==========
Add a callback parameter to UpdateClient::SendUninstallPing.

This is a mechanical change.

This refactoring is needed for future work and it resolves an
issue with the UpdateClientTest::SendUninstallPing, where the test
is missing the RunThreads() call at the end.

This change makes the UpdateClient API consistent, since all
non-blocking APIs now take a callback parameter.
==========

[Sorin Jianu, 2017-04-19 16:21:11]

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

[Sorin Jianu, 2017-04-19 16:21:22]

The CQ bit was checked by sorin@chromium.org

[commit-bot: I haz the power, 2017-04-19 16:21:54]

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-04-19 16:28:04]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492618882507280, "parent_rev":
"f8c258e4a20a601d80ac97a7d61c5c225288fcf9", "commit_rev":
"d176f70e97fa79c9fc5e6d92f865db243278e93f"}

[commit-bot: I haz the power, 2017-04-19 16:28:20]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492618882507280, "parent_rev":
"2b04a50fd504ab457afd30899cefea3cfd05582d", "commit_rev":
"8037ac8c239b75a76ba3392dcbae14bbb046c080"}

[commit-bot: I haz the power, 2017-04-19 16:28:32]

Description was changed from

==========
Add a callback parameter to UpdateClient::SendUninstallPing.

This is a mechanical change.

This refactoring is needed for future work and it resolves an
issue with the UpdateClientTest::SendUninstallPing, where the test
is missing the RunThreads() call at the end.

This change makes the UpdateClient API consistent, since all
non-blocking APIs now take a callback parameter.
==========

to

==========
Add a callback parameter to UpdateClient::SendUninstallPing.

This is a mechanical change.

This refactoring is needed for future work and it resolves an
issue with the UpdateClientTest::SendUninstallPing, where the test
is missing the RunThreads() call at the end.

This change makes the UpdateClient API consistent, since all
non-blocking APIs now take a callback parameter.

Review-Url: https://codereview.chromium.org/2831473002
Cr-Commit-Position: refs/heads/master@{#465632}
Committed:
https://chromium.googlesource.com/chromium/src/+/8037ac8c239b75a76ba3392dcbae...
==========

[commit-bot: I haz the power, 2017-04-19 16:28:36]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/8037ac8c239b75a76ba3392dcbae...