Refactor alternate protocol dictionary parsing.

Refactor alternate protocol dictionary parsing.

This CL breaks out into its own function the parsing of a DictionaryValue to
AlternateProtocolInfo.  This is only called once now, but as soon as we start
storing multiple entries per server, we'll need to call it multiple times.  See
https://codereview.chromium.org/665083009/#ps460001.

BUG=392575
Committed: https://crrev.com/54e76ef7eff2dae72dbac1c0f7c39aad467a9838
Cr-Commit-Position: refs/heads/master@{#314890}

[Bence, 2015-02-05 18:53:35]

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

[Bence, 2015-02-05 18:53:35]

Ryan, PTAL.  Thanks.

[Ryan Hamilton, 2015-02-05 19:11:15]

lgtm

looks good, but it needs a rebase.

https://codereview.chromium.org/897313002/diff/1/net/http/http_server_propert...
File net/http/http_server_properties_manager.cc (right):

https://codereview.chromium.org/897313002/diff/1/net/http/http_server_propert...
net/http/http_server_properties_manager.cc:458: if
(!alternate_protocol_dict.GetInteger("port", &port) ||
nit: I think you might need to rebase since I extracted all of these "magic
strings" into variables like kPortKey.

[Bence, 2015-02-05 19:58:54]

New patchsets have been uploaded after l-g-t-m from rch@chromium.org

[Ryan Hamilton, 2015-02-05 20:02:31]

lgtm

[Ryan Hamilton, 2015-02-05 20:02:32]

lgtm

lgtm

[Bence, 2015-02-05 21:01:49]

The CQ bit was checked by bnc@chromium.org

[commit-bot: I haz the power, 2015-02-05 21:02:45]

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

[commit-bot: I haz the power, 2015-02-05 21:58:26]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-05 22:00:14]

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