[WIP] WebFonts cache-aware timeout adaption

third_party/WebKit/Source/platform/network/ResourceRequest.cpp

 /*
  * Copyright (C) 2003, 2006 Apple Computer, Inc.  All rights reserved.
  * Copyright (C) 2009, 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 70 matching lines @@
   setFetchRequestMode(data->m_fetchRequestMode);
   setFetchCredentialsMode(data->m_fetchCredentialsMode);
   setFetchRedirectMode(data->m_fetchRedirectMode);
   setLoFiState(data->m_loFiState);
   m_referrerPolicy = data->m_referrerPolicy;
   m_didSetHTTPReferrer = data->m_didSetHTTPReferrer;
   m_checkForBrowserSideNavigation = data->m_checkForBrowserSideNavigation;
   m_uiStartTime = data->m_uiStartTime;
   m_isExternalRequest = data->m_isExternalRequest;
   m_inputPerfMetricReportPolicy = data->m_inputPerfMetricReportPolicy;
+  m_isCacheAwareLoadingEnabled = data->m_isCacheAwareLoadingEnabled;
+  m_isCacheAwareLoadingActivated = data->m_isCacheAwareLoadingActivated;
+  m_savedCachePolicy = data->m_savedCachePolicy;
   m_redirectStatus = data->m_redirectStatus;
 }
 
 ResourceRequest::ResourceRequest(const ResourceRequest&) = default;
 
 ResourceRequest& ResourceRequest::operator=(const ResourceRequest&) = default;
 
 std::unique_ptr<CrossThreadResourceRequestData> ResourceRequest::copyData()
     const {
   std::unique_ptr<CrossThreadResourceRequestData> data =
@@ skipping 28 matching lines @@
   data->m_fetchRequestMode = m_fetchRequestMode;
   data->m_fetchCredentialsMode = m_fetchCredentialsMode;
   data->m_fetchRedirectMode = m_fetchRedirectMode;
   data->m_loFiState = m_loFiState;
   data->m_referrerPolicy = m_referrerPolicy;
   data->m_didSetHTTPReferrer = m_didSetHTTPReferrer;
   data->m_checkForBrowserSideNavigation = m_checkForBrowserSideNavigation;
   data->m_uiStartTime = m_uiStartTime;
   data->m_isExternalRequest = m_isExternalRequest;
   data->m_inputPerfMetricReportPolicy = m_inputPerfMetricReportPolicy;
+  data->m_isCacheAwareLoadingEnabled = m_isCacheAwareLoadingEnabled;
+  data->m_isCacheAwareLoadingActivated = m_isCacheAwareLoadingActivated;
+  data->m_savedCachePolicy = m_savedCachePolicy;
   data->m_redirectStatus = m_redirectStatus;
   return data;
 }
 
 bool ResourceRequest::isEmpty() const {
   return m_url.isEmpty();
 }
 
 bool ResourceRequest::isNull() const {
   return m_url.isNull();
@@ skipping 231 matching lines @@
 
 bool ResourceRequest::cacheControlContainsNoStore() const {
   return cacheControlHeader().containsNoStore;
 }
 
 bool ResourceRequest::hasCacheValidatorFields() const {
   return !m_httpHeaderFields.get(HTTPNames::Last_Modified).isEmpty() ||
          !m_httpHeaderFields.get(HTTPNames::ETag).isEmpty();
 }
 
+void ResourceRequest::mayActivateCacheAwareLoading() {
+  if (!m_isCacheAwareLoadingEnabled)
+    return;
+
+  if (m_cachePolicy == WebCachePolicy::BypassingCache ||

[Takashi Toyoshima, 2016/10/03 09:06:35]

Using switch is better to catch new cache policy being added?
We would handle the new policy in a right way when added.

[Shao-Chuan Lee, 2016/10/04 09:08:04]

Done.

+      m_cachePolicy == WebCachePolicy::ReturnCacheDataDontLoad)
+    return;
+
+  m_savedCachePolicy = m_cachePolicy;
+  m_cachePolicy = WebCachePolicy::ReturnCacheDataDontLoad;

[Takashi Toyoshima, 2016/10/03 09:06:35]

Should we have a TODO to use the right flag when //net is ready?
Also it worth noting that this flag may return stale data until then.

[Shao-Chuan Lee, 2016/10/04 01:39:46]

Do we already have a bug for this? Or we should file one for tracking progress.

[Takashi Toyoshima, 2016/10/04 04:12:02]

How about using crbug.com/570205 for this TODO.

[Shao-Chuan Lee, 2016/10/04 09:08:04]

Done.

[Shao-Chuan Lee, 2016/10/05 10:16:09]

Now crbug.com/652649 is available for tracking.

+  m_isCacheAwareLoadingActivated = true;
+}
+
+void ResourceRequest::deactivateCacheAwareLoading() {
+  DCHECK(m_isCacheAwareLoadingActivated);
+  m_cachePolicy = m_savedCachePolicy;
+  m_isCacheAwareLoadingActivated = false;
+}
+
 void ResourceRequest::initialize(const KURL& url) {
   m_url = url;
   m_cachePolicy = WebCachePolicy::UseProtocolCachePolicy;
   m_timeoutInterval = s_defaultTimeoutInterval;
   m_httpMethod = HTTPNames::GET;
   m_allowStoredCredentials = true;
   m_reportUploadProgress = false;
   m_reportRawHeaders = false;
   m_hasUserGesture = false;
   m_downloadToFile = false;
@@ skipping 12 matching lines @@
   m_fetchRedirectMode = WebURLRequest::FetchRedirectModeFollow;
   m_referrerPolicy = ReferrerPolicyDefault;
   m_loFiState = WebURLRequest::LoFiUnspecified;
   m_didSetHTTPReferrer = false;
   m_checkForBrowserSideNavigation = true;
   m_uiStartTime = 0;
   m_isExternalRequest = false;
   m_inputPerfMetricReportPolicy = InputToLoadPerfMetricReportPolicy::NoReport;
   m_redirectStatus = RedirectStatus::NoRedirect;
   m_requestorOrigin = SecurityOrigin::createUnique();
+  m_isCacheAwareLoadingEnabled = false;
+  m_isCacheAwareLoadingActivated = false;
+  m_savedCachePolicy = WebCachePolicy::UseProtocolCachePolicy;
 }
 
 }  // namespace blink