Allow only a user gesture to trigger autofill popup

content/public/test/render_view_test.cc

 // 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.
 
 #include "content/public/test/render_view_test.h"
 
+#include <cctype>
+
 #include "base/run_loop.h"
 #include "content/common/dom_storage/dom_storage_types.h"
 #include "content/common/frame_messages.h"
 #include "content/common/input_messages.h"
 #include "content/common/view_messages.h"
 #include "content/public/browser/content_browser_client.h"
 #include "content/public/browser/native_web_keyboard_event.h"
 #include "content/public/common/content_client.h"
 #include "content/public/common/renderer_preferences.h"
 #include "content/public/renderer/content_renderer_client.h"
 #include "content/public/test/frame_load_waiter.h"
 #include "content/renderer/history_controller.h"
 #include "content/renderer/history_serialization.h"
 #include "content/renderer/render_thread_impl.h"
 #include "content/renderer/render_view_impl.h"
 #include "content/renderer/renderer_blink_platform_impl.h"
 #include "content/renderer/renderer_main_platform_delegate.h"
 #include "content/renderer/scheduler/renderer_scheduler.h"
 #include "content/test/fake_compositor_dependencies.h"
 #include "content/test/mock_render_process.h"
 #include "content/test/test_content_client.h"
 #include "third_party/WebKit/public/platform/WebScreenInfo.h"
 #include "third_party/WebKit/public/platform/WebURLRequest.h"
+#include "third_party/WebKit/public/web/WebDocument.h"
 #include "third_party/WebKit/public/web/WebHistoryItem.h"
+#include "third_party/WebKit/public/web/WebInputElement.h"
 #include "third_party/WebKit/public/web/WebInputEvent.h"
 #include "third_party/WebKit/public/web/WebKit.h"
 #include "third_party/WebKit/public/web/WebLocalFrame.h"
 #include "third_party/WebKit/public/web/WebScriptSource.h"
 #include "third_party/WebKit/public/web/WebView.h"
 #include "ui/base/resource/resource_bundle.h"
