Chromium Code Reviews| Index: net/http/http_vary_data.cc |
| diff --git a/net/http/http_vary_data.cc b/net/http/http_vary_data.cc |
| index a8faf56ffdfb03d8578928597e2471828c0691cb..8f358485a1aba36c06997060ccf30e05b4f862fb 100644 |
| --- a/net/http/http_vary_data.cc |
| +++ b/net/http/http_vary_data.cc |
| @@ -68,9 +68,8 @@ bool HttpVaryData::MatchesRequest( |
| const HttpResponseHeaders& cached_response_headers) const { |
| HttpVaryData new_vary_data; |
| if (!new_vary_data.Init(request_info, cached_response_headers)) { |
| - // This shouldn't happen provided the same response headers passed here |
| - // were also used when initializing |this|. |
| - NOTREACHED(); |
| + // This case can happen if <this> was loaded from a cache that was populated |
|
gavinp
2016/05/23 11:37:35
Nit: |this|
|
| + // by a build before crbug.com/469675 was fixed. |
| return false; |
| } |
| return memcmp(&new_vary_data.request_digest_, &request_digest_, |