<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Jul 26, 2017 at 2:34 PM, Kris Maglione <span dir="ltr"><<a href="mailto:kmaglione@mozilla.com" target="_blank">kmaglione@mozilla.com</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"><span class="gmail-">On Wed, Jul 26, 2017 at 11:03:39AM -0700, Brian Grinstead wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
There’s a modal prompt that opens on startup after every clobber asking me if I want to set the local build as my default browser (I don’t). Are there any objections to defaulting browser.shell.checkDefaultBrow<wbr>ser to false in local builds? We already do this for tests.<br>
</blockquote>
<br></span>
Please yes. This is one of my biggest development pet peeves...</blockquote></div><br></div><div class="gmail_extra">Unpopular opinion: I don't think this is a good idea.  The code involved in checking whether we are the default browser has had performance issues (see <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1357146">https://bugzilla.mozilla.org/show_bug.cgi?id=1357146</a>) and is in my humble opinion not so nice behavior which we've wanted to fix (<a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1143116">https://bugzilla.mozilla.org/show_bug.cgi?id=1143116</a>) for all users.  Hiding this from our developers seems like a disservice to our users, especially to the new users who we are hoping to attract to Firefox as we are working on improving the browser.</div><div class="gmail_extra"><br></div><div class="gmail_extra">I understand the desire to do the easy thing and put a band-aid on where we are bleeding for now, but perhaps we should at least consider this as a reminder to check back with the right folks working on the original effort to fix this properly for all users?<br clear="all"></div><div class="gmail_extra"><br>-- <br><div class="gmail_signature"><div dir="ltr">Ehsan<br></div></div>
</div></div>