| commit | bd97ed30a82143f7e276463468445fe473b1d199 | |
| author | bnc <bnc@chromium.org> | |
| Tue, 4 Aug 2015 10:41:53 +0000 (4 03:41 -0700) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Tue, 4 Aug 2015 10:42:39 +0000 (4 10:42 +0000) | ||
| tree | 95ce30ee1b09c32399e4848f02f0f6fec6f04a74 | treesnapshot (tar.gz zip) |
| parent | f4fae706e1c424f19ee1bee33428f0b846354d1a | commitdiff |
Remove empty alternative service vector from |alternative_service_map_|.
Remove empty alternative service vector from |alternative_service_map_| in
HttpServerPropertiesImpl::GetAlternativeServices().
The reporter of the bug confirmed that Patch Set 1 fixes the crash on their
local build.
The author of this CL confirmed that the regression test added in Patch Set 2
run without the fix in Patch Set 1 does crash on a local build.
Note that this is only half of the bug: the same problem still persists for
canonical hosts. That deserves another regression test and another CL.
BUG=516486
Review URL: https://codereview.chromium.org/1256113005
Cr-Commit-Position: refs/heads/master@{#341708}
Remove empty alternative service vector from |alternative_service_map_| in
HttpServerPropertiesImpl::GetAlternativeServices().
The reporter of the bug confirmed that Patch Set 1 fixes the crash on their
local build.
The author of this CL confirmed that the regression test added in Patch Set 2
run without the fix in Patch Set 1 does crash on a local build.
Note that this is only half of the bug: the same problem still persists for
canonical hosts. That deserves another regression test and another CL.
BUG=516486
Review URL: https://codereview.chromium.org/1256113005
Cr-Commit-Position: refs/heads/master@{#341708}
| net/http/http_server_properties_impl.cc | diffblobblamehistory | |
| net/http/http_server_properties_impl_unittest.cc | diffblobblamehistory |