Observe v= parameter when processing Alternative Services header.

Observe v= parameter when processing Alternative Services header.

BUG=539347
Committed: https://crrev.com/8be55ebb1121fd7fc77791a55ed1fa5c90cb1f66
Cr-Commit-Position: refs/heads/master@{#357094}

[Bence, 2015-10-29 18:31:20]

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

[Bence, 2015-10-29 18:31:21]

Ryan: PTAL.  Thank you.

[Ryan Hamilton, 2015-10-29 20:07:23]

lgtm

https://codereview.chromium.org/1406923010/diff/1/net/http/http_stream_factor...
File net/http/http_stream_factory.cc (right):

https://codereview.chromium.org/1406923010/diff/1/net/http/http_stream_factor...
net/http/http_stream_factory.cc:67: match_found = true;
Up to you, but instead of using std::find, I wonder if would be easier to do:
  for (uint16 alt_version : alternative_service_entry) {
    if (supported == alt_version) {
      match_found = true;
      break;
    }
  }

I hate iterators, but I have lots of pet peeves :> You don't need to share my
peeves :>

[Bence, 2015-10-30 14:02:16]

The CQ bit was checked by bnc@chromium.org

[Bence, 2015-10-30 14:02:16]

The patchset sent to the CQ was uploaded after l-g-t-m from rch@chromium.org
Link to the patchset: https://codereview.chromium.org/1406923010/#ps20001
(title: "Re: #3.")

[commit-bot: I haz the power, 2015-10-30 14:03:59]

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

[commit-bot: I haz the power, 2015-10-30 14:12:17]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-10-30 14:13:18]

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