LocalFrameClientImpl::SetHasReceivedUserGesture currently updates AutofillClient unconditionally w/o checking if |received_previously| is true or false.
However, it updates other clients only if |received_previously| is false.
https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/exported/LocalFrameClientImpl.cpp?rcl=2a5a0d8e3bec22edfa275a32264613fc28430ca4&l=1066
(Note that there seems to be only one other client: RenderFrameImpl::SetHasReceivedUserGesture)
The distinction is not clear since this only updates the sticky gesture bit now.
This is important for us because when we will have our simple activation model (Issue 696617), we need to make the first case conditional so that the transient activation bit gets updated appropriately.
LocalFrameClientImpl::SetHasReceivedUserGesture currently updates AutofillClient unconditionally w/o checking if |received_previously| is true or false.
However, it updates other clients only if |received_previously| is false.
https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/exported/LocalFrameClientImpl.cpp?rcl=2a5a0d8e3bec22edfa275a32264613fc28430ca4&l=1066
(Note that there seems to be only one other client: RenderFrameImpl::SetHasReceivedUserGesture)
The distinction is not clear since this only updates the sticky gesture bit now.
This is important for us because when we will have our simple activation model (Issue 696617), we may make the first case unconditional so that the transient activation bit gets updated appropriately.
Comment 1 by mustaq@chromium.org
, Oct 18 2017