WebMouseEvent coordinates are now fractional & private

chrome/browser/site_per_process_interactive_browsertest.cc

 // Copyright 2016 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 "base/command_line.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/test/scoped_feature_list.h"
 #include "base/test/test_timeouts.h"
 #include "chrome/browser/password_manager/chrome_password_manager_client.h"
 #include "chrome/browser/renderer_context_menu/render_view_context_menu_browsertest_util.h"
@@ skipping 1082 matching lines @@
   content::RenderWidgetHostView* child_view =
       ChildFrameAt(active_web_contents->GetMainFrame(), 0)->GetView();
 
   ContextMenuWaiter menu_waiter(content::NotificationService::AllSources());
 
   // Declaring a lambda to send a right-button mouse event to the embedder
   // frame.
   auto send_right_mouse_event = [](content::RenderWidgetHost* host, int x,
                                    int y, blink::WebInputEvent::Type type) {
     blink::WebMouseEvent event;
-    event.x = x;
-    event.y = y;
+    event.setPositionInWidget(x, y);
     event.button = blink::WebMouseEvent::Button::Right;
     event.setType(type);
     host->ForwardMouseEvent(event);
   };
 
   send_right_mouse_event(child_view->GetRenderWidgetHost(), 10, 20,
                          blink::WebInputEvent::MouseDown);
   send_right_mouse_event(child_view->GetRenderWidgetHost(), 10, 20,
                          blink::WebInputEvent::MouseUp);
   menu_waiter.WaitForMenuOpenAndClose();
 
   gfx::Point point_in_root_window =
       child_view->TransformPointToRootCoordSpace(gfx::Point(10, 20));
 
   EXPECT_EQ(point_in_root_window.x(), menu_waiter.params().x);
   EXPECT_EQ(point_in_root_window.y(), menu_waiter.params().y);
 }
 
 class SitePerProcessAutofillTest : public SitePerProcessInteractiveBrowserTest {
  public:
   SitePerProcessAutofillTest() : SitePerProcessInteractiveBrowserTest() {}
-  ~SitePerProcessAutofillTest() override{};
+  ~SitePerProcessAutofillTest() override {}
 
   void SetUpCommandLine(base::CommandLine* command_line) override {
     SitePerProcessInteractiveBrowserTest::SetUpCommandLine(command_line);
     // We need to set the feature state before the render process is created,
     // in order for it to inherit the feature state from the browser process.
     // SetUp() runs too early, and SetUpOnMainThread() runs too late.
     scoped_feature_list_.InitAndEnableFeature(
         security_state::kHttpFormWarningFeature);
   }
 
  protected:
   class TestAutofillClient : public autofill::TestAutofillClient {
    public:
-    TestAutofillClient() : popup_shown_(false){};
+    TestAutofillClient() : popup_shown_(false) {}
     ~TestAutofillClient() override {}
 
     void WaitForNextPopup() {
       if (popup_shown_)
         return;
       loop_runner_ = new content::MessageLoopRunner();
       loop_runner_->Run();
     }
 
     void ShowAutofillPopup(
@@ skipping 182 matching lines @@
 
   // Ideally, the length of the error vector should be 0.0f. But due to
   // potential rounding errors, we assume a larger limit (which is slightly
   // larger than square root of 2).
   EXPECT_LT(error.Length(), 1.4143f)
       << "Origin of bounds from focused node changed event is '"
       << focus_observer.focused_node_bounds_in_screen().ToString()
       << "' but AutofillClient is reporting '" << bounds_origin.ToString()
       << "'";
 }