chromeos: Remove LibCrosService's ResolveNetworkProxy method

chromeos: Remove LibCrosService's ResolveNetworkProxy method

Remove the ResolveNetworkProxy D-Bus method exported by
org.chromium.LibCrosService, along with support for
requesting that the response be sent via a signal (as
opposed to an async method call).

This method has been renamed to ResolveProxy and lives in a
new org.chromium.NetworkProxyService service. All Chrome OS
callers (tlsdated, crash-reporter, update_engine) have been
updated to use the new service.

BUG=446115, 703217
Review-Url: https://codereview.chromium.org/2880083002
Cr-Commit-Position: refs/heads/master@{#479509}
Committed: https://chromium.googlesource.com/chromium/src/+/67fdc5416dcf8317fe5b43f6afd999100e540c9d

[Daniel Erat, 2017-05-13 15:40:24]

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

[commit-bot: I haz the power, 2017-05-13 15:40:38]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Daniel Erat, 2017-05-13 15:41:06]

derat@chromium.org changed reviewers:
+ teravest@chromium.org

[Daniel Erat, 2017-05-13 15:41:07]

This needs to wait until the 22nd or so to go in so that Chrome OS UI developers
can update their system images.

[commit-bot: I haz the power, 2017-05-13 16:12:13]

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

[commit-bot: I haz the power, 2017-05-13 16:12:14]

Dry run: This issue passed the CQ dry run.

[teravest, 2017-05-15 14:12:54]

lgtm

May as well add 446115 to the BUG line in the description too.

[Daniel Erat, 2017-05-15 14:14:55]

Description was changed from

==========
chromeos: Remove LibCrosService's ResolveNetworkProxy method

Remove the ResolveNetworkProxy D-Bus method exported by
org.chromium.LibCrosService, along with support for
requesting that the response be sent via a signal (as
opposed to an async method call).

This method has been renamed to ResolveProxy and lives in a
new org.chromium.NetworkProxyService service. All Chrome OS
callers (tlsdated, crash-reporter, update_engine) have been
updated to use the new service.

BUG=703217
==========

to

==========
chromeos: Remove LibCrosService's ResolveNetworkProxy method

Remove the ResolveNetworkProxy D-Bus method exported by
org.chromium.LibCrosService, along with support for
requesting that the response be sent via a signal (as
opposed to an async method call).

This method has been renamed to ResolveProxy and lives in a
new org.chromium.NetworkProxyService service. All Chrome OS
callers (tlsdated, crash-reporter, update_engine) have been
updated to use the new service.

BUG=446115,703217
==========

[Daniel Erat, 2017-05-15 14:15:09]

On 2017/05/15 14:12:54, teravest wrote:
> lgtm
> 
> May as well add 446115 to the BUG line in the description too.

thanks, done.

[Daniel Erat, 2017-06-14 20:43:42]

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

[commit-bot: I haz the power, 2017-06-14 20:44:04]

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-06-14 21:26:47]

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

[commit-bot: I haz the power, 2017-06-14 21:26:48]

Dry run: This issue passed the CQ dry run.

[Daniel Erat, 2017-06-14 21:29:30]

The CQ bit was checked by derat@chromium.org

[commit-bot: I haz the power, 2017-06-14 21:29:57]

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-06-14 21:35:07]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1497475770861750, "parent_rev":
"1962064cfb840f9c85ba111bf887e1a9341e4947", "commit_rev":
"67fdc5416dcf8317fe5b43f6afd999100e540c9d"}

[commit-bot: I haz the power, 2017-06-14 21:35:18]

Description was changed from

==========
chromeos: Remove LibCrosService's ResolveNetworkProxy method

Remove the ResolveNetworkProxy D-Bus method exported by
org.chromium.LibCrosService, along with support for
requesting that the response be sent via a signal (as
opposed to an async method call).

This method has been renamed to ResolveProxy and lives in a
new org.chromium.NetworkProxyService service. All Chrome OS
callers (tlsdated, crash-reporter, update_engine) have been
updated to use the new service.

BUG=446115,703217
==========

to

==========
chromeos: Remove LibCrosService's ResolveNetworkProxy method

Remove the ResolveNetworkProxy D-Bus method exported by
org.chromium.LibCrosService, along with support for
requesting that the response be sent via a signal (as
opposed to an async method call).

This method has been renamed to ResolveProxy and lives in a
new org.chromium.NetworkProxyService service. All Chrome OS
callers (tlsdated, crash-reporter, update_engine) have been
updated to use the new service.

BUG=446115,703217

Review-Url: https://codereview.chromium.org/2880083002
Cr-Commit-Position: refs/heads/master@{#479509}
Committed:
https://chromium.googlesource.com/chromium/src/+/67fdc5416dcf8317fe5b43f6afd9...
==========

[commit-bot: I haz the power, 2017-06-14 21:35:21]

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