Closed
Bug 1307522
Opened 8 years ago
Closed 8 years ago
[geckoview] Remove moar JavaPanZoomController (JPZ) from mobile/android
Categories
(GeckoView :: General, defect)
Tracking
(firefox52 fixed)
RESOLVED
FIXED
mozilla52
| Tracking | Status | |
|---|---|---|
| firefox52 | --- | fixed |
People
(Reporter: kats, Assigned: kats)
References
Details
Attachments
(2 files)
+++ 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) |
|
Assignee | |
Comment 3•8 years ago
|
||
Note that in the second patch I left panZoomStopped() in PanZoomTarget because we'll need it to fix bug 1302150.
|
||
Comment 4•8 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•8 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+
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•8 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/ff4d5455a1b9 https://hg.mozilla.org/mozilla-central/rev/d1ea0a1e5efb
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox52:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
Pushed by kgupta@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/84df59f2684e Remove obsolete comment. r=me and DONTBUILD because NPOTB
|
|
||
Comment 10•8 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/84df59f2684e
|
||
Updated•5 years ago
|
Product: Firefox for Android → GeckoView
|
||
Updated•5 years ago
|
Target Milestone: Firefox 52 → mozilla52
You need to log in
before you can comment on or make changes to this bug.
Description
•