Add GetExpirationTimes() to HttpResponseHeader.

content/browser/appcache/appcache_update_job.cc

Index: content/browser/appcache/appcache_update_job.cc
diff --git a/content/browser/appcache/appcache_update_job.cc b/content/browser/appcache/appcache_update_job.cc
index de3e527ed23776f1825d8ced29895b5d6346cf88..a575869a5ed2cea32a51631e04d2cfd94fdcde3a 100644
--- a/content/browser/appcache/appcache_update_job.cc
+++ b/content/browser/appcache/appcache_update_job.cc
@@ -1472,10 +1472,11 @@ void AppCacheUpdateJob::OnResponseInfoLoaded(
     const std::string name = "vary";
     std::string value;
     size_t iter = 0;
+    net::HttpResponseHeaders::ExpirationTimes expiration_times =
+        http_info->headers->GetExpirationTimes(http_info->request_time,
+                                               http_info->response_time);
     if (!http_info->headers.get() ||
-        http_info->headers->RequiresValidation(http_info->request_time,

[gavinp, 2016/04/29 15:04:51]

Scary that this relied on the implicit conversion to bool from the enum.

-                                               http_info->response_time,
-                                               base::Time::Now()) ||
+        base::Time::Now() >= expiration_times.GetFreshnessExpiry() ||
         http_info->headers->EnumerateHeader(&iter, name, &value)) {
       LoadFromNewestCacheFailed(url, response_info);
     } else {