Test for BrowserPlugin-based WebView Focus

content/public/test/browser_test_utils.h

Index: content/public/test/browser_test_utils.h
diff --git a/content/public/test/browser_test_utils.h b/content/public/test/browser_test_utils.h
index 52c443817281d09039b1b5506711bcb426a25f65..3350fadd54bc39f61087f0073777da86c2b3bb3e 100644
--- a/content/public/test/browser_test_utils.h
+++ b/content/public/test/browser_test_utils.h
@@ -310,6 +310,24 @@ void WaitForAccessibilityTreeToContainNodeWithName(WebContents* web_contents,
 // Get a snapshot of a web page's accessibility tree.
 ui::AXTreeUpdate GetAccessibilityTreeSnapshot(WebContents* web_contents);
 
+// Find out if the BrowserPlugin for a guest webcontents is focused. Returns

[Charlie Reis, 2016/07/06 18:28:50]

nit: WebContents

[wjmaclean, 2016/07/06 19:49:31]

Done.

+// false if the WebContents doesn't have a BrowserPlugin.

[Charlie Reis, 2016/07/06 18:28:49]

"doesn't have a BrowserPlugin": I'm not sure how to read this phrase.  It could
mean an embedder WebContents that "has" one or more guest BrowserPlugins, or it
could mean that we're talking about a WebContents which is not itself a guest. 
I think it's the latter?

Maybe there's a clearer way to phrase?

[wjmaclean, 2016/07/06 19:49:31]

Done.

+bool IsWebContentsBrowserPluginFocused(content::WebContents* web_contents);
+
+#if defined(USE_AURA)
+// Send a TouchStart/End sequence routed via the main frame's
+// RenderWidgetHostViewAura.

[Charlie Reis, 2016/07/06 18:28:49]

Maybe mention why these two methods are useful?

[wjmaclean, 2016/07/06 19:49:31]

Done.

+void SendRoutedTouchTapSequence(content::WebContents* web_contents,
+                                gfx::Point point);
+
+// Send a GestureTapDown/GestureTap sequence routed via the main frame's
+// RenderWidgetHostViewAura.
+void SendRoutedGestureTapSequence(content::WebContents* web_contents,
+                                  gfx::Point point);
+#endif
+
+void WaitForSurfaceReady(content::WebContents* web_contents);

[Charlie Reis, 2016/07/06 18:28:49]

nit: Please comment all methods in the public interface.

[wjmaclean, 2016/07/06 19:49:31]

Done.

Ooops, forgot to comment this one!

+
 // Watches title changes on a WebContents, blocking until an expected title is
 // set.
 class TitleWatcher : public WebContentsObserver {