[WIP] WebFonts cache-aware timeout adaption

third_party/WebKit/Source/core/css/RemoteFontFaceSource.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/css/RemoteFontFaceSource.h"
 
 #include "core/css/CSSCustomFontData.h"
 #include "core/css/CSSFontFace.h"
 #include "core/css/CSSFontSelector.h"
 #include "core/dom/Document.h"
@@ skipping 38 matching lines @@
 }
 
 }  // namespace
 
 RemoteFontFaceSource::RemoteFontFaceSource(FontResource* font,
                                            CSSFontSelector* fontSelector,
                                            FontDisplay display)
     : m_font(font),
       m_fontSelector(fontSelector),
       m_display(display),
-      m_period(display == FontDisplaySwap ? SwapPeriod : BlockPeriod),
+      m_period(BlockPeriod),

[Kunihiko Sakamoto, 2016/10/12 06:10:14]

Would this change current behavior when WebFontsCacheAwareTimeoutAdaption flag
is off?

[Shao-Chuan Lee, 2016/10/12 06:50:58]

If the flag is off then isCacheAwareLoadingActivated() will be false, and
|m_period| will be correctly set below.

[Kunihiko Sakamoto, 2016/10/12 08:56:30]

Ah OK, I missed that part.

       m_histograms(font->url().protocolIsData()
                        ? FontLoadHistograms::FromDataURL
                        : font->isLoaded() ? FontLoadHistograms::FromMemoryCache
                                           : FontLoadHistograms::FromUnknown),
       m_isInterventionTriggered(false) {
   ThreadState::current()->registerPreFinalizer(this);
   m_font->addClient(this);
 
   if (shouldTriggerWebFontsIntervention()) {
     m_isInterventionTriggered = true;
     m_period = SwapPeriod;
     m_fontSelector->document()->addConsoleMessage(ConsoleMessage::create(
         OtherMessageSource, InfoMessageLevel,
         "Slow network is detected. Fallback font will be used while loading: " +
             m_font->url().elidedString()));
   }
+
+  if (m_display == FontDisplaySwap &&
+      !m_font->resourceRequest().isCacheAwareLoadingActivated())
+    m_period = SwapPeriod;
 }
 
 RemoteFontFaceSource::~RemoteFontFaceSource() {}
 
 void RemoteFontFaceSource::dispose() {
   m_font->removeClient(this);
   m_font = nullptr;
   pruneTable();
 }
 
@@ skipping 61 matching lines @@
 void RemoteFontFaceSource::fontLoadLongLimitExceeded(FontResource*) {
   if (m_display == FontDisplayBlock ||
       (!m_isInterventionTriggered && m_display == FontDisplayAuto))
     switchToSwapPeriod();
   else if (m_display == FontDisplayFallback)
     switchToFailurePeriod();
 
   m_histograms.longLimitExceeded(m_isInterventionTriggered);
 }
 
+void RemoteFontFaceSource::willReloadAfterDiskCacheMiss(const FontResource*) {
+  if (m_display == FontDisplaySwap)
+    switchToSwapPeriod();
+}
+
 void RemoteFontFaceSource::switchToSwapPeriod() {
   ASSERT(m_period == BlockPeriod);
   m_period = SwapPeriod;
 
   pruneTable();
   if (m_face) {
     m_fontSelector->fontFaceInvalidated();
     m_face->didBecomeVisibleFallback(this);
   }
 
@@ skipping 252 matching lines @@
       return Miss;
     case FromUnknown:
     // Fall through.
     default:
       NOTREACHED();
   }
   return Miss;
 }
 
 }  // namespace blink