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

Unified Diff: components/test_runner/event_sender.cc

Issue 2669663002: Move touch slop suppression to TouchEventManager (Closed)
Patch Set: slop region Created 3 years, 10 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: components/test_runner/event_sender.cc
diff --git a/components/test_runner/event_sender.cc b/components/test_runner/event_sender.cc
index 7f38826f9b52e91b910257135d2fa7d2086bdb54..97c4ac2efb0ecfe366eb8b05b8307df5b524d957 100644
--- a/components/test_runner/event_sender.cc
+++ b/components/test_runner/event_sender.cc
@@ -2239,11 +2239,11 @@ void EventSender::DoLeapForward(int milliseconds) {
void EventSender::GetOptionalTouchArgs(gin::Arguments* args,
bool& moved_beyond_slop_region,
uint32_t& unique_touch_event_id) {
- moved_beyond_slop_region = false;
+ moved_beyond_slop_region = true;
if(!args->PeekNext().IsEmpty() && args->PeekNext()->IsString()) {
std::string arg;
- if (args->GetNext(&arg) && arg == "movedBeyondSlopRegion")
- moved_beyond_slop_region = true;
+ if (args->GetNext(&arg) && arg == "movedInsideSlopRegion")
tdresser 2017/02/06 14:25:59 Can we just get rid of this? It's never used.
lanwei 2017/02/06 21:36:23 Done.
+ moved_beyond_slop_region = false;
else
args->ThrowError();
}

Powered by Google App Engine
This is Rietveld 408576698