Remove WebViewClient::showVirtualKeyboardOnElementFocus()

third_party/WebKit/Source/web/tests/WebFrameTest.cpp

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 11397 matching lines @@
   FrameTestHelpers::TestWebViewClient openerWebViewClient;
   FrameTestHelpers::WebViewHelper openerHelper;
   openerHelper.initialize(false, nullptr, &openerWebViewClient);
   FrameTestHelpers::WebViewHelper helper;
   helper.initializeWithOpener(openerHelper.webView()->mainFrame());
 
   openerHelper.reset();
   EXPECT_EQ(nullptr, helper.webView()->mainFrameImpl()->opener());
 }
 
+class ShowVirtualKeyboardObserverWidgetClient
+    : public FrameTestHelpers::TestWebWidgetClient {
+ public:
+  ShowVirtualKeyboardObserverWidgetClient() : m_didShowVirtualKeyboard(false) {}
+
+  void showVirtualKeyboardOnElementFocus() override {
+    m_didShowVirtualKeyboard = true;
+  }
+
+  bool didShowVirtualKeyboard() const { return m_didShowVirtualKeyboard; }
+
+ private:
+  bool m_didShowVirtualKeyboard;
+};
+
+TEST_F(WebFrameTest, ShowVirtualKeyboardOnElementFocus) {
+  FrameTestHelpers::WebViewHelper webViewHelper;
+  WebViewImpl* webView = webViewHelper.initialize(true);
+  WebRemoteFrameImpl* remoteFrame = static_cast<WebRemoteFrameImpl*>(
+      WebRemoteFrame::create(WebTreeScopeType::Document, nullptr));

[dcheng, 2017/03/09 02:00:25]

Nit: WebRemoteFrameImpl::create will avoid the need for a static cast.

[EhsanK, 2017/03/09 17:19:49]

Acknowledged.

+  webView->setMainFrame(remoteFrame);
+  RefPtr<SecurityOrigin> origin =

[EhsanK, 2017/03/08 23:38:28]

If I don't set this, it will be null for the frame and I will get a crash when
trying to load the local frame.

[dcheng, 2017/03/09 02:00:25]

Nit: we usually we just create a unique origin here.

[EhsanK, 2017/03/09 17:19:49]

Acknowledged.

+      SecurityOrigin::createFromString("http://remote.com");
+  remoteFrame->frame()->securityContext()->setSecurityOrigin(origin);
+
+  ShowVirtualKeyboardObserverWidgetClient webWidgetClient;
+  WebLocalFrameImpl* localFrame = FrameTestHelpers::createLocalChild(
+      remoteFrame, "child", nullptr, &webWidgetClient);
+
+  registerMockedHttpURLLoad("input_field_default.html");
+  FrameTestHelpers::loadFrame(localFrame,
+                              m_baseURL + "input_field_default.html");
+
+  // Simulate an input element focus leading to Element::focus() call with a
+  // user gesture.
+  localFrame->setHasReceivedUserGesture();
+  localFrame->executeScript(
+      WebScriptSource("window.focus();"
+                      "document.querySelector('input').focus();"));
+
+  // Verify that the right WebWidgetClient has been notified.
+  EXPECT_TRUE(webWidgetClient.didShowVirtualKeyboard());
+
+  remoteFrame->close();
+  webViewHelper.reset();
+}
+
 }  // namespace blink