Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(44)

Side by Side Diff: third_party/WebKit/Source/core/css/RemoteFontFaceSource.h

Issue 1894953002: WebFonts: merge fontLoaded() to notifyFinished() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef RemoteFontFaceSource_h 5 #ifndef RemoteFontFaceSource_h
6 #define RemoteFontFaceSource_h 6 #define RemoteFontFaceSource_h
7 7
8 #include "core/css/CSSFontFaceSource.h" 8 #include "core/css/CSSFontFaceSource.h"
9 #include "core/fetch/FontResource.h" 9 #include "core/fetch/FontResource.h"
10 #include "wtf/Allocator.h" 10 #include "wtf/Allocator.h"
(...skipping 20 matching lines...) Expand all
31 ~RemoteFontFaceSource() override; 31 ~RemoteFontFaceSource() override;
32 void dispose(); 32 void dispose();
33 33
34 bool isLoading() const override; 34 bool isLoading() const override;
35 bool isLoaded() const override; 35 bool isLoaded() const override;
36 bool isValid() const override; 36 bool isValid() const override;
37 DisplayPeriod getDisplayPeriod() const { return m_period; } 37 DisplayPeriod getDisplayPeriod() const { return m_period; }
38 38
39 void beginLoadIfNeeded() override; 39 void beginLoadIfNeeded() override;
40 40
41 void fontLoaded(FontResource*) override; 41 void notifyFinished(Resource*) override;
42 void fontLoadShortLimitExceeded(FontResource*) override; 42 void fontLoadShortLimitExceeded(FontResource*) override;
43 void fontLoadLongLimitExceeded(FontResource*) override; 43 void fontLoadLongLimitExceeded(FontResource*) override;
44 String debugName() const override { return "RemoteFontFaceSource"; } 44 String debugName() const override { return "RemoteFontFaceSource"; }
45 45
46 // For UMA reporting 46 // For UMA reporting
47 bool hadBlankText() override { return m_histograms.hadBlankText(); } 47 bool hadBlankText() override { return m_histograms.hadBlankText(); }
48 void paintRequested() { m_histograms.fallbackFontPainted(m_period); } 48 void paintRequested() { m_histograms.fallbackFontPainted(m_period); }
49 49
50 DECLARE_VIRTUAL_TRACE(); 50 DECLARE_VIRTUAL_TRACE();
51 51
(...skipping 29 matching lines...) Expand all
81 Member<CSSFontSelector> m_fontSelector; 81 Member<CSSFontSelector> m_fontSelector;
82 const FontDisplay m_display; 82 const FontDisplay m_display;
83 DisplayPeriod m_period; 83 DisplayPeriod m_period;
84 FontLoadHistograms m_histograms; 84 FontLoadHistograms m_histograms;
85 bool m_isInterventionTriggered; 85 bool m_isInterventionTriggered;
86 }; 86 };
87 87
88 } // namespace blink 88 } // namespace blink
89 89
90 #endif 90 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698