Closed Bug 1331967 Opened 5 years ago Closed 5 years ago

Hang on navigating to XPI file


(Testing :: Marionette, defect)

Version 3
Not set


(firefox55 fixed)

Tracking Status
firefox55 --- fixed


(Reporter: davehunt, Assigned: whimboo)


(Blocks 2 open bugs)


(Whiteboard: [spec][17/04] )

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

When using Marionette to navigate to an add-on file (.xpi), the appropriate popup notifications are displayed, but a response is never sent to the client.

Steps to replicate:

> from selenium.webdriver import Firefox
> from selenium.webdriver.firefox.options import Options
> o = Options()
> o.binary = '/Applications/'
> f = Firefox(firefox_options=o)
> f.get('')

I've use the Selenium python client above, however I believe Henrik has also replicated this using the Marionette client.
Yes, you can see the same when using Marionette directly.

I think that we have no chance to blacklist each and every mimetype for which a load event will not happen. We should find out which logic we use in Firefox and Fennec to actually determine if an event has to be sent.
No longer depends on: 1322772
Summary: Hang on navigating to add-on file → Hang on navigating to XPI file
I don’t exactly know what the expected behaviour here would be.  Navigating to an .xpi file opens the doorhanger to install a new addon, and the content window navigates to about:blank, which is an HTMLDocument type, so I’m not sure why the current navigation algorithm isn’t working.

We have special treatment of about:blank since it does not generate the same DOM load events that other normal documents do.

I would expect the same overall problem to manifest also when navigating to any resource that triggers the download dialogue.
I added the capability to install addons with my patch for click+page load. A test also landed here:
Assignee: nobody → hskupin
Depends on: 1335778
Whiteboard: [spec][17/04]
Target Milestone: --- → mozilla55
Closed: 5 years ago
Resolution: --- → FIXED
No longer depends on: webdriver-navigate
You need to log in before you can comment on or make changes to this bug.