RemoteFontFaceSource should not call startLoadLimitTimers() if startLoad() fails

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 231 matching lines @@
 
 void RemoteFontFaceSource::beginLoadIfNeeded() {
   if (m_fontSelector->document() && m_font->stillNeedsLoad()) {
     if (!m_font->url().protocolIsData() && !m_font->isLoaded() &&
         m_display == FontDisplayAuto &&
         m_font->isLowPriorityLoadingAllowedForRemoteFont()) {
       // Set the loading priority to VeryLow since this font is not required
       // for painting the text.
       m_font->didChangePriority(ResourceLoadPriorityVeryLow, 0);
     }
-    m_fontSelector->document()->fetcher()->startLoad(m_font);
-    if (!m_font->isLoaded())
-      m_font->startLoadLimitTimers();
-    m_histograms.loadStarted();
+    if (m_fontSelector->document()->fetcher()->startLoad(m_font)) {
+      // Start timers only when load is actually started asynchronously.
+      if (!m_font->isLoaded())
+        m_font->startLoadLimitTimers();
+      m_histograms.loadStarted();
+    }
   }
 
   if (m_face)
     m_face->didBeginLoad();
 }
 
 DEFINE_TRACE(RemoteFontFaceSource) {
   visitor->trace(m_font);
   visitor->trace(m_fontSelector);
   CSSFontFaceSource::trace(visitor);
@@ skipping 198 matching lines @@
       return Miss;
     case FromUnknown:
     // Fall through.
     default:
       NOTREACHED();
   }
   return Miss;
 }
 
 }  // namespace blink