Mark Banner (:standard8) Reporter
	Description • 13 years ago

+++ This bug was initially created as a clone of Bug #553092 +++

Bug 553092 did some work to improve initialisation of the permissions manager on startup.

We should port that to Thunderbird - we basically need to port the browser.js part (to LightWeightThemeWebInstaller in msgMail3PaneWindow.js) and the permissions.js part to the like file in Thunderbird.

	:aceman Assignee
	Comment 1 • 13 years ago

What is the way to test this?

	Mark Banner (:standard8) Reporter
	Comment 2 • 13 years ago

You probably want to check that themes installed from somewhere listed in the whitelist prefs aren't blocked:

http://mxr.mozilla.org/comm-central/search?string=xpinstall.whitelist.add&find=%2Fprofile%2F&findi=&filter=^[^\0]*%24&hitlimit=&tree=comm-central

For the permissions.js part, I don't think we actually display that dialog in Thunderbird atm.

	:aceman Assignee
	Comment 3 • 13 years ago

You mean they should work the same before and after the change?
How can I install themes in Thunderbird?

	Mark Banner (:standard8) Reporter
	Comment 4 • 13 years ago

(In reply to :aceman from comment #3)
> You mean they should work the same before and after the change?

Yep

> How can I install themes in Thunderbird?

Tools -> Addons -> Get Add-ons -> See All Themes -> Personas -> Select a persona

(needs to be a persona rather than a general theme though).

	:aceman Assignee
	Comment 5 • 13 years ago

Ok, let's try it.

	:aceman Assignee
	Comment 6 • 13 years ago

Strange, those 2 prefs are reset to "" when I visit that site offering personas.
But the personas still apply (preview).

	:aceman Assignee
	Comment 7 • 13 years ago

After applying this and trying the steps in comment 4 the personas still apply, without asking.

	Mark Banner (:standard8) Reporter
	Comment 8 • 13 years ago

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

For future reference, to fully test this, you either need a new profile, or to nuke permissions.sqlite and then test installing personas.

I did this after a bit of trial and error, and doubly confirmed that everything still works as expected, so r=me.

	:aceman Assignee
	Comment 9 • 13 years ago

Thanks.

	Ryan VanderMeulen [:RyanVM]
	Comment 10 • 13 years ago

http://hg.mozilla.org/comm-central/rev/8adfa5fe33fa