995390 - Make Bing/Yahoo partner codes consistent across channels

Status: RESOLVED FIXED

(Firefox :: Search, defect)

Comment 1

[Mike Connor [:mconnor]]

Attachment: nightlyAuroraCodes

Comment 2

[Mike Connor [:mconnor]]

Comment on attachment 8408584 [details] [diff] [review]
nightlyAuroraCodes

* Bing can just be hardcoded
* For Yahoo it's much easier to change the pref (there are a _lot_ of localized plugins in play here).
* Hooray tests.

Comment 3

[:Gavin Sharp [email: [e-mail]]]

Comment on attachment 8408584 [details] [diff] [review]
nightlyAuroraCodes

>diff --git a/browser/branding/aurora/pref/firefox-branding.js b/browser/branding/aurora/pref/firefox-branding.js

>+// code usage depends on contracts, please contact the Business Affairs team if you have questions

How about "Don't change these without explicit signoff from the Firefox module owner"? Seems more likely to stand the test of time :)

How about moving these prefs to firefox.js now that they're the same regardless of branding?

> pref("browser.search.param.yahoo-fr-cjkt", ""); // now unused

Non-existent prefs seems to be handled fine by the search service (MozParam is ignored), so might as well clean this up?

> pref("browser.search.param.yahoo-f-CN", "");

Is this unused too?

r=me with those changes.

Comment 4

[Mike Connor [:mconnor]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/38bf66433fc6

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/38bf66433fc6

Comment 6

[:Gavin Sharp [email: [e-mail]]]

mconnor, do you remember why we left yahoo-fr as a pref-based MozParams, rather than just hardcoding the values we need in yahoo.xml? Do some distros or locales set the prefs still?

Comment 7

[Mike Connor [:mconnor]]

Long-overdue, but I think it was because of how some partner builds worked.  I think it's all obsolete now.