Test for BrowserPlugin-based WebView Focus

content/public/test/browser_test_utils.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_PUBLIC_TEST_BROWSER_TEST_UTILS_H_
 #define CONTENT_PUBLIC_TEST_BROWSER_TEST_UTILS_H_
 
 #include <queue>
 #include <string>
 #include <vector>
@@ skipping 292 matching lines @@
 // is equal to the given name. If not, sets up a notification waiter
 // that listens for any accessibility event in any frame, and checks again
 // after each event. Keeps looping until the text is found (or the
 // test times out).
 void WaitForAccessibilityTreeToContainNodeWithName(WebContents* web_contents,
                                                    const std::string& name);
 
 // 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 {
  public:
   // |web_contents| must be non-NULL and needs to stay alive for the
   // entire lifetime of |this|. |expected_title| is the title that |this|
   // will wait for.
   TitleWatcher(WebContents* web_contents,
                const base::string16& expected_title);
   ~TitleWatcher() override;
@@ skipping 199 matching lines @@
   blink::WebInputEvent::Type wait_for_type_;
   uint32_t ack_result_;
   base::Closure quit_;
 
   DISALLOW_COPY_AND_ASSIGN(InputMsgWatcher);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_TEST_BROWSER_TEST_UTILS_H_