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

Side by Side Diff: chrome/browser/chromeos/events/event_rewriter_controller.cc

Issue 240173007: Convert KeyboardDrivenEventRewriter to a ui::EventRewriter. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 8 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "chrome/browser/chromeos/events/event_rewriter_controller.h"
6
7 #include "ash/shell.h"
8 #include "ui/aura/window_tree_host.h"
9 #include "ui/events/event_source.h"
10
11 namespace chromeos {
12
13 EventRewriterController::EventRewriterController(ui::EventRewriter* rewriter)
14 : rewriter_(rewriter) {
15 }
16
17 EventRewriterController::~EventRewriterController() {
18 }
19
20 void EventRewriterController::Init() {
21 // Add the rewriter to each existing root window EventSource.
22 aura::Window::Windows windows = ash::Shell::GetAllRootWindows();
23 for (aura::Window::Windows::iterator i = windows.begin(), e = windows.end();
Daniel Erat 2014/04/18 02:50:06 nit: i haven't seen this style used in chromium be
24 i != e;
25 ++i) {
26 AddToEventSource((*i)->GetHost()->GetEventSource());
Daniel Erat 2014/04/18 02:50:06 do you need to unregister it later?
27 }
28 }
29
30 void EventRewriterController::OnHostInitialized(aura::WindowTreeHost* host) {
31 AddToEventSource(host->GetEventSource());
32 }
33
34 void EventRewriterController::AddToEventSource(ui::EventSource* source) {
35 DCHECK(source);
36 event_sources_.push_back(source);
37 source->AddEventRewriter(rewriter_.get());
Daniel Erat 2014/04/18 02:50:06 do you need to unregister it later?
38 }
39
40 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698