1565410 - [api] Don't expose BundleEventListener

Status: RESOLVED FIXED

(GeckoView :: General, defect, P3)

Description

[[ex-Mozilla] Agi Sferro | :agi]

Bug 1539144 added WebExtensionController which implements BundleEventListener. Since BundleEventListener is a private class we should not expose it in the API.

This should have been caught by apilint but wasn't, bug for this is at https://github.com/mozilla-mobile/gradle-apilint/issues/62 (fixed in 0.2.2)

Comment 1

[[ex-Mozilla] Agi Sferro | :agi]

Attachment: Bug 1565410 - Don't expose BundleEventListener in the API.

This also updates apilint to 0.2.2 to ensure that future instances of this
problem will be caught automatically.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d35d90d0322b
Don't expose BundleEventListener in the API. r=esawin

Comment 3

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/d35d90d0322b

Comment 4

[Raul Gurzau (:RaulG)]

This bug was backed out due to conflicts while backing out bug 1557096.
Backout link: https://hg.mozilla.org/mozilla-central/rev/0fb761cd8fa448eb511e8b2520ede5a3230853ed

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a2180268af1d
Don't expose BundleEventListener in the API. r=esawin

Comment 6

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/a2180268af1d

Comment 7

[Chris Peterson [:cpeterson]]

69=wontfix because we don't need to uplift this API fix to GV 69 Beta.