Bug 1249748 - Ensure the mHandledByAPZ flag is set on WidgetTouchEvents that are handled by APZ. r=botond

MozReview-Commit-ID: 4ABY2jgHkGl
This commit is contained in:
Kartikaya Gupta 2016-02-24 13:19:43 -05:00
Родитель e282cff711
Коммит b894c806dd
2 изменённых файлов: 3 добавлений и 1 удалений

Просмотреть файл

@ -665,7 +665,6 @@ APZCTreeManager::ReceiveInputEvent(InputData& aEvent,
switch (aEvent.mInputType) {
case MULTITOUCH_INPUT: {
MultiTouchInput& touchInput = aEvent.AsMultiTouchInput();
touchInput.mHandledByAPZ = true;
result = ProcessTouchInput(touchInput, aOutTargetGuid, aOutInputBlockId);
break;
} case MOUSE_INPUT: {
@ -864,6 +863,7 @@ APZCTreeManager::ProcessTouchInput(MultiTouchInput& aInput,
ScrollableLayerGuid* aOutTargetGuid,
uint64_t* aOutInputBlockId)
{
aInput.mHandledByAPZ = true;
if (aInput.mType == MultiTouchInput::MULTITOUCH_START) {
// If we are panned into overscroll and a second finger goes down,
// ignore that second touch point completely. The touch-start for it is
@ -1125,6 +1125,7 @@ APZCTreeManager::ReceiveInputEvent(WidgetInputEvent& aEvent,
for (size_t i = 0; i < touchInput.mTouches.Length(); i++) {
*touchEvent.touches.AppendElement() = touchInput.mTouches[i].ToNewDOMTouch();
}
touchEvent.mFlags.mHandledByAPZ = touchInput.mHandledByAPZ;
return result;
}
case eWheelEventClass: {

Просмотреть файл

@ -279,6 +279,7 @@ APZEventState::ProcessTouchEvent(const WidgetTouchEvent& aEvent,
// Since APZ doesn't know about it we don't want to send a response for
// this block; we want to just skip over it from the point of view of
// prevent-default notifications.
APZES_LOG("Got a synthetic touch-start!\n");
break;
}
if (isTouchPrevented) {