940494 - when a Social provider has a status button, the provider's icon should go to the status button rather than the social api toolbar button

Status: RESOLVED FIXED

(Firefox Graveyard :: SocialAPI, defect)

Description

[Shane Caraveo (:mixedpuppy)]

In certain situations, a provider without a sidebar shows up in sidebar selection, and notifications go to that rather than the status button.

Comment 1

[Florian Quèze [:florian]]

Attachment: WIP

This patch gives a mostly reasonable behavior for handling of the icons (the icons of social providers providing a statusURL go only to their status button).

Comment 2

[Shane Caraveo (:mixedpuppy)]

Comment on attachment 8336116 [details] [diff] [review]
WIP

This looks good and addresses a couple edge cases I had noticed before.  While were removing the toolbarbutton for 28 (assuming I can land that) this is a good fix for 27 if we can uplift.

Comment 3

[Florian Quèze [:florian]]

Attachment: Patch

Same patch as before, but now the tests pass.

Comment 4

[Florian Quèze [:florian]]

Rephrasing bug summary to reflect the issue we are actually addressing here. We may deal with the provider selection menu later; but it seems less important. The patch here contains the changes we really need for Firefox 27 (aurora). This patch obviously bitrots with the patches in bug 935640, so depending on what happens there and when, we may or may not land this on mozilla-central too.

Comment 5

[Florian Quèze [:florian]]

(In reply to Florian Quèze [:florian] [:flo] from comment #4)

> The patch here contains the changes we really need for Firefox 27
> (aurora). This patch obviously bitrots with the patches in bug 935640, so
> depending on what happens there and when, we may or may not land this on
> mozilla-central too.

I discussed this with Shane Friday evening, given that:
- some of the changes here are needed for Firefox 28,
- having the patch on mozilla-central could simplify the uplift,
- Shane's patches in bug 935640 haven't been reviewed yet and Shane doesn't mind dealing with a touch of bitrot...
I decided to land this for Firefox 28:
https://hg.mozilla.org/integration/fx-team/rev/ffd73f058358

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/ffd73f058358

Comment 7

[Florian Quèze [:florian]]

Comment on attachment 8336880 [details] [diff] [review]
Patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): The feature (SocialAPI status buttons) was implemented in bug 891225 and enabled in bug 906839.
User impact if declined: Very confusing icons on social toolbar buttons for social providers using the new status panel feature.
Testing completed (on m-c, etc.): We confirmed that today's m-c nightly with the patch fixes the issue.
Risk to taking this patch (and alternatives if risky): low.
String or IDL/UUID changes made by this patch: none.

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/8dcdb738ddbc

Comment 9

[u279076]

I don't think this needs QA verification. If anyone thinks that's a mistake please remove the [qa-] whiteboard tag and add the verifyme keyword.