DescriptionRevert 290001 because it's causing test failures:
[20526:20526:0815/151613:1174431576:FATAL:touch_exploration_controller.cc(763)] Check failed: false. tap timer fired in unrecognized state: CORNER_PASSTHROUGH
#0 0x7f485532361e base::debug::StackTrace::StackTrace()
#1 0x7f48553ba882 logging::LogMessage::~LogMessage()
#2 0x7f485314bcfc ui::TouchExplorationController::OnTapTimerFired()
#3 0x7f485314af2b ui::TouchExplorationController::RewriteEvent()
#4 0x7f48531c89a6 ui::EventSource::SendEventToProcessor()
#5 0x0000005f6bc0 ui::EventSourceTestApi::SendEventToProcessor()
#6 0x0000005f396f ui::test::EventGenerator::DoDispatchEvent()
#7 0x0000005f1480 ui::test::EventGenerator::Dispatch()
#8 0x00000051a1a5 ui::TouchExplorationTest::AssertCornerPassthroughWorking()
#9 0x0000005153fb ui::TouchExplorationTest_ActivateLeftCornerPassthrough_Test::TestBody()
#10 0x0000005a49f3 testing::internal::HandleSehExceptionsInMethodIfSupported\u003C>()
#11 0x00000059005e testing::internal::HandleExceptionsInMethodIfSupported\u003C>()
#12 0x000000583ed5 testing::Test::Run()
#13 0x0000005845db testing::TestInfo::Run()
In ActivateLeftCornerPassthrough, ActivateRightCornerPassthrough, and SingleTapLongPress tests.
> Modified state flow for touch explore released in touch_exploration_controller.
>
> Now when performing single tap, the tap timer to begin one finger passthrough is not started. This problem originally arose because touch explore released state and single tap released state are treated similarly in the code, but one finger passthrough should be a state reachable only from single tap released.
>
> Another state change is that if the user presses in single tap/touch explore released and there is no previous touch exploration location, the user is redirected to the wait state.
>
> TEST=TouchExplorationTest.SingleTapLongPress
> BUG=402544
>
> Review URL: https://codereview.chromium.org/465543003
TBR=evy@chromium.org
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=290092
Patch Set 1 #
Messages
Total messages: 2 (0 generated)
|