User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:1.9.2.7) Gecko/20100701 Firefox/3.6.7 Build Identifier: Personas cannot be installed if they reference a data: URI for their headerURL or footerURL. Full details and examples at http://www.chrisfinke.com/files/personas_data_uri_bug.html Reproducible: Always

Reprocuded with a 3.6.7pre build on OS X. Sadly I can't even see an error in the error console which could point to the problem.

FWIW, if you try to install a data URI persona from Chrome-priveleged code by calling LightWeightThemeWebInstaller._install(theme); directly, you get this error in the console: Error: uncaught exception: 2147942487

I wonder if we support data: URIs at all at the moment. Tony or Dao?

We don't.

Any tips on how to add support for data: URIs to personas? I can try and work out a fix myself, but I'm not at all familiar with any of the personas code. (I'd like to update my True Colors extension to generate dynamic personas instead of manually specifying background images for various elements in chrome.)

Comment on attachment 586729 [details] [diff] [review] v0.1 Allow data URIs but don't persist them on the filesystem Review of attachment 586729 [details] [diff] [review]: ----------------------------------------------------------------- ::: toolkit/mozapps/extensions/LightweightThemeManager.jsm @@ +647,5 @@ > function _sanitizeTheme(aData, aBaseURI, aLocal) { > if (!aData || typeof aData != "object") > return null; > > + var resourceProtocols = ["http", "https", "data"]; For added security we could also test for data URIs that have the mediatype image/* since the default value is text/plain which wouldn't be useful. I'm not sure if there are other types we want to support other than image/* though?

Comment on attachment 586729 [details] [diff] [review] v0.1 Allow data URIs but don't persist them on the filesystem This would store potentially large image data in the lightweightThemes.usedThemes preference... I'm concerned that this will stress the pref system too much, it surely wasn't meant for this kind of data :/

(In reply to Dão Gottwald [:dao] from comment #8) > This would store potentially large image data in the > lightweightThemes.usedThemes preference... I'm concerned that this will > stress the pref system too much, it surely wasn't meant for this kind of > data :/ We probably need something like bug 866238 then.

Per policy at https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Inactive_Bugs. If this bug is not an enhancement request or a bug not present in a supported release of Firefox, then it may be reopened.