Create a new API in touch action filter |
||
Issue description
The TouchActionFilter::OnSetTouchAction can be called in the following 2 case:
1. InputRouterImpl::OnSetTouchAction
2. InputRouterImpl::OnTouchEventAck, when it is a touch start and that the ack state is
ack_result == INPUT_EVENT_ACK_STATE_NO_CONSUMER_EXISTS
For the first case, we are setting the touch action because there is a event handler. For the second case, we should provide another API to set the allowed_touch_action_ to auto.
With that change, we can add a DCHECK in the TouchActionFilter::OnSetTouchAction to make sure that there is a touch event handler.
,
Jun 6 2018
sorry, I meant when there is not a touch event handler ^
,
Jun 8 2018
Right, we should be able to remove the OnSetTouchAction call when there is no touch event handler. Just need to make sure that iframe will call OnHasTouchEventHandler(false) to set the touch action to auto first.
,
Jun 10 2018
|
||
►
Sign in to add a comment |
||
Comment 1 by flackr@chromium.org
, Jun 6 2018