Closed Bug 995390 Opened 10 years ago Closed 10 years ago

Make Bing/Yahoo partner codes consistent across channels

Categories

(Firefox :: Search, defect)

x86_64
Windows 8
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 32

People

(Reporter: mconnor, Assigned: mconnor)

Details

Attachments

(1 file)

      No description provided.
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.
Attachment #8408584 - Flags: review?(gavin.sharp)
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.
Attachment #8408584 - Flags: review?(gavin.sharp) → review+
Attachment #8408584 - Attachment is patch: true
https://hg.mozilla.org/mozilla-central/rev/38bf66433fc6
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Firefox 32
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?
Flags: needinfo?(mconnor)
Long-overdue, but I think it was because of how some partner builds worked.  I think it's all obsolete now.
Flags: needinfo?(mconnor)
You need to log in before you can comment on or make changes to this bug.