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

Unified Diff: src/fonts/SkFontMgr_fontconfig.cpp

Issue 23966003: Update the freetype backed fonthost to keep the style and fixed width attributes for a font stream. (Closed) Base URL: https://skia.googlecode.com/svn/trunk
Patch Set: updating per comments Created 7 years, 3 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
« no previous file with comments | « no previous file | src/ports/SkFontConfigTypeface.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/fonts/SkFontMgr_fontconfig.cpp
diff --git a/src/fonts/SkFontMgr_fontconfig.cpp b/src/fonts/SkFontMgr_fontconfig.cpp
index 367faf85284e4ddd948d892bf8d39dadf8634db8..0b4accd254f28cb6edf493a0d967388a56b69a08 100644
--- a/src/fonts/SkFontMgr_fontconfig.cpp
+++ b/src/fonts/SkFontMgr_fontconfig.cpp
@@ -272,9 +272,9 @@ protected:
return NULL; // don't accept too large fonts (>= 1GB) for safety.
}
- // TODO should the caller give us the style?
+ // TODO should the caller give us the style or should we get it from freetype?
SkTypeface::Style style = SkTypeface::kNormal;
- SkTypeface* face = SkNEW_ARGS(FontConfigTypeface, (style, stream));
+ SkTypeface* face = SkNEW_ARGS(FontConfigTypeface, (style, false, stream));
return face;
}
« no previous file with comments | « no previous file | src/ports/SkFontConfigTypeface.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698