Convert KeyboardDrivenEventRewriter to a ui::EventRewriter.

chrome/browser/chromeos/events/event_rewriter_controller.h

+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_CHROMEOS_EVENTS_EVENT_REWRITER_CONTROLLER_H_
+#define CHROME_BROWSER_CHROMEOS_EVENTS_EVENT_REWRITER_CONTROLLER_H_
+
+#include <list>
+
+#include "ui/aura/env_observer.h"
+#include "ui/events/event_rewriter.h"
+
+namespace ui {
+class EventSource;
+}  // namespace ui
+
+namespace chromeos {
+
+class EventRewriterController : public aura::EnvObserver {

[sadrul, 2014/04/22 16:07:04]

Add doc comments

+ public:
+  explicit EventRewriterController(ui::EventRewriter* rewriter);

[Daniel Erat, 2014/04/18 02:50:06]

change the argument to a scoped_ptr to make it obvious that ownership is being
transferred (and then use .Pass() at the calling site)

+  ~EventRewriterController();

[Daniel Erat, 2014/04/18 02:50:06]

virtual

+
+  void Init();
+
+  // aura::EnvObserver overrides:
+  virtual void OnWindowInitialized(aura::Window* window) OVERRIDE {}
+  virtual void OnHostInitialized(aura::WindowTreeHost* host) OVERRIDE;
+
+ private:
+  typedef std::list<ui::EventSource*> EventSourceList;
+
+  void AddToEventSource(ui::EventSource* source);
+
+  // The |EventRewriter| managed by this controller.
+  scoped_ptr<ui::EventRewriter> rewriter_;
+
+  // |EventSource|s to which the rewriter has been added.
+  EventSourceList event_sources_;

[sadrul, 2014/04/22 16:07:04]

I assume you keep this list so that you can remove the rewriter when needed. But
do you need to keep track of this? Can you just iterate over the root-windows in
Shell and remove from each instead?

+
+  DISALLOW_COPY_AND_ASSIGN(EventRewriterController);
+};
+
+}  // namespace chromeos
+
+#endif  // CHROME_BROWSER_CHROMEOS_EVENTS_EVENT_REWRITER_CONTROLLER_H_