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

Unified Diff: ui/chromeos/touch_exploration_controller.h

Issue 330763007: Swipe Gestures for Accessibility (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@gesture-vlogs
Patch Set: Removed line that Evy was working on Created 6 years, 6 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 side-by-side diff with in-line comments
Download patch
Index: ui/chromeos/touch_exploration_controller.h
diff --git a/ui/chromeos/touch_exploration_controller.h b/ui/chromeos/touch_exploration_controller.h
index aa099282d449acfbe89e5e9401239a70d41f1bb6..df496e79af40cfc19e931ab751828b83c9bb607f 100644
--- a/ui/chromeos/touch_exploration_controller.h
+++ b/ui/chromeos/touch_exploration_controller.h
@@ -11,6 +11,7 @@
#include "ui/events/event.h"
#include "ui/events/event_rewriter.h"
#include "ui/events/gesture_detection/gesture_detector.h"
+#include "ui/events/gestures/gesture_provider_aura.h"
#include "ui/gfx/geometry/point.h"
namespace aura {
@@ -21,10 +22,13 @@ namespace ui {
class Event;
class EventHandler;
+class GestureEvent;
+class GestureProviderAura;
class TouchEvent;
// TouchExplorationController is used in tandem with "Spoken Feedback" to
-// make the touch UI accessible.
+// make the touch UI accessible. Gestures are mapped to accessible key
tdresser 2014/06/26 13:31:49 Should "accessible" be "accessibility"?
+// shortcuts.
//
// ** Short version **
//
@@ -32,7 +36,8 @@ class TouchEvent;
// exploring the screen gets turned into mouse moves (which can then be
// spoken by an accessibility service running), a single tap while the user
// is in touch exploration or a double-tap simulates a click, and gestures
-// can be used to send high-level accessibility commands.
+// can be used to send high-level accessibility commands. For example, a swipe
+// right would correspond to the keyboard short cut shift+search+right.
// When two or more fingers are pressed initially, from then on the events
// are passed through, but with the initial finger removed - so if you swipe
// down with two fingers, the running app will see a one-finger swipe.
@@ -61,6 +66,13 @@ class TouchEvent;
// location. This allows the user to perform a single tap
// anywhere to activate it.
//
+// The user can perform swipe gestures in one of the four cardinal directions
+// which will be interpreted and used to control the UI. The gesture will only
+// be registered if the finger moves outside the slop and completed within the
+// grace period. If the gesture fails to be completed within the grace period,
+// the state changes to touch exploration mode and any gestures made during the
+// grace period are discarded.
+//
// If the user double-taps, the second tap is passed through, allowing the
// user to click - however, the double-tap location is changed to the location
// of the last successful touch exploration - that allows the user to explore
@@ -85,8 +97,9 @@ class TouchEvent;
//
// The caller is expected to retain ownership of instances of this class and
// destroy them before |root_window| is destroyed.
-class UI_CHROMEOS_EXPORT TouchExplorationController :
- public ui::EventRewriter {
+class UI_CHROMEOS_EXPORT TouchExplorationController
+ : public ui::EventRewriter,
+ public ui::GestureProviderAuraClient {
public:
explicit TouchExplorationController(aura::Window* root_window);
virtual ~TouchExplorationController();
@@ -94,6 +107,7 @@ class UI_CHROMEOS_EXPORT TouchExplorationController :
void CallTapTimerNowForTesting();
void SetEventHandlerForTesting(ui::EventHandler* event_handler_for_testing);
bool IsInNoFingersDownStateForTesting() const;
+ bool IsInGestureInProgressStateForTesting() const;
private:
// Overridden from ui::EventRewriter
@@ -114,10 +128,14 @@ class UI_CHROMEOS_EXPORT TouchExplorationController :
const ui::TouchEvent& event, scoped_ptr<ui::Event>* rewritten_event);
ui::EventRewriteStatus InTouchExploration(
const ui::TouchEvent& event, scoped_ptr<ui::Event>* rewritten_event);
+ ui::EventRewriteStatus InGestureInProgress(
+ const ui::TouchEvent& event,
+ scoped_ptr<ui::Event>* rewritten_event);
ui::EventRewriteStatus InPassthroughMinusOne(
const ui::TouchEvent& event, scoped_ptr<ui::Event>* rewritten_event);
ui::EventRewriteStatus InTouchExploreSecondPress(
const ui::TouchEvent& event, scoped_ptr<ui::Event>* rewritten_event);
+
// This timer is started every time we get the first press event, and
// it fires after the double-click timeout elapses (300 ms by default).
// If the user taps and releases within 300 ms and doesn't press again,
@@ -127,6 +145,20 @@ class UI_CHROMEOS_EXPORT TouchExplorationController :
// Dispatch a new event outside of the event rewriting flow.
void DispatchEvent(ui::Event* event);
+ // Overridden from GestureProviderAuraClient.
+ //
+ // The gesture provider keeps track of all the touch events after
+ // the user moves fast enough to trigger a gesture. After the user
+ // completes their gesture, this method will decide what keyboard
+ // input their gesture corresponded to.
+ void OnGestureEvent(ui::GestureEvent* gesture) OVERRIDE;
+
+ void OnSwipeEvent(ui::GestureEvent* swipe_gesture);
+
+ // Dispatches the keyboard short cut Shift+Search+<arrow key>
+ // outside the event rewritting flow.
+ void DispatchShiftSearchKeyEvent(const ui::KeyboardCode direction);
+
scoped_ptr<ui::Event> CreateMouseMoveEvent(const gfx::PointF& location,
int flags);
@@ -169,6 +201,15 @@ class UI_CHROMEOS_EXPORT TouchExplorationController :
// mode until all fingers are lifted.
TOUCH_EXPLORATION,
+ // If the user moves their finger faster than the threshold velocity after a
+ // single tap, the touch events that follow will be translated into gesture
+ // events. If the user successfully completes a gesture within the grace
+ // period, the gesture will be interpreted and used to control the UI via
+ // discrete actions - currently by synthesizing key events corresponding to
+ // each gesture Otherwise, the collected gestures are discarded and the
+ // state changes to touch_exploration.
+ GESTURE_IN_PROGRESS,
+
// The user placed two or more fingers down within the grace period.
// We're now in passthrough mode until all fingers are lifted. Initially
// the first finger is ignored and other fingers are passed through
@@ -231,6 +272,9 @@ class UI_CHROMEOS_EXPORT TouchExplorationController :
// timeout and pixel slop constants.
ui::GestureDetector::Config gesture_detector_config_;
+ // Gesture Handler to interpret the touch events.
+ ui::GestureProviderAura gesture_provider_;
+
// The previous state entered.
State prev_state_;
« no previous file with comments | « no previous file | ui/chromeos/touch_exploration_controller.cc » ('j') | ui/chromeos/touch_exploration_controller_unittest.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698