946802 - Update bing icon to new triangular icon

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[:Margaret Leibovic]

This is a follow-up to bug 903084 comment 24.

From IRC:
[05 09:54:48] <kar> margaret - my preference is that we go with their latest branding. I think it's been delayed on desktop and if we can get it into mobile, all the better. It won't be an enormous deal if we cn't (and it goes to the next release), but I'd like a case made to see if we can get it in in time.

I can write a patch to do this for en-US.

My one concern is that the icon included in bug 903084 is 70x70, but we usually include 32x32 icons.

Comment 1

[Francesco Lodolo [:flod]]

Nothing really to do on l10n side: all locales using Bing rely on the en-US version, so as soon as you update mobile and browser (standard and metro) we're good.

Wondering where the idea of having a 70px icon came from...

Comment 2

[:Margaret Leibovic]

Attachment: patch

This patch updates the bing icon to use the 70x70 yellow "b" on grey background icon from bug 903084.

Comment 3

[:Margaret Leibovic]

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

Comment 4

[:Margaret Leibovic]

Comment on attachment 8343221 [details] [diff] [review]
patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): Bing updated its branding
User impact if declined: we show an older (and smaller) Bing icon
Testing completed (on m-c, etc.): just landed on fx-team
Risk to taking this patch (and alternatives if risky): low-risk icon change (although we should make sure nothing weird happens because this is a 70x70 icon)
String or IDL/UUID changes made by this patch: none

I wonder if we should also try to jam this into beta before release. My instinct tells me it's too late for that, but 26 will be the first time we ship Bing (and customizable search engines), so it could be nice to have the updated branding.

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/1810169dc57f

Comment 6

[bhavana bajaj [:bajaj]]

Comment on attachment 8343221 [details] [diff] [review]
patch

The ship for beta(Firefox 26) has sailed :( approving for aurora though.

Comment 7

[Ryan VanderMeulen [:RyanVM]]

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

Comment 8

[Teodora Vermesan (:TeoVermesan)]

Verified fixed on:
Build: Firefox for Android 27 Beta 1 (2013-12-11)
Device: Samsung Galaxy Nexus
OS: Android 4.2.1