Chromium Code Reviews| 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 ------------------------------------------ |