1130682 - Update Adobe EME with official name, description and homepage URL in addons manager

Status: RESOLVED FIXED

(Toolkit :: Add-ons Manager, defect)

Description

[Stephen A Pohl [:spohl]]

Adobe needs to confirm the official name, description and homepage URL for Adobe EME. Bug 1089867 will land Adobe EME pref'd off and has the following (temporary) values:

Name: Primetime Content Decryption Module provided by Adobe Systems, Incorporated
Description: Play back protected web video.
Homepage URL: https://www.adobe.com/

Comment 1

[Chris Peterson [:cpeterson]]

Adobe says the official branding decision may take a couple weeks.

Comment 2

[Javaun Moradi [:javaun]]

Stephen and I are chatting about this.

Comment 3

[Javaun Moradi [:javaun]]

Too soon! Stephen and I are chatting about this. It sounds like at least the description portion may require translation. February 23 is when 38 goes to Developer Edition and we hit string freeze and begin L10N.

I assume we're doing all the translation for description? (License is held on their end, so not ours to translate)

ni'ing Dolske. I'm guessing we need final text (name, description, url) very soon, or a placeholder they're comfortable living with for a while.

Comment 4

[Joe Steele]

I should have the URLs by end of day (California time). I am hoping to have buy-in on the above text as well, although if you need to move sooner than that I think we can live with that text initially.

Comment 5

[Joe Steele]

Let's go with these URLs for now:
Homepage URL:  http://help.adobe.com/en_US/primetime/drm/index.html 
EULA URL:  http://help.adobe.com/en_US/primetime/drm/HTML5_CDM_EULA/index.html 

The content is not on those pages yet, but is under construction. I do not have buy-in on the text yet, but I am pushing on that. If we hit string freeze without resolution from us, go ahead with the above text.

Comment 6

[Joe Steele]

I confirmed with our product management that we are ok with the text and URLs above.

Comment 7

[Justin Dolske [:Dolske]]

Sounds like from comment 6 that the original text that landed (see comment 0) is ok, so all that's left to do is update the URLs, and that's unaffected by string-freeze.

Joe: Just to be clear, did you want all localizations of Firefox to link to the exact URLs in comment 5, or do you want Firefox to replace the "en_US" in the URL with the user's actual locale? Either is easy for us, I just want to make sure we don't send users to 404s.

Comment 8

[Joe Steele]

Please send everyone to the exact URLs above for now. I have not gotten confirmation from the folks managing the site that they will redirect properly yet. If I do get confirmation -- I will file a bug with the details.

Comment 9

[Stephen A Pohl [:spohl]]

Attachment: Patch

Comment 10

[Georg Fritzsche [:gfritzsche]]

Comment on attachment 8568075 [details] [diff] [review]
Patch

I don't see myself getting to other reviews before i leave, sorry.
Dave, can you maybe review this?

Comment 11

[Stephen A Pohl [:spohl]]

https://hg.mozilla.org/integration/fx-team/rev/fad92c7e2f57

Comment 12

[Stephen A Pohl [:spohl]]

Comment on attachment 8568075 [details] [diff] [review]
Patch

Approval Request Comment
[Feature/regressing bug #]: Adobe EME
[User impact if declined]: The addons manager will display an incorrect homepage URL for Adobe EME.
[Describe test coverage new/current, TreeHerder]: This was a simple URL change. Successfully tested locally.
[Risks and why]: none
[String/UUID change made/needed]: none

Comment 13

[Stephen A Pohl [:spohl]]

Attachment: Patch for beta

Approval Request Comment
[Feature/regressing bug #]: Adobe EME
[User impact if declined]: The addons manager will display an incorrect homepage URL for Adobe EME.
[Describe test coverage new/current, TreeHerder]: This was a simple URL change. Successfully tested locally.
[Risks and why]: none
[String/UUID change made/needed]: none

Comment 14

[Lawrence Mandel [:lmandel] (use needinfo)]

Comment on attachment 8570650 [details] [diff] [review]
Patch for beta

Trivial change. Beta+

Comment 15

[Lawrence Mandel [:lmandel] (use needinfo)]

Comment on attachment 8568075 [details] [diff] [review]
Patch

Aurora+

Comment 16

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/d373bc0db438

Comment 17

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/1ba1e8df6e2f

Comment 18

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/fad92c7e2f57