Provide NativePanZoomController API

RESOLVED FIXED in Firefox 59

Status

()

RESOLVED FIXED
a year ago
a year ago

People

(Reporter: jchen, Assigned: jchen)

Tracking

unspecified
Firefox 59
All
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox59 fixed)

Details

Attachments

(4 attachments)

(Assignee)

Description

a year 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);
>     }
> }
(Assignee)

Updated

a year ago
No longer blocks: 1415994
Depends on: 1415994
(Assignee)

Updated

a year ago
Assignee: nobody → nchen
Status: NEW → ASSIGNED
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 5

a year ago
mozreview-review
Comment on attachment 8933136 [details]
Bug 1416316 - 1. Move overscroll to LayerSession;

https://reviewboard.mozilla.org/r/204104/#review209842
Attachment #8933136 - Flags: review?(rbarker) → review+

Comment 6

a year ago
mozreview-review
Comment on attachment 8933137 [details]
Bug 1416316 - 2. Clean up NativePanZoomController;

https://reviewboard.mozilla.org/r/204106/#review209844
Attachment #8933137 - Flags: review?(rbarker) → review+

Comment 7

a year ago
mozreview-review
Comment on attachment 8933138 [details]
Bug 1416316 - 3. Update NPZC usages;

https://reviewboard.mozilla.org/r/204108/#review209852
Attachment #8933138 - Flags: review?(rbarker) → review+
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
(Assignee)

Comment 12

a year ago
mozreview-review
Comment on attachment 8933139 [details]
Bug 1416316 - 4. Update comments and bindings;

https://reviewboard.mozilla.org/r/204110/#review209930
Attachment #8933139 - Flags: review+

Comment 13

a year ago
Pushed by nchen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d7abdb31418e
1. Move overscroll to LayerSession; r=rbarker
https://hg.mozilla.org/integration/autoland/rev/70246e37dfc8
2. Clean up NativePanZoomController; r=rbarker
https://hg.mozilla.org/integration/autoland/rev/e177b3dd4380
3. Update NPZC usages; r=rbarker
https://hg.mozilla.org/integration/autoland/rev/e23185b58116
4. Update comments and bindings; r=jchen
(Assignee)

Updated

a year ago
Blocks: 1422137

Comment 14

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/d7abdb31418e
https://hg.mozilla.org/mozilla-central/rev/70246e37dfc8
https://hg.mozilla.org/mozilla-central/rev/e177b3dd4380
https://hg.mozilla.org/mozilla-central/rev/e23185b58116
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
status-firefox59: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 59
You need to log in before you can comment on or make changes to this bug.