Chromium Code Reviews| Index: ui/events/event_handler.cc |
| diff --git a/ui/events/event_handler.cc b/ui/events/event_handler.cc |
| index 0f79c3556fb477e3cffda4d75920fe9cc767101b..66f1edd48aa0759467cd191ad8573b780e9b008a 100644 |
| --- a/ui/events/event_handler.cc |
| +++ b/ui/events/event_handler.cc |
| @@ -21,16 +21,14 @@ EventHandler::~EventHandler() { |
| } |
| void EventHandler::OnEvent(Event* event) { |
| - // TODO(tdanderson): Encapsulate static_casts in ui::Event for all |
| - // event types. |
|
kylix_rd
2016/09/29 18:12:19
I noticed this comment and had a couple of cycles
|
| if (event->IsKeyEvent()) |
| - OnKeyEvent(static_cast<KeyEvent*>(event)); |
| + OnKeyEvent(event->AsKeyEvent()); |
| else if (event->IsMouseEvent()) |
| - OnMouseEvent(static_cast<MouseEvent*>(event)); |
| + OnMouseEvent(event->AsMouseEvent()); |
| else if (event->IsScrollEvent()) |
| - OnScrollEvent(static_cast<ScrollEvent*>(event)); |
| + OnScrollEvent(event->AsScrollEvent()); |
| else if (event->IsTouchEvent()) |
| - OnTouchEvent(static_cast<TouchEvent*>(event)); |
| + OnTouchEvent(event->AsTouchEvent()); |
| else if (event->IsGestureEvent()) |
| OnGestureEvent(event->AsGestureEvent()); |
| else if (event->type() == ET_CANCEL_MODE) |