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

Status: RESOLVED FIXED

(GeckoView :: General, defect)

Description

[Kartikaya Gupta (email:[e-mail])]

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

There's more JPZ goop that can be removed.

Comment 1

[Kartikaya Gupta (email:[e-mail])]

Attachment: Bug 1307522 - Delete a bunch of classes and related goop no longer used now that JPZ is gone.

Review commit: https://reviewboard.mozilla.org/r/83370/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/83370/

Comment 2

[Kartikaya Gupta (email:[e-mail])]

Attachment: Bug 1307522 - Remove a bunch of unnecessary methods from the PanZoomTarget interface now that JPZ is gone.

Review commit: https://reviewboard.mozilla.org/r/83372/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/83372/

Comment 3

[Kartikaya Gupta (email:[e-mail])]

Note that in the second patch I left panZoomStopped() in PanZoomTarget because we'll need it to fix bug 1302150.

Comment 4

[James Willcox (:snorp) ([e-mail]) (he/him)]

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

Comment 5

[James Willcox (:snorp) ([e-mail]) (he/him)]

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

Comment 6

[Pulsebot]

Pushed by [e-mail]:
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

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/ff4d5455a1b9
https://hg.mozilla.org/mozilla-central/rev/d1ea0a1e5efb

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/84df59f2684e
Remove obsolete comment. r=me and DONTBUILD because NPOTB

Comment 9

[Kartikaya Gupta (email:[e-mail])]

More deletion on the way in bug 1235513

Comment 10

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/84df59f2684e