1414613 - browserAction.setPopup with an empty string does not clear the popup

Status: RESOLVED DUPLICATE of bug 1426484

(WebExtensions :: Android, defect, P1)

Description

[Manish Jethani]

Attachment: example.xpi

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/62.0.3202.62 Safari/537.36

Steps to reproduce:

 1.  Install the attached extension
 2.  Tap on the "Example" menu item


Actual results:

Opens popup.html


Expected results:

Opens https://example.com/

Comment 1

[Manish Jethani]

The problem clearly is this line:

https://hg.mozilla.org/mozilla-central/rev/965791d09bdc#l1.31

It should check for an empty string.

Comment 2

[Wladimir Palant]

For reference, since this isn't mentioned explicitly: Firefox desktop does this correctly, calling setPopup({popup: ""}) disables the pop-up.

Comment 3

[MrOtherGuy]

Hello, 
Is there any progress on this bug or is there perhaps going to be some other method of disabling browserAction popup?
Right now there isn't any way to disable the popup if it has been set; this makes it impossible to use onClicked() if the extension would ever need to show some popup.

Personally I would prefer is there was a method to explicitly disable popup like browserAction.disablePopup() because current options can be somewhat confusing. 

For example calling setPopup({popup:""}) disables the popup but setPopup({popup:null}) returns it to global value.

Comment 4

[Andrew Swan [:aswan]]

In the time since this bug was filed, setPopup() on desktop has been changed (see bug 1419940).  We have bug 1426484 to bring Android in sync with desktop which I think would address this.  Luca, you had assigned this to yourself, if you're still interested in this I think working on the dup would be the right thing...