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

Side by Side Diff: ash/touch/touch_exploration_controller.h

Issue 240333007: wip: Second crack at implementing the touch exploration mode. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Don't extend DisplayController::Observer - TouchExplorationController is owned by the root window (… 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 #ifndef ASH_TOUCH_TOUCH_EXPLORATION_CONTROLLER_H_
6 #define ASH_TOUCH_TOUCH_EXPLORATION_CONTROLLER_H_
7
8 #include "ash/ash_export.h"
9 #include "base/values.h"
10 #include "ui/aura/window.h"
11 #include "ui/events/event.h"
12 #include "ui/events/event_rewriter.h"
13
14 namespace ash {
15
16 // TODO: How much screen/display logic (DisplayController::Observer,
Daniel Erat 2014/04/18 00:44:08 nit: please include your username in these TODOs (
17 // gfx::DisplayObserver, etc) do we need here? This is managed from
18 // RootWindowController
19 // TODO: Maybe implement gfx::DisplayObserver
20 // TODO: Do we need to inherit from OutputConfigurator::Observer to react to
21 // OnDisplayModeChanged()?
sadrul 2014/04/22 16:46:12 Depends on what you are trying to do. If you want
mfomitchev 2014/04/23 18:27:37 After thinking about this, I don't think I need to
22 class ASH_EXPORT TouchExplorationController :
23 public ui::EventRewriter {
24 public:
25
Daniel Erat 2014/04/18 00:44:08 delete extra blank line
26 explicit TouchExplorationController(aura::Window* root_window);
27 virtual ~TouchExplorationController();
28
29 private:
30 //Overridden from ui::EventRewriter
Daniel Erat 2014/04/18 00:44:08 add a space after '//'
31 virtual ui::EventRewriteStatus RewriteEvent(
32 const ui::Event& event, scoped_ptr<ui::Event>* rewritten_event) OVERRIDE;
33 virtual ui::EventRewriteStatus NextDispatchEvent(
34 const ui::Event& last_event, scoped_ptr<ui::Event>* new_event) OVERRIDE;
35
36 ui::Event* CreateMouseMoveEvent(gfx::Point location, int flags);
Daniel Erat 2014/04/18 00:44:08 add a comment describing what this does
37
38 void EnterMouseMoveMode();
Daniel Erat 2014/04/18 00:44:08 add a comment
39
40 std::vector<int> touch_ids_;
41 // map of touch ids to their last know locations
Daniel Erat 2014/04/18 00:44:08 s/map/Map/, s/know/known/, add trailing period
42 std::map<int, gfx::Point> touch_locations_;
43
44 // Initialized from RewriteEvent() and dispatched in NextDispatchEvent().
45 scoped_ptr<ui::Event> next_dispatch_event_;
46
47 const int64 display_id_;
48 aura::Window* root_window_;
49
50 DISALLOW_COPY_AND_ASSIGN(TouchExplorationController);
51 };
52
53 } // namespace ash
54
55 #endif // ASH_TOUCH_TOUCH_EXPLORATION_CONTROLLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698