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

Unified Diff: chrome/browser/ui/views/autofill/autofill_dialog_views.cc

Issue 12088015: Add ability for EventGenerator to generate Scroll events asynchronously (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 11 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/ui/views/autofill/autofill_dialog_views.cc
diff --git a/chrome/browser/ui/views/autofill/autofill_dialog_views.cc b/chrome/browser/ui/views/autofill/autofill_dialog_views.cc
index d307887a49f4f836b795c6c607383283fe51fa1a..aabe094279f9b5de6ac9b462584a8d9dcedb92c6 100644
--- a/chrome/browser/ui/views/autofill/autofill_dialog_views.cc
+++ b/chrome/browser/ui/views/autofill/autofill_dialog_views.cc
@@ -166,7 +166,9 @@ void AutofillDialogViews::SectionContainer::OnMouseEntered(
// TODO(estade): use the correct color.
set_background(views::Background::CreateSolidBackground(SK_ColorLTGRAY));
- proxy_button_->OnMouseEntered(ProxyEvent(event));
+ ui::MouseEvent event_copy(event);
sky 2013/01/28 19:10:55 Is there a reason for doing these changes? Seems l
DaveMoore 2013/01/28 23:25:20 When I made the MouseEvent() constructor explicit
+ event_copy.set_location(gfx::Point());
+ proxy_button_->OnMouseEntered(event_copy);
SchedulePaint();
}
@@ -176,7 +178,9 @@ void AutofillDialogViews::SectionContainer::OnMouseExited(
return;
set_background(NULL);
- proxy_button_->OnMouseExited(ProxyEvent(event));
+ ui::MouseEvent event_copy(event);
+ event_copy.set_location(gfx::Point());
+ proxy_button_->OnMouseExited(event_copy);
SchedulePaint();
}
@@ -185,7 +189,9 @@ bool AutofillDialogViews::SectionContainer::OnMousePressed(
if (!forward_mouse_events_)
return false;
- return proxy_button_->OnMousePressed(ProxyEvent(event));
+ ui::MouseEvent event_copy(event);
+ event_copy.set_location(gfx::Point());
+ return proxy_button_->OnMousePressed(event_copy);
}
void AutofillDialogViews::SectionContainer::OnMouseReleased(
@@ -193,15 +199,9 @@ void AutofillDialogViews::SectionContainer::OnMouseReleased(
if (!forward_mouse_events_)
return;
- proxy_button_->OnMouseReleased(ProxyEvent(event));
-}
-
-// static
-ui::MouseEvent AutofillDialogViews::SectionContainer::ProxyEvent(
- const ui::MouseEvent& event) {
- ui::MouseEvent event_copy = event;
+ ui::MouseEvent event_copy(event);
event_copy.set_location(gfx::Point());
- return event_copy;
+ proxy_button_->OnMouseReleased(event_copy);
}
// AutofilDialogViews::SuggestionView ------------------------------------------

Powered by Google App Engine
This is Rietveld 408576698