Fork: Disable quic in Android WebView.

Fork: Disable quic in Android WebView.

This is a fork in m48 due to a refactor made after m48 branch.

BUG=566156
Review URL: https://codereview.chromium.org/1497113005

Cr-Commit-Position: refs/heads/master@{#363376}
(cherry picked from commit 292ab9f948fd6ff4e6f2d4583724fb7acee19ef0)

[hush (inactive), 2015-12-08 18:54:51]

Description was changed from

==========
Fork: Disable quic in Android WebView.

BUG=566156

Review URL: https://codereview.chromium.org/1497113005

Cr-Commit-Position: refs/heads/master@{#363376}
(cherry picked from commit 292ab9f948fd6ff4e6f2d4583724fb7acee19ef0)
==========

to

==========
Fork: Disable quic in Android WebView.

This is a fork in m48 due to a refactor made after m48 branch.

BUG=566156

Review URL: https://codereview.chromium.org/1497113005

Cr-Commit-Position: refs/heads/master@{#363376}
(cherry picked from commit 292ab9f948fd6ff4e6f2d4583724fb7acee19ef0)
==========

[hush (inactive), 2015-12-08 18:57:41]

hush@chromium.org changed reviewers:
+ sgurun@chromium.org

[hush (inactive), 2015-12-08 18:57:42]

Hi Selim,
PTAL.

This refactor https://codereview.chromium.org/1448193002 changed the code a lot,
so this is not a clean merge.

[hush (inactive), 2015-12-08 19:12:42]

https://codereview.chromium.org/1507343002/diff/1/android_webview/browser/net...
File android_webview/browser/net/aw_url_request_context_getter.cc (right):

https://codereview.chromium.org/1507343002/diff/1/android_webview/browser/net...
android_webview/browser/net/aw_url_request_context_getter.cc:242: false /*
set_up_quic_server_info */));
actually this does not seem to turn off quic. At least not explicitly. It just
turns off the ability to track information about servers supporting quic.
We should probably disable it somewhere else.

[hush (inactive), 2015-12-08 19:32:40]

On 2015/12/08 19:12:42, hush wrote:
>
https://codereview.chromium.org/1507343002/diff/1/android_webview/browser/net...
> File android_webview/browser/net/aw_url_request_context_getter.cc (right):
> 
>
https://codereview.chromium.org/1507343002/diff/1/android_webview/browser/net...
> android_webview/browser/net/aw_url_request_context_getter.cc:242: false /*
> set_up_quic_server_info */));
> actually this does not seem to turn off quic. At least not explicitly. It just
> turns off the ability to track information about servers supporting quic.
> We should probably disable it somewhere else.

Actually, are you sure WebView enabled quic in m48? I checkout m48, and logged
in URLRequestContextBuilder::Build. |enable_quic| was false there.

[hush (inactive), 2015-12-08 20:07:40]

Okay. Looks like we didn't enable quic in m48 after all. Closing it.