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

Unified Diff: Source/platform/fonts/shaping/HarfBuzzShaper.h

Issue 1241613006: Reland "Change fallback font collection in HarfBuzzShaper" (Closed) Base URL: https://chromium.googlesource.com/chromium/blink@master
Patch Set: Created 5 years, 5 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 side-by-side diff with in-line comments
Download patch
Index: Source/platform/fonts/shaping/HarfBuzzShaper.h
diff --git a/Source/platform/fonts/shaping/HarfBuzzShaper.h b/Source/platform/fonts/shaping/HarfBuzzShaper.h
index 6d419d51abb41092cbd9f67e26a1ce1f6c290945..e8e98718737676ae48e91e52d72884a3a47e94b5 100644
--- a/Source/platform/fonts/shaping/HarfBuzzShaper.h
+++ b/Source/platform/fonts/shaping/HarfBuzzShaper.h
@@ -63,10 +63,7 @@ public:
FloatRect bounds() { return m_glyphBoundingBox; }
int offsetForPosition(float targetX);
unsigned numCharacters() const { return m_numCharacters; }
- const HashSet<const SimpleFontData*>* fallbackFonts() const
- {
- return &m_fallbackFonts;
- }
+ void fallbackFonts(HashSet<const SimpleFontData*>*) const;
static float fillGlyphBuffer(Vector<RefPtr<ShapeResult>>&,
GlyphBuffer*, const TextRun&, unsigned from, unsigned to);
@@ -103,7 +100,7 @@ private:
float m_width;
FloatRect m_glyphBoundingBox;
Vector<RunInfo*> m_runs;
- HashSet<const SimpleFontData*> m_fallbackFonts;
+ const SimpleFontData* m_primaryFont;
unsigned m_numCharacters;
unsigned m_numGlyphs : 31;
@@ -118,8 +115,7 @@ private:
class PLATFORM_EXPORT HarfBuzzShaper final : public Shaper {
public:
- HarfBuzzShaper(const Font*, const TextRun&,
- HashSet<const SimpleFontData*>* fallbackFonts);
+ HarfBuzzShaper(const Font*, const TextRun&);
PassRefPtr<ShapeResult> shapeResult();
~HarfBuzzShaper() { }
« no previous file with comments | « Source/platform/fonts/shaping/CachingWordShaperTest.cpp ('k') | Source/platform/fonts/shaping/HarfBuzzShaper.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698