[Web] Disable onPointerMove on hover on gestures other than Hover

[m-bert]

Description

I've noticed that onPointerMove callback is triggered on each gesture, even if correct button is not pressed. Though events are not send, it still means that handlers do unnecessary work under the hood. This PR changes PointerEventManager so that it doesn't trigger onPointerMove callback without active pointer on gestures other than Hover

Test plan

Tested on example app with console.log added into onPointerMove in PanGestureHandler

[j-piasecki]

Can we handle that in another way (like a method or property)?

[m-bert]

Sure, we can after #3729 is merged 😅

[akwasniewski]

Correct me if I am wrong, but shouldn't onPointerMove be also triggered for ManualGestures?

[m-bert]

Correct me if I am wrong, but shouldn't onPointerMove be also triggered for ManualGestures?

tl;dr
I don't think it is necessary to consider this.

This PR doesn't disable onPointerMove callbacks completely, but only if there's no active pointer on the given view. If we follow android implementation, you can see that the only method that Manual overrides is onHandle . Now, onHandle is called inside handle method in parent class. If you look into implementation, you'll see that first it checks for active pointers. So it will behave in the same way 😅