Index: chrome/browser/apps/guest_view/web_view_browsertest.cc |
diff --git a/chrome/browser/apps/guest_view/web_view_browsertest.cc b/chrome/browser/apps/guest_view/web_view_browsertest.cc |
index 9dc906ce740d031bb3f5781cbcb7011c4dc9925d..19849cb4e70d87532d0a49aa1b6ad5a521c469e6 100644 |
--- a/chrome/browser/apps/guest_view/web_view_browsertest.cc |
+++ b/chrome/browser/apps/guest_view/web_view_browsertest.cc |
@@ -2,6 +2,8 @@ |
// Use of this source code is governed by a BSD-style license that can be |
// found in the LICENSE file. |
+#include <queue> |
+ |
#include "base/location.h" |
#include "base/path_service.h" |
#include "base/process/process.h" |
@@ -52,8 +54,14 @@ |
#include "net/test/embedded_test_server/embedded_test_server.h" |
#include "net/test/embedded_test_server/http_request.h" |
#include "net/test/embedded_test_server/http_response.h" |
+#include "ui/aura/window.h" |
+#include "ui/compositor/compositor.h" |
+#include "ui/compositor/compositor_observer.h" |
+#include "ui/events/event_switches.h" |
#include "ui/gfx/switches.h" |
#include "ui/gl/gl_switches.h" |
+#include "ui/views/view.h" |
+#include "ui/views/widget/widget.h" |
#if defined(ENABLE_PLUGINS) |
#include "content/public/browser/plugin_service.h" |
@@ -206,6 +214,32 @@ void ExecuteScriptWaitForTitle(content::WebContents* web_contents, |
EXPECT_EQ(expected_title, title_watcher.WaitAndGetTitle()); |
} |
+#if defined (USE_AURA) |
+void GiveItSomeTime(int ms) { |
+ base::RunLoop run_loop; |
+ base::ThreadTaskRunnerHandle::Get()->PostDelayedTask( |
+ FROM_HERE, run_loop.QuitClosure(), base::TimeDelta::FromMilliseconds(ms)); |
+ run_loop.Run(); |
+} |
+ |
+views::View* FindWebView(views::View* v) { |
nasko
2015/06/22 08:40:52
nit: Using just one letter for variable names (oth
wjmaclean
2015/06/22 13:39:43
Done.
|
+ std::queue<views::View*> queue; |
+ queue.push(v); |
+ while (!queue.empty()) { |
+ views::View* current = queue.front(); |
+ queue.pop(); |
+ if (std::string(current->GetClassName()).find("WebView") != |
+ std::string::npos) { |
+ return current; |
+ } |
+ |
+ for (int i = 0; i < current->child_count(); ++i) |
+ queue.push(current->child_at(i)); |
+ } |
+ return nullptr; |
+} |
+#endif |
+ |
} // namespace |
// This class intercepts media access request from the embedder. The request |
@@ -659,6 +693,9 @@ class WebViewTest : public extensions::PlatformAppBrowserTest { |
GuestViewManager::set_factory_for_testing(&factory_); |
} |
+ protected: |
+ scoped_refptr<content::FrameWatcher> frame_watcher_; |
+ |
private: |
bool UsesFakeSpeech() { |
const testing::TestInfo* const test_info = |
@@ -2372,3 +2409,91 @@ IN_PROC_BROWSER_TEST_F(WebViewTest, Shim_TestGarbageCollect) { |
TestHelper("testGarbageCollect", "web_view/shim", NO_TEST_SERVER); |
GetGuestViewManager()->WaitForSingleViewGarbageCollected(); |
} |
+ |
+#if defined(USE_AURA) |
+class WebViewFocusTest : public WebViewTest { |
+ public: |
+ ~WebViewFocusTest() override {} |
+ |
+ void SetUpCommandLine(base::CommandLine* command_line) override { |
+ WebViewTest::SetUpCommandLine(command_line); |
+ |
+ command_line->AppendSwitchASCII(switches::kTouchEvents, |
+ switches::kTouchEventsEnabled); |
+ } |
+ |
+ void ForceCompositorFrame() { |
+ if (!frame_watcher_) { |
+ frame_watcher_ = new content::FrameWatcher(); |
+ frame_watcher_->AttachTo(GetEmbedderWebContents()); |
+ } |
+ |
+ while (!RequestFrame(GetEmbedderWebContents())) |
+ GiveItSomeTime(10 /* milliseconds */); |
+ frame_watcher_->WaitFrames(1); |
+ } |
+ |
+ private: |
+ scoped_refptr<content::FrameWatcher> frame_watcher_; |
+}; |
+ |
+// The following test verifies that a views::WebView hosting an embedder |
+// gains focus on touchstart. |
+IN_PROC_BROWSER_TEST_F(WebViewFocusTest, TouchFocusesEmbedder) { |
+ LoadAppWithGuest("web_view/accept_touch_events"); |
+ |
+ content::WebContents* web_contents = GetEmbedderWebContents(); |
+ content::RenderViewHost* embedder_rvh = web_contents->GetRenderViewHost(); |
+ |
+ bool embedder_has_touch_handler = |
+ content::RenderViewHostTester::HasTouchEventHandler(embedder_rvh); |
+ EXPECT_FALSE(embedder_has_touch_handler); |
+ |
+ SendMessageToGuestAndWait("install-touch-handler", "installed-touch-handler"); |
+ |
+ // Note that we need to wait for the installed/registered touch handler to |
+ // appear in browser process before querying |embedder_rvh|. |
+ // In practice, since we do a roundrtip from browser process to guest and |
+ // back, this is sufficient. |
+ embedder_has_touch_handler = |
+ content::RenderViewHostTester::HasTouchEventHandler(embedder_rvh); |
+ EXPECT_TRUE(embedder_has_touch_handler); |
+ |
+ extensions::AppWindow* app_window = GetFirstAppWindowForBrowser(browser()); |
+ aura::Window* window = app_window->GetNativeWindow(); |
+ EXPECT_TRUE(app_window); |
+ EXPECT_TRUE(window); |
+ views::Widget* widget = views::Widget::GetWidgetForNativeView(window); |
+ EXPECT_TRUE(widget->GetRootView()); |
+ // We only expect a single views::webview in the view hierarchy. |
+ views::View* aura_webview = FindWebView(widget->GetRootView()); |
+ ASSERT_TRUE(aura_webview); |
+ gfx::Rect bounds(aura_webview->bounds()); |
+ EXPECT_TRUE(aura_webview->IsFocusable()); |
+ |
+ views::View* other_focusable_view = new views::View(); |
+ other_focusable_view->SetBounds(bounds.x() + bounds.width(), 0, 100, 100); |
+ other_focusable_view->SetFocusable(true); |
+ aura_webview->parent()->AddChildView(other_focusable_view); |
+ other_focusable_view->SetPosition(gfx::Point(bounds.x() + bounds.width(), 0)); |
+ |
+ // Sync changes to compositor. |
+ ForceCompositorFrame(); |
+ |
+ EXPECT_TRUE(aura_webview->HasFocus()); |
+ other_focusable_view->RequestFocus(); |
+ EXPECT_TRUE(other_focusable_view->HasFocus()); |
+ EXPECT_FALSE(aura_webview->HasFocus()); |
+ |
+ // Generate and send synthetic touch event. |
+ // TODO(wjmaclean): This is fragile ... if anyone alters the location/size |
+ // of the webview in accept_touch_events then this may miss its target. |
+ content::SimulateTouchPressAt(GetEmbedderWebContents(), gfx::Point(10, 10)); |
+ |
+ // Wait for the TouchStart to propagate and restore focus. Test times out |
+ // on failure. |
+ while (!aura_webview->HasFocus()) |
+ base::MessageLoop::current()->RunUntilIdle(); |
+ GiveItSomeTime(10 /* milliseconds */); |
+} |
+#endif |