QUIC - minor cleanup of the following field trials and used defaults for

QUIC - minor cleanup of the following field trials and used defaults for
them.

   "enable_truncated_connection_ids": "true",
   "load_server_info_time_to_srtt": "0.25"

Deleted "load_server_info_timeout" field trial, because
"load_server_info_time_to_srtt" field trial accomplishes the same in a
better manner.

R=rch@chromium.org

Committed: https://crrev.com/ca75eb1bbd333f49980b85a4a261999661db348f
Cr-Commit-Position: refs/heads/master@{#318957}

[ramant (doing other things), 2015-03-03 02:24:56]

rtenneti@chromium.org changed reviewers:
+ rch@chromium.org

[ramant (doing other things), 2015-03-03 02:25:32]

Hi Ryan,
  Changed the code to use defaults as we had talked offline. PTAL. thanks.

[Ryan Hamilton, 2015-03-03 05:11:13]

One minor suggestion

https://codereview.chromium.org/975613002/diff/1/chrome/browser/io_thread.cc
File chrome/browser/io_thread.cc (right):

https://codereview.chromium.org/975613002/diff/1/chrome/browser/io_thread.cc#...
chrome/browser/io_thread.cc:1345: return 0.25f;
I think that the default should probably live in //net somewhere. Perhaps
HttpNetworkSessionParams?

https://codereview.chromium.org/975613002/diff/1/net/http/http_network_sessio...
File net/http/http_network_session.cc (right):

https://codereview.chromium.org/975613002/diff/1/net/http/http_network_sessio...
net/http/http_network_session.cc:93:
quic_load_server_info_timeout_srtt_multiplier(0.0f),
If we think .25 is the "right" magic number, I'd put it here...

[ramant (doing other things), 2015-03-03 17:15:11]

https://codereview.chromium.org/975613002/diff/1/chrome/browser/io_thread.cc
File chrome/browser/io_thread.cc (right):

https://codereview.chromium.org/975613002/diff/1/chrome/browser/io_thread.cc#...
chrome/browser/io_thread.cc:1345: return 0.25f;
On 2015/03/03 05:11:13, Ryan Hamilton wrote:
> I think that the default should probably live in //net somewhere. Perhaps
> HttpNetworkSessionParams?

Done.

https://codereview.chromium.org/975613002/diff/1/net/http/http_network_sessio...
File net/http/http_network_session.cc (right):

https://codereview.chromium.org/975613002/diff/1/net/http/http_network_sessio...
net/http/http_network_session.cc:93:
quic_load_server_info_timeout_srtt_multiplier(0.0f),
On 2015/03/03 05:11:13, Ryan Hamilton wrote:
> If we think .25 is the "right" magic number, I'd put it here...

Done.

[Ryan Hamilton, 2015-03-03 21:56:20]

lgtm

[ramant (doing other things), 2015-03-03 22:01:00]

The CQ bit was checked by rtenneti@chromium.org

[commit-bot: I haz the power, 2015-03-03 22:01:41]

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

[commit-bot: I haz the power, 2015-03-03 23:09:59]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-03 23:11:11]

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