1065997 - Update localized search plugins to use rel="searchform"

Status: RESOLVED WONTFIX

(Firefox :: Search, defect, P4)

Description

[Marco Bonardo [:mak]]

Once bug 990799 is done and tested for en-US, we should updated the localized search plugins too.

Comment 1

[Francesco Lodolo [:flod]]

I was peeking at the discussion on #fx-team about bug 990799, and the impression is that not all searchplugins can be modified 

What is the reasoning behind this change? Meaning: what do we gain in terms of features, etc.

Because between this and the idea of making https a requirement that I read in another post, it starts to become scary.

Comment 2

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

The benefit is that we avoid having to update two URLs when updating search plugins, and the behavior between "empty searches" and normal searches is more consistent.

Consistency is most important when Mozilla-specific referral codes are involved, so for l10n plugins that likely means Yahoo/Google.

Comment 3

[Francesco Lodolo [:flod]]

OK. If we're talking about referrals. it might include eBay and Amazon: eBay is pending an update, I never had to do updates to Amazon so far, so I'm not really sure of the situation either.

Google and Bing are centralized, so it's not really an issue. But, then we have the odd one (Japanese).

Mike, do you know if we still need that? I read bug 534125 and bug 508793 and I don't really understand why only Japanese has these differences.

Comment 4

[Francesco Lodolo [:flod]]

Is this something we're still interested in doing? Searchplugins are now centralized, so it would be a single patch against mozilla-central

Comment 5

[Mike Kaply [:mkaply]]

Bug 990799 provided no detail on the "Why". Is it just a correctness issue?

Comment 6

[Francesco Lodolo [:flod]]

(In reply to Mike Kaply [:mkaply] from comment #5)
> Bug 990799 provided no detail on the "Why". Is it just a correctness issue?

I had a needinfo open for over 3 years from mconnor, at this point I have no clue. As far as I'm concerned, we should WONTFIX this and add rel="searchform" for new searchplugins, or when touching old ones.

Comment 7

[Florian Quèze [:florian]]

(In reply to Mike Kaply [:mkaply] from comment #5)
> Bug 990799 provided no detail on the "Why". Is it just a correctness issue?

Using rel="searchform" typically makes us send affiliate codes (and process the 'purpose' value to send the correct one), whereas <SearchForm> typically doesn't.

Comment 8

[Mike Kaply [:mkaply]]

> Using rel="searchform" typically makes us send affiliate codes (and process the 'purpose' value to send the correct one), whereas <SearchForm> typically doesn't.

That makes a difference at least for Yandex and Yahoo. We should at least update those (and test).

Comment 9

[Mark Banner (:standard8)]

Since the switch to moving everything into mozilla-central and also to WebExtensions, I believe this is no longer relevant.