+#include "ui/events/keycodes/keyboard_codes.h"
 #include "v8/include/v8.h"
 
 #if defined(OS_MACOSX)
 #include "base/mac/scoped_nsautorelease_pool.h"
 #endif
 
 using blink::WebGestureEvent;
 using blink::WebInputEvent;
 using blink::WebLocalFrame;
 using blink::WebMouseEvent;
 using blink::WebScriptSource;
 using blink::WebString;
 using blink::WebURLRequest;
 
 namespace {
+
 const int32 kOpenerId = -2;
 const int32 kRouteId = 5;
 const int32 kMainFrameRouteId = 6;
 const int32 kNewWindowRouteId = 7;
 const int32 kNewFrameRouteId = 10;
 const int32 kSurfaceId = 42;
 
+// Converts |ascii_character| into |key_code| and returns true on success.
+// Handles only the characters needed by tests.
+bool GetWindowsKeyCode(char ascii_character, int* key_code) {
+  if (isalnum(ascii_character)) {
+    *key_code = base::ToUpperASCII(ascii_character);
+    return true;
+  }
+
+  switch (ascii_character) {
+    case '@':
+      *key_code = '2';
+      return true;
+    case '_':
+      *key_code = ui::VKEY_OEM_MINUS;
+      return true;
+    case '.':
+      *key_code = ui::VKEY_OEM_PERIOD;
+      return true;
+    case ui::VKEY_BACK:
+      *key_code = ui::VKEY_BACK;
+      return true;
+    default:
+      return false;
+  }
+}
+
 }  // namespace
 
 namespace content {
 
 class RendererBlinkPlatformImplNoSandboxImpl
     : public RendererBlinkPlatformImpl {
  public:
   RendererBlinkPlatformImplNoSandboxImpl(RendererScheduler* scheduler)
       : RendererBlinkPlatformImpl(scheduler) {}
 
@@ skipping 318 matching lines @@
   params.physical_backing_size = new_size;
   params.top_controls_height = 0.f;
   params.top_controls_shrink_blink_size = false;
   params.resizer_rect = resizer_rect;
   params.is_fullscreen_granted = is_fullscreen_granted;
   params.display_mode = blink::WebDisplayModeBrowser;
   scoped_ptr<IPC::Message> resize_message(new ViewMsg_Resize(0, params));
   OnMessageReceived(*resize_message);
 }
 
+void RenderViewTest::SimulateUserTypingASCIICharacter(char ascii_character) {
+  blink::WebKeyboardEvent event;
+  event.text[0] = ascii_character;
+  ASSERT_TRUE(GetWindowsKeyCode(ascii_character, &event.windowsKeyCode));
+  if (isupper(ascii_character) || ascii_character == '@' ||
+      ascii_character == '_') {
+    event.modifiers = blink::WebKeyboardEvent::ShiftKey;
+  }
+
+  event.type = blink::WebKeyboardEvent::RawKeyDown;
+  SendWebKeyboardEvent(event);
+
+  event.type = blink::WebKeyboardEvent::Char;
+  SendWebKeyboardEvent(event);
+
+  event.type = blink::WebKeyboardEvent::KeyUp;
+  SendWebKeyboardEvent(event);
+}
+
+void RenderViewTest::ProcessInputForAutofill() {

[Jay Civelli, 2015/04/23 21:57:43]

I am confused on whether this actually does something that triggers processing
of autofill or if it makes us wait for a state when we know auotfill has been
applied.

[please use gerrit instead, 2015/04/27 21:20:34]

Running the message loop will trigger autofill.

When a text changes in an input field,  AutofillAgent::textFieldDidChange will
post the function that will trigger the autofill field. The actual autofill
happens in AutofillAgent::TextFieldDidChangeImpl.

[Jay Civelli, 2015/04/27 22:17:19]

OK, why do we need to re-layout though? Wouldn't the
AutofillAgent::textFieldDidChange be called without it?
I also wonder if we could merge this method with
SimulateUserTypingASCIICharacter.
It could take a bool parameter to specify whether to flush the message loop.
WDYT?

[please use gerrit instead, 2015/04/27 23:49:54]

It seems we don't need re-layout. The original comment must
have been outdated. After removing re-layout from tests, all tests are still
passing.

I've added |flush_message_loop| parameter to SimulateUserTypingASCIICharacter
and removed ProcessInputForAutofill entirely.

+  // Processing is delayed because of a Blink bug:
+  // https://bugs.webkit.org/show_bug.cgi?id=16976 See
+  // PasswordAutofillAgent::TextDidChangeInTextField() for details.
+
+  // Autocomplete will trigger a style recalculation when we put up the next
+  // frame, but we don't want to wait that long. Instead, trigger a style
+  // recalcuation manually after TextFieldDidChangeImpl runs.
+  base::MessageLoop::current()->PostTask(
+      FROM_HERE,
+      base::Bind(&RenderViewTest::LayoutMainFrame, base::Unretained(this)));
+
+  base::MessageLoop::current()->RunUntilIdle();
+}
+
+void RenderViewTest::SimulateUserInputChangeForElement(
+    blink::WebInputElement* input,
+    const std::string& new_value) {
+  while (!input->focused())
+    input->document().frame()->view()->advanceFocus(false);

[Jay Civelli, 2015/04/23 21:57:43]

Wouldn't simulating a click on the input element achieve the same result in a
more straight-forward way?

[please use gerrit instead, 2015/04/27 21:20:34]

I tried that and for some reason focus does not change in Blink. I'm not sure
what's at play here, but I'm leaning toward keeping that code as is, because it
remains after copying it from
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/renderer/au....
What do you think?

[Jay Civelli, 2015/04/27 22:17:19]

Sounds good.

+
+  size_t previous_length = input->value().length();
+  for (size_t i = 0; i < previous_length; ++i)
+    SimulateUserTypingASCIICharacter(ui::VKEY_BACK);
+
+  EXPECT_TRUE(input->value().utf8().empty());
+  ASSERT_TRUE(base::IsStringASCII(new_value));

[Jay Civelli, 2015/04/23 21:57:43]

Nit: this assert should probably be moved to the top of the method.

[please use gerrit instead, 2015/04/27 21:20:34]

Done.

+  for (size_t i = 0; i < new_value.size(); ++i)
+    SimulateUserTypingASCIICharacter(new_value[i]);
+
+  // Compare only beginning, because autocomplete may have filled out the
+  // form.
+  EXPECT_EQ(new_value, input->value().utf8().substr(0, new_value.length()));
+
+  ProcessInputForAutofill();
+}
+
 bool RenderViewTest::OnMessageReceived(const IPC::Message& msg) {
   RenderViewImpl* impl = static_cast<RenderViewImpl*>(view_);
   return impl->OnMessageReceived(msg);
 }
 
 void RenderViewTest::DidNavigateWithinPage(blink::WebLocalFrame* frame,
                                            bool is_new_navigation) {
   RenderViewImpl* impl = static_cast<RenderViewImpl*>(view_);
   blink::WebHistoryItem item;
   item.initialize();
@@ skipping 50 matching lines @@
   request_params.current_history_list_length = history_list_length;
 
   impl->GetMainRenderFrame()->OnNavigate(common_params, StartNavigationParams(),
                                          request_params);
 
   // The load actually happens asynchronously, so we pump messages to process
   // the pending continuation.
   FrameLoadWaiter(view_->GetMainRenderFrame()).Wait();
 }
 
+void RenderViewTest::LayoutMainFrame() {
+  GetMainFrame()->view()->layout();
+}
+
 }  // namespace content