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

Unified Diff: ppapi/c/dev/ppb_font_dev.h

Issue 7053022: Use the WebKit default fonts when specifying generic font families. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 7 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: ppapi/c/dev/ppb_font_dev.h
===================================================================
--- ppapi/c/dev/ppb_font_dev.h (revision 86427)
+++ ppapi/c/dev/ppb_font_dev.h (working copy)
@@ -56,18 +56,31 @@
struct PP_FontDescription_Dev {
/**
* Font face name as a string. This can also be an undefined var, in which
- * case the generic family will be obeyed.
+ * case the generic family will be obeyed. If the face is not available on
+ * the system, the browser will attempt to do font fallback or pick a default
+ * font.
*/
struct PP_Var face;
/**
- * When face is an undefined var, this specifies the generic font family type
- * to use. If the face is specified, this will be ignored.
+ * When Create()ing a font and the face is an undefined var, the family
+ * specifies the generic font family type to use. If the face is specified,
+ * this will be ignored.
+ *
+ * When Describe()ing a font, the family will be the value you passed in when
+ * the font was created. In other words, if you specify a face name, the
+ * family will not be updated to reflect whether the font name you requested
+ * is serif or sans serif.
*/
PP_FontFamily_Dev family;
/**
* Size in pixels.
+ *
+ * You can specify 0 to get the default font size. The default font size
+ * may vary depending on the requested font. The typical example is that
+ * the the user may have a different font size for the default monospace font
yzshen1 2011/05/25 23:40:11 Please remove one 'the' at the beginning.
+ * to give it a similar optical size to the proportionally spaced fonts.
*/
uint32_t size;

Powered by Google App Engine
This is Rietveld 408576698