Pepper: Autogenerate thunk for PPB_TrueTypeFont.

ppapi/api/dev/ppb_truetype_font_dev.idl

Index: ppapi/api/dev/ppb_truetype_font_dev.idl
diff --git a/ppapi/api/dev/ppb_truetype_font_dev.idl b/ppapi/api/dev/ppb_truetype_font_dev.idl
index 9775d98713e71409b6f412c522e9b98558455538..920f1b82c3b49a7f22dfcc526d335c9addb9302d 100644
--- a/ppapi/api/dev/ppb_truetype_font_dev.idl
+++ b/ppapi/api/dev/ppb_truetype_font_dev.idl
@@ -8,6 +8,9 @@
  * interface exposes font table data for 'sfnt' fonts on the host system. These
  * include TrueType and OpenType fonts.
  */
+
+[generate_thunk,thunk_include="ppapi/thunk/ppb_truetype_font_singleton_api.h"]

[yzshen1, 2013/04/10 17:00:48]

Does it make sense to deduce the include file name and omit the "thunk_include"
part? I mean, we could decide to include ppb_truetype_font_singleton_api.h when
we see [singleton,api=PPB_TrueTypeFont_Singleton_API].

+
 label Chrome {
   M26 = 0.1
 };
@@ -146,6 +149,7 @@ interface PPB_TrueTypeFont_Dev {
    * @return If >= 0, the number of family names returned, otherwise an error
    * code from <code>pp_errors.h</code>.
    */
+  [singleton,api=PPB_TrueTypeFont_Singleton_API]
   int32_t GetFontFamilies([in] PP_Instance instance,
                           [in] PP_ArrayOutput output,
                           [in] PP_CompletionCallback callback);
@@ -169,6 +173,7 @@ interface PPB_TrueTypeFont_Dev {
    * @return <code>PP_TRUE</code> if the resource is a
    * <code>PPB_TrueTypeFont_Dev</code>, <code>PP_FALSE</code> otherwise.
    */
+  [is_function]

[yzshen1, 2013/04/10 17:00:48]

(1) Maybe we should change its name to IsTrueTypeFont() to make it conform to
the naming rule? I think we don't need the label in that case, right?

(2) If we decide not to do (1), maybe we could use a more descriptive label? For
example, 'resource_type_tester'. When I saw 'is_function', I thought it means
'this is a function'.

[bbudge, 2013/04/10 17:38:18]

I think renaming to IsTrueTypeFont is a good idea. Hopefully we follow the
convention everywhere else so we wouldn't need this annotation. I doubt there
are many users of this API.
On 2013/04/10 17:00:48, yzshen1 wrote:

   PP_Bool IsFont([in] PP_Resource resource);
 
   /**