Persist the alternate protocol as a string not an int to allow the enum to be changed without affec…

Persist the alternate protocol as a string not an int to allow the enum to be changed without affecting persisted values.

BUG=121967
TEST=HttpServerPropertiesManagerTest.*


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=131601

[ramant (doing other things), 2012-04-07 00:41:22]

http://codereview.chromium.org/10006047/diff/1005/net/http/http_server_proper...
File net/http/http_server_properties.cc (right):

http://codereview.chromium.org/10006047/diff/1005/net/http/http_server_proper...
net/http/http_server_properties.cc:39: for (int i = NPN_SPDY_1; i < NPN_SPDY_3;
i++)
nit: i++ -> ++i

Should we support "npn-spdy/3"?

[Ryan Hamilton, 2012-04-09 16:37:20]

On 2012/04/07 00:41:22, ramant wrote:
>
http://codereview.chromium.org/10006047/diff/1005/net/http/http_server_proper...
> File net/http/http_server_properties.cc (right):
> 
>
http://codereview.chromium.org/10006047/diff/1005/net/http/http_server_proper...
> net/http/http_server_properties.cc:39: for (int i = NPN_SPDY_1; i <
NPN_SPDY_3;
> i++)
> nit: i++ -> ++i

Done!

> Should we support "npn-spdy/3"?

Done.  (That was a typo on my part.)

[ramant (doing other things), 2012-04-09 16:50:19]

lgtm

[commit-bot: I haz the power, 2012-04-09 18:18:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rch@chromium.org/10006047/5001

[commit-bot: I haz the power, 2012-04-09 19:24:57]

Try job failure for 10006047-5001 (retry) on win for step "compile" (clobber
build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win&number...

[commit-bot: I haz the power, 2012-04-09 20:52:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rch@chromium.org/10006047/14001

[commit-bot: I haz the power, 2012-04-09 21:56:39]

Try job failure for 10006047-14001 (retry) on win for step "compile" (clobber
build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win&number...

[commit-bot: I haz the power, 2012-04-10 16:52:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rch@chromium.org/10006047/16001

[commit-bot: I haz the power, 2012-04-10 19:11:40]

Change committed as 131601