Closed
Bug 592338
Opened 14 years ago
Closed 14 years ago
Provide proper feedback when trying to install a persona with a non-default theme
Categories
(Toolkit :: Add-ons Manager, defect)
Toolkit
Add-ons Manager
Tracking
()
VERIFIED
FIXED
mozilla2.0b7
Tracking | Status | |
---|---|---|
blocking2.0 | --- | beta7+ |
People
(Reporter: mossop, Assigned: mossop)
References
Details
(Whiteboard: [strings])
Attachments
(1 file)
12.27 KB,
patch
|
Gavin
:
review+
|
Details | Diff | Splinter Review |
We're not going to do bug 520124 for Firefox 4 but we still need some of the work from there for notifications on installing a persona.
Assignee | ||
Updated•14 years ago
|
blocking2.0: --- → beta6+
Assignee | ||
Comment 1•14 years ago
|
||
This is basically a vastly simpler version of the browser patch in bug 520124. It attempts to apply the theme and watches the events that come from doing that. If enabling the new theme requires a restart it displays a notification offering to do that. This is basically a simpler form of what we have in 3.6 where the add-ons manager would appear and ask the user to restart.
Also includes a fix for another test that was leaving the test theme installed.
Attachment #470999 -
Flags: review?(gavin.sharp)
Assignee | ||
Updated•14 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Updated•14 years ago
|
Whiteboard: [strings]
Comment 2•14 years ago
|
||
Comment on attachment 470999 [details] [diff] [review]
patch rev 1
Is it worth adding aAddon(.id) == newLWTheme(.id) sanity checks to the listener?
Attachment #470999 -
Flags: review?(gavin.sharp) → review+
Assignee | ||
Comment 3•14 years ago
|
||
(In reply to comment #2)
> Comment on attachment 470999 [details] [diff] [review]
> patch rev 1
>
> Is it worth adding aAddon(.id) == newLWTheme(.id) sanity checks to the
> listener?
aAddon and newLWTheme are slightly different objects and for reasons that escape me right now their id's differ so we can't really do this. Given that the code relies on the operation happening synchronously though we can be pretty much sure that nothing else is going to come in in the middle.
Landed: http://hg.mozilla.org/mozilla-central/rev/324ba96157fb
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Flags: in-testsuite+
Flags: in-litmus-
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b6
Comment 4•14 years ago
|
||
So, lightweight themes sometimes trigger a notification bar, and sometimes a doorhanger... sigh...
Comment 5•14 years ago
|
||
Verified fixed with Mozilla/5.0 (Windows NT 5.1; rv:2.0b7pre) Gecko/20100922 Firefox/4.0b7pre
(In reply to comment #4)
> So, lightweight themes sometimes trigger a notification bar, and sometimes a
> doorhanger... sigh...
Neil, please file a new bug for that particular issue. Thanks.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•