Uses ProxyConfigServiceLinux in the chromoting host to configure proxy.

Use ProxyConfigServiceLinux in the chromoting host to configure proxy.

Previously chromoting host wasn't using proxies on linux because
ProxyConfigServiceLinux doesn't work in NPAPI plugins. Now it's not
a problem because we no longer support NPAPI.

BUG=410521, 125104
R=jamiewalch@chromium.org, mmenke@chromium.org

Committed: https://chromium.googlesource.com/chromium/src/+/2eea434809cdc785f80a1281bea5e634a2cb474c

[Sergey Ulanov, 2014-09-03 22:51:06]

sergeyu@chromium.org changed reviewers:
+ jamiewalch@chromium.org, mmenke@chromium.org

[Sergey Ulanov, 2014-09-03 22:51:07]

This CL depends on https://codereview.chromium.org/540593002/ .

mmenke - please approve net/*
jamiewalch - please review

[mmenke, 2014-09-04 16:17:40]

LGTM

https://codereview.chromium.org/542473002/diff/20001/remoting/base/url_reques...
File remoting/base/url_request_context_getter.cc (left):

https://codereview.chromium.org/542473002/diff/20001/remoting/base/url_reques...
remoting/base/url_request_context_getter.cc:42: return new
net::ProxyConfigServiceWin();
So this was relying on NET_EXPORT_PRIVATE not actually being enforceable?

[Sergey Ulanov, 2014-09-05 00:14:18]

On 2014/09/04 16:17:40, mmenke wrote:
> LGTM
> 
>
https://codereview.chromium.org/542473002/diff/20001/remoting/base/url_reques...
> File remoting/base/url_request_context_getter.cc (left):
> 
>
https://codereview.chromium.org/542473002/diff/20001/remoting/base/url_reques...
> remoting/base/url_request_context_getter.cc:42: return new
> net::ProxyConfigServiceWin();
> So this was relying on NET_EXPORT_PRIVATE not actually being enforceable?

yes

[Sergey Ulanov, 2014-09-05 00:15:07]

Jamie: ping

[Jamie, 2014-09-05 00:21:56]

I'm not sure how much value I can add to this review, since I don't know this
code at all well. Was the something in particular you wanted me to look at? If
it's just an OWNERS rubber-stamp, then LGTM.

[Sergey Ulanov, 2014-09-05 00:32:05]

On 2014/09/05 00:21:56, Jamie wrote:
> I'm not sure how much value I can add to this review, since I don't know this
> code at all well. Was the something in particular you wanted me to look at? If
> it's just an OWNERS rubber-stamp, then LGTM.

Thanks!
Just wanted somebody from chromoting team to take a look. Would normally send it
to Renato, but he is OOO.

[Sergey Ulanov, 2014-09-05 17:01:07]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2014-09-05 17:01:53]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sergeyu@chromium.org/542473002/20001

[commit-bot: I haz the power, 2014-09-05 17:20:01]

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

[commit-bot: I haz the power, 2014-09-05 17:20:03]

Try jobs failed on following builders:
  linux_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu/builds/...)
  linux_chromium_gn_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Sergey Ulanov, 2014-09-05 18:45:38]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2014-09-05 18:47:00]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sergeyu@chromium.org/542473002/20001

[commit-bot: I haz the power, 2014-09-05 19:03:44]

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

[commit-bot: I haz the power, 2014-09-05 19:03:45]

Try jobs failed on following builders:
  android_dbg_tests_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg_tes...)

[Sergey Ulanov, 2014-09-05 19:39:31]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2014-09-05 19:40:38]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sergeyu@chromium.org/542473002/40001

[commit-bot: I haz the power, 2014-09-06 01:41:19]

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

[commit-bot: I haz the power, 2014-09-06 01:41:20]

Exceeded time limit waiting for builds to trigger.

[Sergey Ulanov, 2014-09-06 02:23:54]

Committed patchset #3 (id:40001) manually as 2eea434 (presubmit successful).

[commit-bot: I haz the power, 2014-09-10 03:42:11]

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