Manish Jethani Reporter
	Description • 7 years ago

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/

	Manish Jethani Reporter
	Comment 1 • 7 years ago

The problem clearly is this line:

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

It should check for an empty string.

	Wladimir Palant
	Comment 2 • 7 years ago

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

	MrOtherGuy
	Comment 3 • 7 years ago

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.

	Andrew Swan [:aswan]
	Comment 4 • 7 years ago

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...