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

Unified Diff: ppapi/api/ppp_instance.idl

Issue 10694026: Plumb through the page focus lost event to Pepper plugins. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 6 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 | « content/renderer/render_view_impl.cc ('k') | ppapi/c/ppp_instance.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/api/ppp_instance.idl
diff --git a/ppapi/api/ppp_instance.idl b/ppapi/api/ppp_instance.idl
index 7b0b1d36ee5906a68c1bace639516d97c3f1d90a..a5026ae23a762cacf0e3b0630a61eb169510ed61 100644
--- a/ppapi/api/ppp_instance.idl
+++ b/ppapi/api/ppp_instance.idl
@@ -177,14 +177,22 @@ interface PPP_Instance {
* Having focus means that keyboard events will be sent to the instance.
* An instance's default condition is that it will not have focus.
*
+ * The focus flag takes into account both browser tab and window focus as
+ * well as focus of the plugin element on the page. In order to be deemed
+ * to have focus, the browser window must be topmost, the tab must be
+ * selected in the window, and the instance must be the focused element on
+ * the page.
+ *
* <strong>Note:</strong>Clicks on instances will give focus only if you
* handle the click event. Return <code>true</code> from
* <code>HandleInputEvent</code> in <code>PPP_InputEvent</code> (or use
* unfiltered events) to signal that the click event was handled. Otherwise,
* the browser will bubble the event and give focus to the element on the page
* that actually did end up consuming it. If you're not getting focus, check
- * to make sure you're returning true from the mouse click in
- * <code>HandleInputEvent</code>.
+ * to make sure you're either requesting them via
+ * <code>RequestInputEvents()<code> (which implicitly marks all input events
+ * as consumed) or via <code>RequestFilteringInputEvents()</code> and
+ * returning true from your event handler.
*
* @param[in] instance A <code>PP_Instance</code> identifying the instance
* receiving the input event.
« no previous file with comments | « content/renderer/render_view_impl.cc ('k') | ppapi/c/ppp_instance.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698