615200 - Update eBay DE Search Plugin

Status: VERIFIED FIXED

(Mozilla Localizations :: de / German, defect)

Description

[Kev Needham [:kev]]

Attachment: Patch for eBay DE Search Plugin

Per eBay's request, please update the search plugin to be included with Mozilla products per the attached patch. The changes add suggested search (not yet active) and cut the current number of redirects from the initial search query from 4 to 1. This change should block Fx4 for the DE locale.

Comment 1

[Kev Needham [:kev]]

Attachment: Patch for eBay DE Search Plugin

Apologies, forgot to update the favicon as well, per eBay request.

Comment 2

[Seth Bindernagel]

Comment on attachment 493674 [details] [diff] [review]
Patch for eBay DE Search Plugin

This looks good.  Thank you, Kev!

As mentioned in the bug, can you please check this into your
L10n-central repository, adding a comment that references bug 615200, this change, and r=sethb.  

Please test the new build and sign off.  Once you have signed off, please resolve this bug again as fixed.

Thanks to all.

Comment 3

[Kadir Topal [:atopal]]

checked in: http://hg.mozilla.org/l10n-central/de/rev/3669624dcef5

Comment 4

[Seth Bindernagel]

Thanks, Kadir.  After some nightly testing, can you sign off on this change and then resolve this bug as fixed?  I appreciate the quick turnaround.

Comment 5

[Kadir Topal [:atopal]]

bot sure how to test, but ebay search works, so I guess this is done.

Comment 6

[Kevin Brosnan]

Can verify the icon update, no more brown color in the 'e'. Though I am getting 2 301 moved responses.

[17:45:42.729] GET http://rover.ebay.com/rover/1/707-37276-17702-4/4?mpre=http://shop.ebay.de/?_nkw=desire [HTTP/1.1 301 Moved Permanently 297ms]
[17:45:43.028] GET http://rover.ebay.de/rover/1/707-37276-17702-4/4?mpre=http%3A%2F%2Fshop.ebay.de%2F%3F_nkw%3Ddesire%26rvr_id%3D207339811434&cguid=f2db0f7812d0a0aa14e4d7e4ff800a4e [HTTP/1.1 301 Moved Permanently 391ms]

Not something we have control over though.

Mozilla/5.0 (Windows NT 6.0; WOW64; rv:2.0b11) Gecko/20100101 Firefox/4.0b11 DE locale

Comment 7

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

Note: the added |type="image/x-icon"| on the <Image> is unnecessary - the search service does not make use of the "type" attribute on Image elements. You should feel free to remove it.