Closed Bug 1047669 Opened 10 years ago Closed 9 years ago

"Download YouTube Videos as MP4" add-on does not show Download drop-down menu with e10s

Categories

(Firefox :: Extension Compatibility, defect)

34 Branch
x86
Windows 8.1
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
e10s + ---

People

(Reporter: jbecerra, Unassigned)

References

Details

(Keywords: addon-compat, dogfood, Whiteboard: [e10s][testday-20140801][awe:{b9bfaf1c-a63f-47cd-8b9a-29526ced9060} ])

Download drop-down does not show up in e10s mode.

Steps:
1. On an e10s enabled nightly install:
https://addons.mozilla.org/en-US/firefox/addon/download-youtube/?src=ss
2. Go to youtube.com and look for a video.

Expected: You should see a Download drop-down menu that allows you to download the media in several formats.

Actual: No menu is shown.

It works in non-e10s mode.
Mentor: cpeterson
Summary: "Download YouTube Videos as MP4" add-on does not work with e10s → "Download YouTube Videos as MP4" add-on does not show Download drop-down menu with e10s
Whiteboard: [e10s][testday-20140801] → [e10s][testday-20140801][e10s-top-addon]
Keywords: dogfood
The developer has been notified through AMO, though it looks like they were working on it already.
I used one of your github examples and changed the extension. It mostly works, but xmlhttprequest doesn't work cross-domain inside the sandbox. I couldn't find a way to fix xmlhttprequest and it's important for this extension.

https://www.dropbox.com/s/grtps8pk3qul6ju/firefox-electro.xpi?dl=0
I used Cc['@mozilla.org/xmlextras/xmlhttprequest;1'].createInstance(Ci.nsIXMLHttpRequest)
The xpi I linked before works for non-Vevo videos like this one: http://www.youtube.com/watch?v=xEhaVhta7sI I tested it in the latest Nightly
This add-on is based on Greasemonkey, so it looks like we need to make that work first.
Depends on: 1048164
The latest update (1.8.1) adds support for e10s. I tested it in Firefox 20, Firefox 36 stable, Firefox 38, latest Firefox Nightly with e10s enabled/disabled.
I can confirm it's working on Nightly.

Thanks!
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
This doesn't appear to be working correctly: I see the added youtube download button but when I click on it nothing happens. No errors are reported in the console.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(In reply to Jim Mathies [:jimm] from comment #8)
> This doesn't appear to be working correctly: I see the added youtube
> download button but when I click on it nothing happens. No errors are
> reported in the console.

Hmm, after a restart, it appears to work.
Status: REOPENED → RESOLVED
Closed: 9 years ago9 years ago
Resolution: --- → FIXED
Download YouTube Videos as MP4
Assignee: Jim Mathies
Last update: July 2015
Link to add-on:
https://github.com/gantt/downloadyoutube
Contact info for add-on: email deadline.support@gmail.com or create a new issue at github.com.
Add-on ID: {b9bfaf1c-a63f-47cd-8b9a-29526ced9060}
How well does it work?: 100%
- the download button is added to content and the context menu displays
- downloading video appears to work
Any obvious performance problems? none
Chromium version: the addon is based on greasemonkey and works where ever that is available
type:xul
Updating this bug with information on webextensions migration information.

The last contact with the add-on author they were planning to upload the new webextension version before 57 goes to Release.  They had just been waiting for the last few webextension API fixes to land.
Whiteboard: [e10s][testday-20140801][e10s-top-addon] → [e10s][testday-20140801][awe:{b9bfaf1c-a63f-47cd-8b9a-29526ced9060} ]
You need to log in before you can comment on or make changes to this bug.