<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Great, I filed <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1385444">https://bugzilla.mozilla.org/show_bug.cgi?id=1385444</a> to track this work under the <span id="gmail-field-value-blocked"><a class="gmail-bz_bug_link gmail-bz_status_NEW" title="NEW - Stuff we can remove when XPCOM extensions are no longer supported" href="https://bugzilla.mozilla.org/show_bug.cgi?id=1347507">post-57-api-changes</a></span><span id="gmail-field-value-blocked"> meta bug.</span>
    </div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 23, 2017 at 1:44 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="gmail-">On Thu, Mar 16, 2017 at 7:48 PM, Matthew N. <span dir="ltr"><<a href="mailto:MattN@mozilla.com" target="_blank">MattN@mozilla.com</a>></span> wrote:<br><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div style="font-family:arial,helvetica,sans-serif">Once we <a href="https://wiki.mozilla.org/Add-ons/Themes/FAQ#What.E2.80.99s_going_to_happen_to_existing_themes.3F" target="_blank">remove complete themes in 57</a> I think this should change since the primary factor for the separation will be gone.</div></div></blockquote><div><br></div><br> <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div style="font-family:arial,helvetica,sans-serif"> I think the reasons are obvious (e.g. duplication of selectors between files, confusing for new contributors, etc.) so I won't go into those now. I think there are two high-level paths forward:<br>A) Consolidate all styles in one package (I think it would be "content" but see discussion below¹)<br></div><div style="font-family:arial,helvetica,sans-serif">** We can still use a preprocessor in CSS or in jar manifests to vary OS-specific styles<br></div></div></blockquote></span><div><div><br>I support this model, and it dovetails with my desire to get rid of all the skin-selection code in the chrome registry. I would like to end up in a state where chrome://package/skin/ doesn't exist.<br></div><br> </div><span class="gmail-"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div style="font-family:arial,helvetica,sans-serif"></div><br><div style="font-family:arial,helvetica,sans-serif"><br></div><div style="font-family:arial,helvetica,sans-serif">¹ The "content" package is chrome-privileged whereas the "skin" package isn't (even though they both use chrome URIs). The chrome privileges are sometimes necessary with our style to avoid cross-origin issues related to SVG and possibly other examples I don't know off-hand. Is there any practical security benefit to using "skin" for everything (if possible)?<br><br></div><div style="font-family:arial,helvetica,sans-serif">What do others think about changing/removing the "skin" vs. "content" distinction?<br><br>I personally think (B) is simplest to transition to while still providing a clear separation for OS-specific styles so I lean towards it but if we can come up with a good convention for handling OS-specific styles with option (A) then that's also fine with me.<br></div></div></blockquote><div><br></div></span><div>If it's possible to do this using CSS selectors optionally @importing OS/OSversion specific stylesheets, I'd much prefer this version.<br><br></div><div>The reason we have in the past removed chrome privileges from skin packages is so that Firefox themes can't pwn users (or can't pwn users as easily). If it worked it would be a nice risk mitigation technique, but in practice it doesn't work and so let's not worry about that.<br><br></div><div>--BDS<br></div></div></div></div></blockquote></div></div></div>