<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 28, 2017 at 5:22 AM, Gijs Kruitbosch <span dir="ltr"><<a href="mailto:gkruitbosch@mozilla.com" target="_blank">gkruitbosch@mozilla.com</a>></span> wrote:<br><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 28/03/2017 00:51, Justin Dolske wrote:<br></span><span class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The other option is to land Photon pieces behind prefs. This is the option I'd strongly prefer, as it lets us get Nightly testing and the benefits of our normal infra come by default. Of course, nothing is free... Some things are difficult to put behind a pref. And we'd likely need to run a non-Photon project branch, to make sure Aurora isn't busted after an uplift.<br>
</blockquote>
<br></span>
I would actually probably prefer ifdefs over prefs for some things. As you note, some things (e.g. XUL layout and CSS) are a bit tricky to adjust solely based on a pref. Ehsan also recently posted to dev-platform about the runtime impact of prefs (though we could improve that by using defineLazyPreferenceGetter).<br></blockquote><div><br></div><div>Yeah, #ifdefs are fine. The primary goal is just to make sure we can get testing of Photon stuff on Nightly-55, Nightly-56, and the 57 train, while being able to turn it off elsewhere. Prefs have a few (obvious?) advantages when they're feasible, but otherwise I'm not too concerned about the specific mechanism.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
If we end up e.g. forking browser.xul and/or some of the XBL bindings behind a pref (similar to what we're doing with about:preferences), we lose a lot of the benefits of not using a project branch, in that we'd still have to merge all the things all the time, except then we'd have to do it manually. :-(<span class=""><br></span></blockquote><br></div><div class="gmail_quote">Yes, that's one of the costs. I don't see a better alternative. (So be mindful about forking, and carefully consider if alternatives might have a bit more up-front effort but be a win overall.)<br></div><div class="gmail_quote"><br></div><div class="gmail_quote">Justin<br></div></div></div>