Detect whether a PAC url is present on Android

Detect whether a PAC url is present on Android

This CL detects whether a PAC url is set in Android's system settings.
If there is a PAC url, use that PAC url to construct a ProxyConfig.

BUG=372475
Committed: https://crrev.com/b50a78b4dabfffa36b1d847167458d4a961c29ca
Cr-Commit-Position: refs/heads/master@{#295489}

[xunjieli, 2014-09-16 18:10:20]

xunjieli@chromium.org changed reviewers:
+ eroman@chromium.org

[xunjieli, 2014-09-16 18:10:20]

Hi Eric,

I have divided up my previous CL. This new one only contains proxy config
service changes. Could you take a look?

Thanks!

[eroman, 2014-09-18 15:55:21]

lgtm

https://codereview.chromium.org/573013002/diff/1/net/proxy/proxy_config_servi...
File net/proxy/proxy_config_service_android.h (right):

https://codereview.chromium.org/573013002/diff/1/net/proxy/proxy_config_servi...
net/proxy/proxy_config_service_android.h:44: // The third argument indicates the
pac url.
nit: pac --> PAC

[xunjieli, 2014-09-18 16:18:18]

Thanks Eric for the review!

https://codereview.chromium.org/573013002/diff/1/net/proxy/proxy_config_servi...
File net/proxy/proxy_config_service_android.h (right):

https://codereview.chromium.org/573013002/diff/1/net/proxy/proxy_config_servi...
net/proxy/proxy_config_service_android.h:44: // The third argument indicates the
pac url.
On 2014/09/18 15:55:20, eroman wrote:
> nit: pac --> PAC

Done.

[xunjieli, 2014-09-18 16:18:26]

The CQ bit was checked by xunjieli@chromium.org

[commit-bot: I haz the power, 2014-09-18 16:19:16]

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

[commit-bot: I haz the power, 2014-09-18 17:34:05]

Committed patchset #2 (id:20001) as 2a27a5906b85ef5d66b54cdbd370e98ae9572027

[commit-bot: I haz the power, 2014-09-18 17:34:36]

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