Do not unbreak alternative service in ClearAlternativeService.
commit8784b360bebebbe19e362cf487457513eb4eadb7
authorbnc <bnc@chromium.org>
Wed, 24 Jun 2015 11:48:39 +0000 (24 04:48 -0700)
committerCommit bot <commit-bot@chromium.org>
Wed, 24 Jun 2015 11:49:03 +0000 (24 11:49 +0000)
tree209760938481c98bfa7fd5cf9b00d162d47dba2a
parentc4dc6eae784ef7f6878b669d8101ca4e4b2d895f
Do not unbreak alternative service in ClearAlternativeService.

Do not unbreak alternative service in
HttpServerProperties::ClearAlternativeService().

* This is wrong, because only because an alternative service is not mapped to
  one origin any longer, if it was broken, it remains broken and should not be
  used for other origins.
* QuicStreamFactory::ProcessGoingAwaySession() used to call
  ClearAlternativeService(), and right after that, SetAlternativeService, to
  mark an alternative service recently broken.  Now QuicStreamFactory uses
  MarkAlternativeServiceRecentlyBroken calls instead, so this behavior in
  ClearAlternativeService is not required any more.
* Note that despite what the TODO comment says, broken alternative services
  should remain in the mapping (until explicitly cleared by
  ClearAlternativeService).

BUG=392575

Review URL: https://codereview.chromium.org/1198363003

Cr-Commit-Position: refs/heads/master@{#335907}
net/http/http_server_properties_impl.cc
net/http/http_server_properties_impl_unittest.cc