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

Side by Side Diff: content/browser/renderer_host/input/mouse_wheel_event_queue_unittest.cc

Issue 2486673008: Touchpad scroll latching enabled for Mac behind flag. (Closed)
Patch Set: GSE/GSB are not sent when a scroll in inertial phase begins. Created 4 years 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
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/renderer_host/input/mouse_wheel_event_queue.h" 5 #include "content/browser/renderer_host/input/mouse_wheel_event_queue.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <memory> 9 #include <memory>
10 #include <utility> 10 #include <utility>
(...skipping 300 matching lines...) Expand 10 before | Expand all | Expand 10 after
311 const WebGestureEvent::ScrollUnits scroll_units = 311 const WebGestureEvent::ScrollUnits scroll_units =
312 high_precision ? WebGestureEvent::PrecisePixels 312 high_precision ? WebGestureEvent::PrecisePixels
313 : WebGestureEvent::Pixels; 313 : WebGestureEvent::Pixels;
314 314
315 SendMouseWheelWithPhase(kWheelScrollX, kWheelScrollY, kWheelScrollGlobalX, 315 SendMouseWheelWithPhase(kWheelScrollX, kWheelScrollY, kWheelScrollGlobalX,
316 kWheelScrollGlobalY, 1, 1, 0, high_precision, 316 kWheelScrollGlobalY, 1, 1, 0, high_precision,
317 WebMouseWheelEvent::PhaseBegan, 317 WebMouseWheelEvent::PhaseBegan,
318 WebMouseWheelEvent::PhaseNone); 318 WebMouseWheelEvent::PhaseNone);
319 EXPECT_EQ(1U, GetAndResetSentEventCount()); 319 EXPECT_EQ(1U, GetAndResetSentEventCount());
320 SendMouseWheelEventAck(INPUT_EVENT_ACK_STATE_NOT_CONSUMED); 320 SendMouseWheelEventAck(INPUT_EVENT_ACK_STATE_NOT_CONSUMED);
321 EXPECT_EQ(3U, all_sent_events().size()); 321 if (scroll_latching_enabled_) {
322 EXPECT_GESTURE_SCROLL_BEGIN_WITH_PHASE(sent_gesture_event(0)); 322 EXPECT_EQ(2U, all_sent_events().size());
323 EXPECT_GESTURE_SCROLL_UPDATE_WITH_PHASE(sent_gesture_event(1)); 323 EXPECT_GESTURE_SCROLL_BEGIN_WITH_PHASE(sent_gesture_event(0));
324 EXPECT_SYNTHETIC_GESTURE_SCROLL_END(sent_gesture_event(2)); 324 EXPECT_GESTURE_SCROLL_UPDATE_WITH_PHASE(sent_gesture_event(1));
325 EXPECT_EQ(3U, GetAndResetSentEventCount()); 325 EXPECT_EQ(2U, GetAndResetSentEventCount());
326 } else {
327 EXPECT_EQ(3U, all_sent_events().size());
328 EXPECT_GESTURE_SCROLL_BEGIN_WITH_PHASE(sent_gesture_event(0));
329 EXPECT_GESTURE_SCROLL_UPDATE_WITH_PHASE(sent_gesture_event(1));
330 EXPECT_SYNTHETIC_GESTURE_SCROLL_END(sent_gesture_event(2));
331 EXPECT_EQ(3U, GetAndResetSentEventCount());
332 }
326 333
327 SendMouseWheelWithPhase(kWheelScrollX, kWheelScrollY, kWheelScrollGlobalX, 334 SendMouseWheelWithPhase(kWheelScrollX, kWheelScrollY, kWheelScrollGlobalX,
328 kWheelScrollGlobalY, 5, 5, 0, high_precision, 335 kWheelScrollGlobalY, 5, 5, 0, high_precision,
329 WebMouseWheelEvent::PhaseChanged, 336 WebMouseWheelEvent::PhaseChanged,
330 WebMouseWheelEvent::PhaseNone); 337 WebMouseWheelEvent::PhaseNone);
331 EXPECT_EQ(1U, GetAndResetSentEventCount()); 338 EXPECT_EQ(1U, GetAndResetSentEventCount());
332 SendMouseWheelEventAck(INPUT_EVENT_ACK_STATE_NOT_CONSUMED); 339 SendMouseWheelEventAck(INPUT_EVENT_ACK_STATE_NOT_CONSUMED);
333 EXPECT_EQ(3U, all_sent_events().size()); 340 if (scroll_latching_enabled_) {
334 EXPECT_SYNTHETIC_GESTURE_SCROLL_BEGIN(sent_gesture_event(0)); 341 EXPECT_EQ(1U, all_sent_events().size());
335 EXPECT_GESTURE_SCROLL_UPDATE_WITH_PHASE(sent_gesture_event(1)); 342 EXPECT_GESTURE_SCROLL_UPDATE_WITH_PHASE(sent_gesture_event(0));
336 EXPECT_SYNTHETIC_GESTURE_SCROLL_END(sent_gesture_event(2)); 343 EXPECT_EQ(1U, GetAndResetSentEventCount());
337 EXPECT_EQ(3U, GetAndResetSentEventCount()); 344 } else {
345 EXPECT_EQ(3U, all_sent_events().size());
346 EXPECT_SYNTHETIC_GESTURE_SCROLL_BEGIN(sent_gesture_event(0));
347 EXPECT_GESTURE_SCROLL_UPDATE_WITH_PHASE(sent_gesture_event(1));
348 EXPECT_SYNTHETIC_GESTURE_SCROLL_END(sent_gesture_event(2));
349 EXPECT_EQ(3U, GetAndResetSentEventCount());
350 }
338 351
339 SendMouseWheelWithPhase(kWheelScrollX, kWheelScrollY, kWheelScrollGlobalX, 352 SendMouseWheelWithPhase(kWheelScrollX, kWheelScrollY, kWheelScrollGlobalX,
340 kWheelScrollGlobalY, 0, 0, 0, high_precision, 353 kWheelScrollGlobalY, 0, 0, 0, high_precision,
341 WebMouseWheelEvent::PhaseEnded, 354 WebMouseWheelEvent::PhaseEnded,
342 WebMouseWheelEvent::PhaseNone); 355 WebMouseWheelEvent::PhaseNone);
343 EXPECT_EQ(1U, GetAndResetSentEventCount()); 356 EXPECT_EQ(1U, GetAndResetSentEventCount());
344 SendMouseWheelEventAck(INPUT_EVENT_ACK_STATE_NOT_CONSUMED); 357 SendMouseWheelEventAck(INPUT_EVENT_ACK_STATE_NOT_CONSUMED);
345 EXPECT_EQ(2U, all_sent_events().size()); 358 if (scroll_latching_enabled_) {
346 EXPECT_SYNTHETIC_GESTURE_SCROLL_BEGIN(sent_gesture_event(0)); 359 // ScrollEnd is not sent right away, a timer starts instead,
347 EXPECT_GESTURE_SCROLL_END_WITH_PHASE(sent_gesture_event(1)); 360 // to see if a wheel event with momentumPhase arrives or not.
348 EXPECT_EQ(2U, GetAndResetSentEventCount()); 361 EXPECT_EQ(0U, all_sent_events().size());
362 EXPECT_EQ(0U, GetAndResetSentEventCount());
363 } else {
364 EXPECT_EQ(2U, all_sent_events().size());
365 EXPECT_SYNTHETIC_GESTURE_SCROLL_BEGIN(sent_gesture_event(0));
366 EXPECT_GESTURE_SCROLL_END_WITH_PHASE(sent_gesture_event(1));
367 EXPECT_EQ(2U, GetAndResetSentEventCount());
368 }
349 369
350 // Send a double phase end; OSX does it consistently. 370 // Send a double phase end; OSX does it consistently.
351 SendMouseWheelWithPhase(kWheelScrollX, kWheelScrollY, kWheelScrollGlobalX, 371 SendMouseWheelWithPhase(kWheelScrollX, kWheelScrollY, kWheelScrollGlobalX,
352 kWheelScrollGlobalY, 0, 0, 0, high_precision, 372 kWheelScrollGlobalY, 0, 0, 0, high_precision,
353 WebMouseWheelEvent::PhaseEnded, 373 WebMouseWheelEvent::PhaseEnded,
354 WebMouseWheelEvent::PhaseNone); 374 WebMouseWheelEvent::PhaseNone);
355 EXPECT_EQ(1U, GetAndResetSentEventCount()); 375 EXPECT_EQ(1U, GetAndResetSentEventCount());
356 SendMouseWheelEventAck(INPUT_EVENT_ACK_STATE_NOT_CONSUMED); 376 SendMouseWheelEventAck(INPUT_EVENT_ACK_STATE_NOT_CONSUMED);
357 EXPECT_EQ(0U, all_sent_events().size()); 377 EXPECT_EQ(0U, all_sent_events().size());
358 EXPECT_EQ(0U, GetAndResetSentEventCount()); 378 EXPECT_EQ(0U, GetAndResetSentEventCount());
359 379
360 SendMouseWheelWithPhase(kWheelScrollX, kWheelScrollY, kWheelScrollGlobalX, 380 SendMouseWheelWithPhase(kWheelScrollX, kWheelScrollY, kWheelScrollGlobalX,
361 kWheelScrollGlobalY, 5, 5, 0, high_precision, 381 kWheelScrollGlobalY, 5, 5, 0, high_precision,
362 WebMouseWheelEvent::PhaseNone, 382 WebMouseWheelEvent::PhaseNone,
363 WebMouseWheelEvent::PhaseBegan); 383 WebMouseWheelEvent::PhaseBegan);
364 EXPECT_EQ(1U, GetAndResetSentEventCount()); 384 EXPECT_EQ(1U, GetAndResetSentEventCount());
365 SendMouseWheelEventAck(INPUT_EVENT_ACK_STATE_NOT_CONSUMED); 385 SendMouseWheelEventAck(INPUT_EVENT_ACK_STATE_NOT_CONSUMED);
366 EXPECT_EQ(3U, all_sent_events().size()); 386 if (scroll_latching_enabled_) {
367 EXPECT_INERTIAL_GESTURE_SCROLL_BEGIN(sent_gesture_event(0)); 387 // A fling has started, no ScrollEnd/ScrollBegin is sent.
368 EXPECT_INERTIAL_GESTURE_SCROLL_UPDATE(sent_gesture_event(1)); 388 EXPECT_EQ(1U, all_sent_events().size());
369 EXPECT_SYNTHETIC_INERTIAL_GESTURE_SCROLL_END(sent_gesture_event(2)); 389 EXPECT_INERTIAL_GESTURE_SCROLL_UPDATE(sent_gesture_event(0));
370 EXPECT_EQ(3U, GetAndResetSentEventCount()); 390 EXPECT_EQ(1U, GetAndResetSentEventCount());
391 } else {
392 EXPECT_EQ(3U, all_sent_events().size());
393 EXPECT_INERTIAL_GESTURE_SCROLL_BEGIN(sent_gesture_event(0));
394 EXPECT_INERTIAL_GESTURE_SCROLL_UPDATE(sent_gesture_event(1));
395 EXPECT_SYNTHETIC_INERTIAL_GESTURE_SCROLL_END(sent_gesture_event(2));
396 EXPECT_EQ(3U, GetAndResetSentEventCount());
397 }
371 398
372 SendMouseWheelWithPhase(kWheelScrollX, kWheelScrollY, kWheelScrollGlobalX, 399 SendMouseWheelWithPhase(kWheelScrollX, kWheelScrollY, kWheelScrollGlobalX,
373 kWheelScrollGlobalY, 5, 5, 0, high_precision, 400 kWheelScrollGlobalY, 5, 5, 0, high_precision,
374 WebMouseWheelEvent::PhaseNone, 401 WebMouseWheelEvent::PhaseNone,
375 WebMouseWheelEvent::PhaseChanged); 402 WebMouseWheelEvent::PhaseChanged);
376 EXPECT_EQ(1U, GetAndResetSentEventCount()); 403 EXPECT_EQ(1U, GetAndResetSentEventCount());
377 SendMouseWheelEventAck(INPUT_EVENT_ACK_STATE_NOT_CONSUMED); 404 SendMouseWheelEventAck(INPUT_EVENT_ACK_STATE_NOT_CONSUMED);
378 EXPECT_EQ(3U, all_sent_events().size()); 405 if (scroll_latching_enabled_) {
379 EXPECT_SYNTHETIC_INERTIAL_GESTURE_SCROLL_BEGIN(sent_gesture_event(0)); 406 EXPECT_EQ(1U, all_sent_events().size());
380 EXPECT_INERTIAL_GESTURE_SCROLL_UPDATE(sent_gesture_event(1)); 407 EXPECT_INERTIAL_GESTURE_SCROLL_UPDATE(sent_gesture_event(0));
381 EXPECT_SYNTHETIC_INERTIAL_GESTURE_SCROLL_END(sent_gesture_event(2)); 408 EXPECT_EQ(1U, GetAndResetSentEventCount());
382 EXPECT_EQ(3U, GetAndResetSentEventCount()); 409 } else {
410 EXPECT_EQ(3U, all_sent_events().size());
411 EXPECT_SYNTHETIC_INERTIAL_GESTURE_SCROLL_BEGIN(sent_gesture_event(0));
412 EXPECT_INERTIAL_GESTURE_SCROLL_UPDATE(sent_gesture_event(1));
413 EXPECT_SYNTHETIC_INERTIAL_GESTURE_SCROLL_END(sent_gesture_event(2));
414 EXPECT_EQ(3U, GetAndResetSentEventCount());
415 }
383 416
384 SendMouseWheelWithPhase(kWheelScrollX, kWheelScrollY, kWheelScrollGlobalX, 417 SendMouseWheelWithPhase(kWheelScrollX, kWheelScrollY, kWheelScrollGlobalX,
385 kWheelScrollGlobalY, 0, 0, 0, high_precision, 418 kWheelScrollGlobalY, 0, 0, 0, high_precision,
386 WebMouseWheelEvent::PhaseNone, 419 WebMouseWheelEvent::PhaseNone,
387 WebMouseWheelEvent::PhaseEnded); 420 WebMouseWheelEvent::PhaseEnded);
388 EXPECT_EQ(1U, GetAndResetSentEventCount()); 421 EXPECT_EQ(1U, GetAndResetSentEventCount());
389 SendMouseWheelEventAck(INPUT_EVENT_ACK_STATE_NOT_CONSUMED); 422 SendMouseWheelEventAck(INPUT_EVENT_ACK_STATE_NOT_CONSUMED);
390 EXPECT_EQ(2U, all_sent_events().size()); 423 if (scroll_latching_enabled_) {
391 EXPECT_SYNTHETIC_INERTIAL_GESTURE_SCROLL_BEGIN(sent_gesture_event(0)); 424 // MomentumPhase is ended, the scroll is done, and GSE is sent
392 EXPECT_INERTIAL_GESTURE_SCROLL_END(sent_gesture_event(1)); 425 // immediately.
393 EXPECT_EQ(2U, GetAndResetSentEventCount()); 426 EXPECT_EQ(1U, all_sent_events().size());
427 EXPECT_INERTIAL_GESTURE_SCROLL_END(sent_gesture_event(0));
428 EXPECT_EQ(1U, GetAndResetSentEventCount());
429 } else {
430 EXPECT_EQ(2U, all_sent_events().size());
431 EXPECT_SYNTHETIC_INERTIAL_GESTURE_SCROLL_BEGIN(sent_gesture_event(0));
432 EXPECT_INERTIAL_GESTURE_SCROLL_END(sent_gesture_event(1));
433 EXPECT_EQ(2U, GetAndResetSentEventCount());
434 }
394 } 435 }
395 436
396 std::unique_ptr<MouseWheelEventQueue> queue_; 437 std::unique_ptr<MouseWheelEventQueue> queue_;
397 std::vector<std::unique_ptr<WebInputEvent>> sent_events_; 438 std::vector<std::unique_ptr<WebInputEvent>> sent_events_;
398 size_t acked_event_count_; 439 size_t acked_event_count_;
399 InputEventAckState last_acked_event_state_; 440 InputEventAckState last_acked_event_state_;
400 base::MessageLoopForUI message_loop_; 441 base::MessageLoopForUI message_loop_;
401 WebMouseWheelEvent last_acked_event_; 442 WebMouseWheelEvent last_acked_event_;
402 int64_t scroll_end_timeout_ms_; 443 int64_t scroll_end_timeout_ms_;
403 bool scroll_latching_enabled_; 444 bool scroll_latching_enabled_;
(...skipping 29 matching lines...) Expand all
433 EXPECT_EQ(0U, GetAndResetSentEventCount()); 474 EXPECT_EQ(0U, GetAndResetSentEventCount());
434 EXPECT_EQ(1U, GetAndResetAckedEventCount()); 475 EXPECT_EQ(1U, GetAndResetAckedEventCount());
435 EXPECT_EQ(WebInputEvent::MouseWheel, acked_event().type); 476 EXPECT_EQ(WebInputEvent::MouseWheel, acked_event().type);
436 } 477 }
437 478
438 TEST_P(MouseWheelEventQueueTest, GestureSending) { 479 TEST_P(MouseWheelEventQueueTest, GestureSending) {
439 GestureSendingTest(false); 480 GestureSendingTest(false);
440 } 481 }
441 482
442 TEST_P(MouseWheelEventQueueTest, GestureSendingPrecisePixels) { 483 TEST_P(MouseWheelEventQueueTest, GestureSendingPrecisePixels) {
443 GestureSendingTest(false); 484 GestureSendingTest(true);
444 } 485 }
445 486
446 TEST_P(MouseWheelEventQueueTest, GestureSendingWithPhaseInformation) { 487 TEST_P(MouseWheelEventQueueTest, GestureSendingWithPhaseInformation) {
447 PhaseGestureSendingTest(false); 488 PhaseGestureSendingTest(false);
448 } 489 }
449 490
450 TEST_P(MouseWheelEventQueueTest, 491 TEST_P(MouseWheelEventQueueTest,
451 GestureSendingWithPhaseInformationPrecisePixels) { 492 GestureSendingWithPhaseInformationPrecisePixels) {
452 PhaseGestureSendingTest(true); 493 PhaseGestureSendingTest(true);
453 } 494 }
(...skipping 168 matching lines...) Expand 10 before | Expand all | Expand 10 after
622 EXPECT_EQ(1U, sent_gesture_event(updateEventIndex)->data.scrollUpdate.deltaY); 663 EXPECT_EQ(1U, sent_gesture_event(updateEventIndex)->data.scrollUpdate.deltaY);
623 EXPECT_EQ(static_cast<size_t>(updateEventIndex + 1), 664 EXPECT_EQ(static_cast<size_t>(updateEventIndex + 1),
624 GetAndResetSentEventCount()); 665 GetAndResetSentEventCount());
625 } 666 }
626 667
627 INSTANTIATE_TEST_CASE_P(MouseWheelEventQueueTests, 668 INSTANTIATE_TEST_CASE_P(MouseWheelEventQueueTests,
628 MouseWheelEventQueueTest, 669 MouseWheelEventQueueTest,
629 testing::Bool()); 670 testing::Bool());
630 671
631 } // namespace content 672 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698