Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4414)

Unified Diff: chrome/browser/site_per_process_interactive_browsertest.cc

Issue 2782893002: WebMouseEvent coordinates are now fractional & private (Closed)
Patch Set: Rebased, fixed a comment in web_input_event_builders_mac.mm Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/site_per_process_interactive_browsertest.cc
diff --git a/chrome/browser/site_per_process_interactive_browsertest.cc b/chrome/browser/site_per_process_interactive_browsertest.cc
index eb7b3a26c4a95be515748c9899f08e72b60173aa..a70957281afadf435b28004b1bac835c89df0a7e 100644
--- a/chrome/browser/site_per_process_interactive_browsertest.cc
+++ b/chrome/browser/site_per_process_interactive_browsertest.cc
@@ -1100,8 +1100,7 @@ IN_PROC_BROWSER_TEST_F(SitePerProcessInteractivePDFTest,
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);
@@ -1123,7 +1122,7 @@ IN_PROC_BROWSER_TEST_F(SitePerProcessInteractivePDFTest,
class SitePerProcessAutofillTest : public SitePerProcessInteractiveBrowserTest {
public:
SitePerProcessAutofillTest() : SitePerProcessInteractiveBrowserTest() {}
- ~SitePerProcessAutofillTest() override{};
+ ~SitePerProcessAutofillTest() override {}
void SetUpCommandLine(base::CommandLine* command_line) override {
SitePerProcessInteractiveBrowserTest::SetUpCommandLine(command_line);
@@ -1137,7 +1136,7 @@ class SitePerProcessAutofillTest : public SitePerProcessInteractiveBrowserTest {
protected:
class TestAutofillClient : public autofill::TestAutofillClient {
public:
- TestAutofillClient() : popup_shown_(false){};
+ TestAutofillClient() : popup_shown_(false) {}
~TestAutofillClient() override {}
void WaitForNextPopup() {

Powered by Google App Engine
This is Rietveld 408576698