Closed Bug 1159814 Opened 9 years ago Closed 9 years ago

Change the Adobe CDM's homepage URL to "http://help.adobe.com/en_US/primetime/drm/HTML5_CDM"

Categories

(Core :: Audio/Video, defect, P1)

defect

Tracking

()

RESOLVED FIXED
mozilla40
Tracking Status
firefox37 --- unaffected
firefox38 --- fixed
firefox38.0.5 --- fixed
firefox39 --- fixed
firefox40 --- fixed
firefox-esr38 --- fixed

People

(Reporter: cpeterson, Assigned: Dolske)

Details

Attachments

(2 files, 1 obsolete file)

No -- not the license URL. The "homepage" URL needs to change.
Dolske: can you please change the Adobe CDM's homepage URL from "https://www.adobe.com/marketing-cloud/primetime-tv-platform.html" to "http://help.adobe.com/en_US/primetime/drm/HTML5_CDM"?

https://mxr.mozilla.org/mozilla-central/source/toolkit/mozapps/extensions/internal/GMPProvider.jsm#63
Summary: Change the Adobe CDM's license URL to "http://help.adobe.com/en_US/primetime/drm/HTML5_CDM" → Change the Adobe CDM's homepage URL to "http://help.adobe.com/en_US/primetime/drm/HTML5_CDM"
Attached patch Patch v.1 (obsolete) — Splinter Review
Assignee: nobody → dolske
Flags: needinfo?(dolske)
Comment on attachment 8599545 [details] [diff] [review]
Patch v.1

We should ask them to make this support https.
Attachment #8599545 - Flags: review+
Attachment #8599545 - Attachment is obsolete: true
Made this apply cleanly on beta (since bug 1129219 never landed there).
Setting checkin-needed since all trees are closed. All. the. trees.
Keywords: checkin-needed
Comment on attachment 8599587 [details] [diff] [review]
Patch v.1 (for checkin on m-c and m-a)

[Triage Comment]
Attachment #8599587 - Flags: approval-mozilla-aurora+
Comment on attachment 8599588 [details] [diff] [review]
Patch v.1 (for checkin on m-b)

Approval Request Comment
[Feature/regressing bug #]: n/a
[User impact if declined]: CDM links to URL not preferred by Adobe
[Describe test coverage new/current, TreeHerder]: n/a
[Risks and why]: Just changes what page is linked to from within the Adobe CDM module in the addon-manager UI. So very very low.
[String/UUID change made/needed]: No.

I'd use my a+ granting powers here, except the last beta has already gone to build. And so just want relmanagers to be explicitly aware/approve that this would be landing for the 38.0 RC. I'm comfortable with this being very low risk.
Attachment #8599588 - Flags: approval-mozilla-beta?
https://hg.mozilla.org/mozilla-central/rev/db801cb03216
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Comment on attachment 8599588 [details] [diff] [review]
Patch v.1 (for checkin on m-b)

RyanVM tells me this should actually be a? for mozilla-release, to get it into 38.0, due to the unusual state of the trains for the 38.0.5 special release.
Attachment #8599588 - Flags: approval-mozilla-beta? → approval-mozilla-release?
Comment on attachment 8599588 [details] [diff] [review]
Patch v.1 (for checkin on m-b)

should be in 38 rc1
Attachment #8599588 - Flags: approval-mozilla-release? → approval-mozilla-release+
Attachment #8599588 - Attachment is patch: true
You need to log in before you can comment on or make changes to this bug.