video player no longer works after bug 1186948




2 years ago
2 years ago


(Reporter: Trevor Rowbotham, Unassigned)


({regression, site-compat})

49 Branch
regression, site-compat

Firefox Tracking Flags

(firefox48 unaffected, firefox49+ fixed)




2 years ago
After landing bug 1186948, the video player no longer loads.  The user is shown a message stating "Hulu requires Flash Player or higher. Please download and install the latest version of Flash Player before continuing.".

Hulu attempts to detect that flash is installed prior to attempting to load the player in [1] on line 10695.

Additionally, the article at [2] might need to be updated to reflect the changes of bug 1186948 if that change rides the train all the way to release.

[Tracking Requested - why for this release]:
tracking-firefox49: --- → ?
Keywords: regression
Tracking this regression for 49+ - this is a popular video site.
tracking-firefox49: ? → +


2 years ago
Keywords: site-compat

Comment 3

2 years ago
Trevor, do you see either the plugin icon in the location bar? Since you didn't mention it, I presume the plugin infobar doesn't show?
Flags: needinfo?(smokey101stair)

Comment 4

2 years ago
Correct, both the plugin icon in the location bar and the plugin infobar are not present. Presumably, this is because the object tag is only inserted if both navigator.plugins["Shockwave Flash"] and navigator.mimeType["application/x-shockwave-flash"] are present; both of which are no longer present after bug 1186948 when Flash is set to Ask to Activate.

Hulu appears to be using the following to determine whether to insert the Flash object into the page:

var b = navigator.plugins['Shockwave Flash'],
c = navigator.mimeTypes ? navigator.mimeTypes['application/x-shockwave-flash'] : !1;
b && b.description && c && c.enabledPlugin && (a = new deconcept.PlayerVersion(b.description.replace(/([a-zA-Z]|\s)+/, '').replace(/(\s+r|\s+b[0-9]+)/, '.').split('.')));

If the above fails, Hulu simply assumes that the browser is IE and falls back to trying to create ActiveXObjects.
Flags: needinfo?(smokey101stair)

Comment 5

2 years ago
Last I checked, Hulu had an experimental HTML player using DASH, but it only worked in Chrome since it only supported the Widevine CDM.  Now that nightly has Widevine, perhaps someone could convince Hulu to enable the HTML player for nightly users if they feel comfortable doing that.  Does anyone at Mozilla have a contact at Hulu?

Comment 6

2 years ago
Resolved via backout.
Last Resolved: 2 years ago
Resolution: --- → WORKSFORME
status-firefox49: affected → fixed
You need to log in before you can comment on or make changes to this bug.