QUIC - Persist 1000 MRU alternate protocols to preferences file.

QUIC - Persist 1000 MRU alternate protocols to preferences file.

+ Flip a coin and if we have persisted 1000 MRU servers, then load only
  either 200 or 1000 servers from disk.

+ Persist only one server (or don't duplicate server names) that
  spoke QUIC for canonical suffixes like:
  + .c.youtube.com
  + .googlevideo.com

+ Bumped up the version number of http_server_properties.

+ Added two histograms to measure the impact of persisting 1000
  alternate protocol servers.
  ++ Net.AlternateProtocolUsage.200Truncated
  ++ Net.AlternateProtocolUsage.1000Truncated

R=jar@chromium.org, rch@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=272836

[jar (doing other things), 2014-05-24 00:19:03]

LGTM % the change in randomization

https://codereview.chromium.org/298683010/diff/20001/chrome/browser/net/http_...
File chrome/browser/net/http_server_properties_manager.cc (right):

https://codereview.chromium.org/298683010/diff/20001/chrome/browser/net/http_...
chrome/browser/net/http_server_properties_manager.cc:367: if (base::RandInt(0,
1) == 0) {
I was chatting with Ryan.... maybe we should make this RandInt(0, 99) == 0 so
that there is only a 1 in 100 chance of truncating folks.  The fear is that
truncating every other time will drive down the cache size. 

 .... although... once you get rid of saving canonicals... we *may* not need
much saved!!

[ramant (doing other things), 2014-05-24 00:34:06]

https://codereview.chromium.org/298683010/diff/20001/chrome/browser/net/http_...
File chrome/browser/net/http_server_properties_manager.cc (right):

https://codereview.chromium.org/298683010/diff/20001/chrome/browser/net/http_...
chrome/browser/net/http_server_properties_manager.cc:367: if (base::RandInt(0,
1) == 0) {
On 2014/05/24 00:19:03, jar wrote:
> I was chatting with Ryan.... maybe we should make this RandInt(0, 99) == 0 so
> that there is only a 1 in 100 chance of truncating folks.  The fear is that
> truncating every other time will drive down the cache size. 
> 
>  .... although... once you get rid of saving canonicals... we *may* not need
> much saved!!

Done.

[Ryan Hamilton, 2014-05-24 03:35:36]

lgtm

https://codereview.chromium.org/298683010/diff/60001/chrome/browser/net/http_...
File chrome/browser/net/http_server_properties_manager.cc (right):

https://codereview.chromium.org/298683010/diff/60001/chrome/browser/net/http_...
chrome/browser/net/http_server_properties_manager.cc:452: if (count <
alternate_protocols_to_load) {
perhaps:

if (count >= alternate_protocols_to_load)
  continue;

...

[ramant (doing other things), 2014-05-25 05:07:32]

https://codereview.chromium.org/298683010/diff/60001/chrome/browser/net/http_...
File chrome/browser/net/http_server_properties_manager.cc (right):

https://codereview.chromium.org/298683010/diff/60001/chrome/browser/net/http_...
chrome/browser/net/http_server_properties_manager.cc:452: if (count <
alternate_protocols_to_load) {
On 2014/05/24 03:35:36, Ryan Hamilton wrote:
> perhaps:
> 
> if (count >= alternate_protocols_to_load)
>   continue;
> 
> ...

Done.

[ramant (doing other things), 2014-05-25 05:07:44]

The CQ bit was checked by rtenneti@chromium.org

[commit-bot: I haz the power, 2014-05-25 05:07:57]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rtenneti@chromium.org/298683010/80001

[commit-bot: I haz the power, 2014-05-25 07:51:36]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  android_aosp on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/7...)
  win_chromium_x64_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_x64_rel/...)

[commit-bot: I haz the power, 2014-05-25 07:56:44]

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

[commit-bot: I haz the power, 2014-05-25 07:56:45]

Try jobs failed on following builders:
  android_aosp on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/7...)

[jar (doing other things), 2014-05-25 08:24:33]

On 2014/05/25 07:56:45, I haz the power (commit-bot) wrote:
> Try jobs failed on following builders:
>   android_aosp on tryserver.chromium
>
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/7...)

This failure on android_aosp seems unrelated to the patch, and is hitting a
completely different CL as well.

I've reported this problem to troupers.... but they may be on vacation this
weekend.

[ramant (doing other things), 2014-05-25 14:20:26]

The CQ bit was checked by rtenneti@chromium.org

[commit-bot: I haz the power, 2014-05-25 14:20:35]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rtenneti@chromium.org/298683010/80001

[commit-bot: I haz the power, 2014-05-25 14:25:25]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  android_aosp on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/7...)

[commit-bot: I haz the power, 2014-05-25 14:32:05]

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

[commit-bot: I haz the power, 2014-05-25 14:32:05]

Try jobs failed on following builders:
  android_aosp on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/7...)

[jar (doing other things), 2014-05-25 16:29:35]

The CQ bit was checked by jar@chromium.org

[commit-bot: I haz the power, 2014-05-25 16:29:48]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rtenneti@chromium.org/298683010/80001

[commit-bot: I haz the power, 2014-05-25 16:35:44]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  android_aosp on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/7...)

[commit-bot: I haz the power, 2014-05-25 16:40:41]

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

[commit-bot: I haz the power, 2014-05-25 16:40:41]

Try jobs failed on following builders:
  android_aosp on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/7...)

[ramant (doing other things), 2014-05-26 01:41:07]

The CQ bit was checked by rtenneti@chromium.org

[commit-bot: I haz the power, 2014-05-26 01:41:38]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rtenneti@chromium.org/298683010/80001

[commit-bot: I haz the power, 2014-05-26 16:06:52]

Change committed as 272836