[geckoview] Remove moar JavaPanZoomController (JPZ) from mobile/android

RESOLVED FIXED in Firefox 52

Status

()

Firefox for Android
GeckoView
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: kats, Assigned: kats)

Tracking

unspecified
Firefox 52
Unspecified
Android
Points:
---

Firefox Tracking Flags

(firefox52 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments)

+++ This bug was initially created as a clone of Bug #1291373 +++

There's more JPZ goop that can be removed.
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Note that in the second patch I left panZoomStopped() in PanZoomTarget because we'll need it to fix bug 1302150.

Comment 4

2 years ago
mozreview-review
Comment on attachment 8797722 [details]
Bug 1307522 - Delete a bunch of classes and related goop no longer used now that JPZ is gone.

https://reviewboard.mozilla.org/r/83370/#review81878
Attachment #8797722 - Flags: review?(snorp) → review+

Comment 5

2 years ago
mozreview-review
Comment on attachment 8797723 [details]
Bug 1307522 - Remove a bunch of unnecessary methods from the PanZoomTarget interface now that JPZ is gone.

https://reviewboard.mozilla.org/r/83372/#review81880
Attachment #8797723 - Flags: review?(snorp) → review+

Comment 6

2 years ago
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ff4d5455a1b9
Delete a bunch of classes and related goop no longer used now that JPZ is gone. r=snorp
https://hg.mozilla.org/integration/autoland/rev/d1ea0a1e5efb
Remove a bunch of unnecessary methods from the PanZoomTarget interface now that JPZ is gone. r=snorp

Comment 7

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/ff4d5455a1b9
https://hg.mozilla.org/mozilla-central/rev/d1ea0a1e5efb
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox52: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52

Comment 8

2 years ago
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/84df59f2684e
Remove obsolete comment. r=me and DONTBUILD because NPOTB
More deletion on the way in bug 1235513
See Also: → bug 1235513
You need to log in before you can comment on or make changes to this bug.