<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jul 30, 2015 at 9:46 PM, Benjamin Smedberg <span dir="ltr"><<a href="mailto:benjamin@smedbergs.us" target="_blank">benjamin@smedbergs.us</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
<br>
On 7/30/2015 9:44 PM, Ehsan Akhgari wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Does anyone know why we have the "Firefox default theme" installed by default in Firefox?  In the really old days, the default browser theme was implement as a Theme, and as far as I can tell, the code in browser/app/profile/extensions/{972ce4c6-7e08-4474-a285-3208198ce6fd} is remnants of that old theme (for historical reference, this is the old theme code that I'm referring to: <<a href="https://github.com/mozilla/gecko-dev/tree/203538d39b33bed2e37a49a10b8c3d6e7a26cda5/browser/base/skin" rel="noreferrer" target="_blank">https://github.com/mozilla/gecko-dev/tree/203538d39b33bed2e37a49a10b8c3d6e7a26cda5/browser/base/skin</a>>).<br>
<br>
This unremovable default theme is unnecessary, and it's not even localized.  I'm planning to remove it unless someone has any objections?<br>
</blockquote>
<br></span>
I don't understand what part of this you object to.<br>
<br>
As far as I understand it, the way the current code works you cannot switch back to the default theme from a custom theme in the addons manager unless there is an addon item which represents the default theme.<br></blockquote><div><br></div><div>Oh, you're right!  I don't know how I missed that!<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Are you worried about performance, or general code hygiene, or something else?</blockquote><div><br></div><div>Worried is a strong word.   I was just wondering whether there is a point to it and didn't think of how one should revert from a custom theme back to the default.  Sorry for the false alarm!<br></div></div></div></div>