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

Issue 113283005: aura: Start using EventPrcessor interface for event dispatch. (Closed)

Created:
7 years ago by sadrul
Modified:
7 years ago
CC:
chromium-reviews, ben+aura_chromium.org, kalyank, tdanderson
Visibility:
Public.

Description

aura: Start using EventPrcessor interface for event dispatch. Start converting the RootWindowHostDelegate implementations for event-dispatch use the EventProcessor interface under-the-hood to allow easy transition to the new dispatch process. This patch makes this change for Key and Scroll events. BUG=318879 R=ben@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=240683

Patch Set 1 #

Patch Set 2 : . #

Patch Set 3 : . #

Patch Set 4 : . #

Patch Set 5 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+125 lines, -52 lines) Patch
M ui/aura/root_window.h View 4 chunks +6 lines, -1 line 0 comments Download
M ui/aura/root_window.cc View 1 2 3 4 13 chunks +55 lines, -47 lines 0 comments Download
M ui/aura/root_window_unittest.cc View 1 chunk +2 lines, -3 lines 0 comments Download
M ui/aura/window_targeter.h View 2 chunks +9 lines, -0 lines 0 comments Download
M ui/aura/window_targeter.cc View 1 2 3 2 chunks +36 lines, -0 lines 0 comments Download
M ui/events/event_dispatcher.cc View 1 2 1 chunk +5 lines, -1 line 0 comments Download
M ui/events/event_processor.h View 1 2 3 1 chunk +8 lines, -0 lines 0 comments Download
M ui/events/event_processor.cc View 2 chunks +4 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
sadrul
7 years ago (2013-12-12 01:14:10 UTC) #1
sadrul
The similar change for touch-events is at: https://codereview.chromium.org/110483004/
7 years ago (2013-12-13 03:59:28 UTC) #2
Ben Goodger (Google)
lgtm
7 years ago (2013-12-13 06:33:33 UTC) #3
sadrul
7 years ago (2013-12-13 17:17:46 UTC) #4
Message was sent while issue was closed.
Committed patchset #5 manually as r240683 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698