QUIC - Deleted disable_loading_server_info_for_new_servers field trial.

QUIC - Deleted disable_loading_server_info_for_new_servers field trial.
Changed the code not to load server config if there is no alternative
protocol entry for QUIC. During our experiments, we found that
disable_loading_server_info_for_new_servers being set to true has
reduced the latency until crypto handshake completion.

This is a revert of https://codereview.chromium.org/811073004/

We check for alternative protocols only if there is a http_server_properties_ object.

R=rch@chromium.org

Committed: https://crrev.com/6a454624b8f2827af3ec154bb96e61e7fdcf9e3a
Cr-Commit-Position: refs/heads/master@{#312741}

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

[ramant (doing other things), 2015-01-22 05:25:14]

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

[ramant (doing other things), 2015-01-22 05:25:14]

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

[ramant (doing other things), 2015-01-22 18:48:20]

Patchset #1 (id:1) has been deleted

[ramant (doing other things), 2015-01-22 18:50:13]

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

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

Hi Ryan,
  We have been running with flag enabled in M41-Dev (via field trial). As we had
talked, would like to turn this on by default. PTAL.

thanks
raman

[Ryan Hamilton, 2015-01-22 19:22:38]

lgtm

[ramant (doing other things), 2015-01-22 19:23:20]

The CQ bit was checked by rtenneti@chromium.org

[commit-bot: I haz the power, 2015-01-22 19:24:22]

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

[commit-bot: I haz the power, 2015-01-22 20:10:17]

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

[commit-bot: I haz the power, 2015-01-22 20:10:17]

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

[ramant (doing other things), 2015-01-22 20:29:38]

The CQ bit was checked by rtenneti@chromium.org

[commit-bot: I haz the power, 2015-01-22 20:30:44]

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

[commit-bot: I haz the power, 2015-01-22 23:51:28]

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

[commit-bot: I haz the power, 2015-01-22 23:51:29]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[ramant (doing other things), 2015-01-23 00:27:49]

The CQ bit was checked by rtenneti@chromium.org

[commit-bot: I haz the power, 2015-01-23 00:28:36]

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

[commit-bot: I haz the power, 2015-01-23 00:30:15]

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

[commit-bot: I haz the power, 2015-01-23 00:30:16]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[ramant (doing other things), 2015-01-23 01:13:02]

The CQ bit was checked by rtenneti@chromium.org

[commit-bot: I haz the power, 2015-01-23 01:15:00]

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

[commit-bot: I haz the power, 2015-01-23 01:54:25]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-01-23 01:55:38]

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

[benwells, 2015-01-23 03:04:50]

A revert of this CL (patchset #2 id:40001) has been created in
https://codereview.chromium.org/871833002/ by benwells@chromium.org.

The reason for reverting is: This caused a compile failure on iOS bots.

http://build.chromium.org/p/chromium.mac/builders/iOS%20Device/builds/21417

Error from log copied below:
Undefined symbols for architecture armv7:
  "net::QuicStreamFactory::QuicStreamFactory(net::HostResolver*,
net::ClientSocketFactory*, base::WeakPtr<net::HttpServerProperties>,
net::CertVerifier*, net::ChannelIDService*, net::TransportSecurityState*,
net::QuicCryptoClientStreamFactory*, net::QuicRandom*, net::QuicClock*, unsigned
long, std::__1::basic_string<char, std::__1::char_traits<char>,
std::__1::allocator<char> > const&, std::__1::vector<net::QuicVersion,
std::__1::allocator<net::QuicVersion> > const&, bool, bool, bool, int, bool,
float, bool, std::__1::vector<unsigned int, std::__1::allocator<unsigned int> >
const&)", referenced from:
      net::test::QuicStreamFactoryTest::QuicStreamFactoryTest() in
quic_stream_factory_test.o
ld: symbol(s) not found for architecture armv7
clang:error: linker command failed with exit code 1 (use -v to see invocation)
.

[ramant (doing other things), 2015-01-23 17:42:05]

On 2015/01/23 03:04:50, benwells wrote:
> A revert of this CL (patchset #2 id:40001) has been created in
> https://codereview.chromium.org/871833002/ by mailto:benwells@chromium.org.
> 
> The reason for reverting is: This caused a compile failure on iOS bots.
> 
> http://build.chromium.org/p/chromium.mac/builders/iOS%20Device/builds/21417
> 
> Error from log copied below:
> Undefined symbols for architecture armv7:
>   "net::QuicStreamFactory::QuicStreamFactory(net::HostResolver*,
> net::ClientSocketFactory*, base::WeakPtr<net::HttpServerProperties>,
> net::CertVerifier*, net::ChannelIDService*, net::TransportSecurityState*,
> net::QuicCryptoClientStreamFactory*, net::QuicRandom*, net::QuicClock*,
unsigned
> long, std::__1::basic_string<char, std::__1::char_traits<char>,
> std::__1::allocator<char> > const&, std::__1::vector<net::QuicVersion,
> std::__1::allocator<net::QuicVersion> > const&, bool, bool, bool, int, bool,
> float, bool, std::__1::vector<unsigned int, std::__1::allocator<unsigned int>
>
> const&)", referenced from:
>       net::test::QuicStreamFactoryTest::QuicStreamFactoryTest() in
> quic_stream_factory_test.o
> ld: symbol(s) not found for architecture armv7
> clang:error: linker command failed with exit code 1 (use -v to see invocation)
> .

The above error doesn't make sense. 

Change in
https://codereview.chromium.org/858323002/diff/40001/net/quic/quic_stream_fac...

has "....bool, bool, bool, int, float, bool,..."
but the above error line says "...bool, bool, bool, int, bool,float, bool,..."

Wondering if the bot had issues. This CL passed CQ and it builds on all
platforms. If the above error was true, it shouldn't have built on other
platforms also.

[ramant (doing other things), 2015-01-23 19:46:23]

The CQ bit was checked by rtenneti@chromium.org

[commit-bot: I haz the power, 2015-01-23 19:46:59]

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

[commit-bot: I haz the power, 2015-01-23 19:51:17]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2015-01-23 19:52:20]

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

[ramant (doing other things), 2015-01-23 20:09:52]

On 2015/01/23 19:52:20, I haz the power (commit-bot) wrote:
> Patchset 3 (id:??) landed as
> https://crrev.com/c7fe0a44d9af66aa3072df1460f6c85491f21e41
> Cr-Commit-Position: refs/heads/master@{#312904}

It compiled without issues today. Not sure why it failed before.

http://build.chromium.org/p/chromium.mac/builders/iOS%20Device/builds/21551