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

Issue 11299285: events: Dispatch all events through EventHandler::OnEvent from the dispatcher. (Closed)

Created:
8 years ago by sadrul
Modified:
8 years ago
CC:
chromium-reviews
Visibility:
Public.

Description

events: Dispatch all events through EventHandler::OnEvent from the dispatcher. The EventDispatcher dispatches all events to EventHandler::OnEvent. The default implementation routes various events to more appropriate handlers (e.g. OnKeyEvent for KeyEvent etc.) BUG=163606 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=170641 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=170656

Patch Set 1 #

Patch Set 2 : self-nit #

Patch Set 3 : mac-dbg-build-fix #

Patch Set 4 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+25 lines, -36 lines) Patch
M ui/base/events/event.h View 2 chunks +3 lines, -1 line 0 comments Download
M ui/base/events/event_dispatcher.h View 1 chunk +0 lines, -10 lines 0 comments Download
M ui/base/events/event_dispatcher.cc View 1 chunk +0 lines, -25 lines 0 comments Download
M ui/base/events/event_handler.h View 1 chunk +4 lines, -0 lines 0 comments Download
M ui/base/events/event_handler.cc View 1 2 chunks +12 lines, -0 lines 0 comments Download
M ui/ui.gyp View 1 2 3 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
sadrul
8 years ago (2012-11-30 22:55:43 UTC) #1
Ben Goodger (Google)
8 years ago (2012-11-30 23:03:32 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698