Provide NativePanZoomController API

NEW
Unassigned

Status

()

Firefox for Android
GeckoView
9 days ago
9 days ago

People

(Reporter: jchen, Unassigned)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

9 days ago
For the session to accept touch/motion events, I think the session class should provide a NativePanZoomController getter, e.g. `LayerSession.getPanZoomController()`, which returns a NativePanZoomController instance.

We should clean up NativePanZoomController so its only public methods are documented APIs, e.g. `NativePanZoomController.onTouchEvent()`, which correctly handle edge cases like before the Compositor is ready, etc.

Finally, we should have GeckoView use the API, e.g.

> @Override
> public boolean onTouchEvent(final MotionEvent event) {
>     if (mSession != null) {
>         mSession.getPanZoomController().onTouchEvent(event);
>     }
> }
(Reporter)

Updated

9 days ago
No longer blocks: 1415994
Depends on: 1415994
You need to log in before you can comment on or make changes to this bug.