Index: ppapi/api/trusted/ppb_browser_font_trusted.idl |
diff --git a/ppapi/api/trusted/ppb_browser_font_trusted.idl b/ppapi/api/trusted/ppb_browser_font_trusted.idl |
index 7c0c633f344286c7048f057d64295fff3f5f9d8b..79cee8cefd149f5969dbec3ea6cc233f3d3efb8c 100644 |
--- a/ppapi/api/trusted/ppb_browser_font_trusted.idl |
+++ b/ppapi/api/trusted/ppb_browser_font_trusted.idl |
@@ -237,6 +237,10 @@ interface PPB_BrowserFont_Trusted { |
* the string. This handles complex scripts such as Arabic, where characters |
* may be combined or replaced depending on the context. Returns (uint32)-1 |
* on failure. |
+ * |
+ * TODO(brettw) this function may be broken. See the CharPosRTL test. It |
+ * seems to tell you "insertion point" rather than painting position. This |
+ * is useful but maybe not what we intended here. |
*/ |
uint32_t CharacterOffsetForPixel( |
[in] PP_Resource font, |