Closed Bug 1151544 Opened 11 years ago Closed 11 years ago

Categories

(Toolkit :: Add-ons Manager, defect, P1)

defect

Tracking

()

RESOLVED FIXED
mozilla40
Tracking Status
firefox37 --- unaffected
firefox38 + fixed
firefox39 --- fixed
firefox40 --- fixed

People

(Reporter: cpeterson, Assigned: spohl)

References

Details

Attachments

(2 files, 1 obsolete file)

Adobe asked us to change toolkit/mozapps/extensions/internal/GMPProvider.jsm's `homepageURL`: - homepageURL: "http://help.adobe.com/en_US/primetime/drm/index.html", + homepageURL: "http://www.adobe.com/marketing-cloud/primetime-tv-platform.html"
Assignee: nobody → spohl.mozilla.bugs
Attached patch Patch (obsolete) — Splinter Review
Attachment #8588729 - Flags: review?(gfritzsche)
Priority: -- → P1
I am waiting to hear back from Adobe whether they want us to use HTTPS instead of HTTP: https://www.adobe.com/marketing-cloud/primetime-tv-platform.html
Attachment #8588729 - Flags: review?(gfritzsche) → review+
Thanks, Georg! Chris, please let me know once you've heard back. Thanks!
Flags: needinfo?(cpeterson)
Please us the HTTPS version of this URL. When we have an HTTPS version of the EULA URL, we will ask you to update that as well.
Thanks, Joe. Stephen, we should uplift this change to Aurora 39 and Beta 38.
Flags: needinfo?(cpeterson)
Summary: Change Adobe CDM's homepageURL to "http://www.adobe.com/marketing-cloud/primetime-tv-platform.html" → Change Adobe CDM's homepageURL to "https://www.adobe.com/marketing-cloud/primetime-tv-platform.html"
Attached patch PatchSplinter Review
Updated patch with https:// URL. Carrying over r+.
Attachment #8588729 - Attachment is obsolete: true
Attachment #8589210 - Flags: review+
[Tracking Requested - why for this release]: EME is slated for Firefox 38 and we should have a proper homepage URL for Adobe EME in the addons manager.
Comment on attachment 8589210 [details] [diff] [review] Patch Approval Request Comment [Feature/regressing bug #]: Adobe EME [User impact if declined]: Incorrect homepage URL dispayed for Adobe EME in addons manager. [Describe test coverage new/current, TreeHerder]: Local testing confirms that this URL is working properly. [Risks and why]: none [String/UUID change made/needed]: none
Attachment #8589210 - Flags: approval-mozilla-aurora?
Attached patch Patch for betaSplinter Review
Approval Request Comment [Feature/regressing bug #]: Adobe EME [User impact if declined]: Incorrect homepage URL dispayed for Adobe EME in addons manager. [Describe test coverage new/current, TreeHerder]: Local testing confirms that this URL is working properly. [Risks and why]: none [String/UUID change made/needed]: none
Attachment #8589236 - Flags: review+
Attachment #8589236 - Flags: approval-mozilla-beta?
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Attachment #8589210 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment on attachment 8589236 [details] [diff] [review] Patch for beta Should be in 38 beta 3.
Attachment #8589236 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: