Closed Bug 1191008 Opened 4 years ago Closed 4 years ago

Notification to users about disabled signed add-ons on startup only displays on one window

Categories

(Toolkit :: Add-ons Manager, defect)

defect
Not set

Tracking

()

VERIFIED FIXED
mozilla42
Tracking Status
firefox41 + verified
firefox42 + verified

People

(Reporter: mossop, Assigned: mossop)

References

Details

Attachments

(1 file, 1 obsolete file)

When a user starts a Firefox session with multiple windows and we show the notification about disabling some of their signed add-ons we only display the notification on one of the windows and at least in some cases (maybe all) it won't be the foreground window.

Markus, showing the notification on the foreground window is the obvious thing here, should we also be showing it on all the other windows?
Flags: needinfo?(mjaritz)
Assignee: nobody → dtownsend
Tracked for 41 and 42, to ensure the notification is visible to the user in the released version.
Attached patch patch (obsolete) — Splinter Review
This moves the check to after all windows have been restored so we display on the front-most window. This is a pre-requisite to displaying on all windows if that is what UX want so might as well do this now and do all windows in a follow-up.
Attachment #8643842 - Flags: review?(dao)
Attached patch patchSplinter Review
Accidentally snuck in another change
Attachment #8643842 - Attachment is obsolete: true
Attachment #8643842 - Flags: review?(dao)
Attachment #8643846 - Flags: review?(dao)
For release, showing it in the foreground window is perfect.
For pre-release it would be great to have it more prominently on all windows. (If we can clear the notification from all windows once the user acts on it in one of the windows.)
This might help make this message more prominent for add-on developers and ensure that they do not miss it, even if the foreground window was not relevant and quickly closed.
If it is not possible to differentiate the behavior in release/pre-release showing the notification in the foreground window is fine.
Flags: needinfo?(mjaritz)
Attachment #8643846 - Flags: review?(dao) → review+
https://hg.mozilla.org/mozilla-central/rev/dac4fbb3f906
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
QA Contact: vasilica.mihasca
Comment on attachment 8643846 [details] [diff] [review]
patch

Approval Request Comment
[Feature/regressing bug #]: Signed add-ons
[User impact if declined]: Some users may not see the notification about unsigned add-ons being disabled as it would appear on a background window
[Describe test coverage new/current, TreeHerder]: Tested manually, we don't have tests for the feature during Firefox startup
[Risks and why]: The change is a straightforward code move so low risk
[String/UUID change made/needed]: None
Attachment #8643846 - Flags: approval-mozilla-aurora?
Comment on attachment 8643846 [details] [diff] [review]
patch

Let's uplift to Aurora as the fix seems straightforward.
Attachment #8643846 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Flags: qe-verify+
Verified fixed on Firefox 41 Beta 2 (20150817163452) and Firefox 42.0a2 (2015-08-20) under Ubuntu 14.04 32-bit and Windows 7 64-bit. 

There is an exception when the notification is not displayed on the foreground window. This happens in Firefox 42.0a2 when the Developer Edition firstrun page is the only opened tab in one of the opened windows. Updating the Developer Edition from a version where the add-ons signatures is not required (e.g Firefox 41.0a2 from 2015-07-21) to a latest version where the pref xpinstall.signatures.required is set to true, the foreground window in this situation is that window which contains the Dev Edition firstrun tab. The notification about disabling some of their signed add-ons is displayed in the second window.

See screenshot: http://i.imgur.com/meernQY.jpg

But this is an edge case and I am not sure if it’s worth filed a bug for this issue. Dave what do you think?
Flags: needinfo?(dtownsend)
I'm guessing that the use of the UI tour is bringing that window to the front after we've already shown the notification elsewhere. Presumably this requires the user to already have a window like that open before they update? In that case I think we don't need to track it.
Flags: needinfo?(dtownsend)
(In reply to Dave Townsend [:mossop] from comment #11)
> I'm guessing that the use of the UI tour is bringing that window to the
> front after we've already shown the notification elsewhere. Presumably this
> requires the user to already have a window like that open before they
> update? In that case I think we don't need to track it.

Yes, that window needs to be opened before update in order to reproduce this issue.

Based on the above mentioned I am marking this bug as Verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.