<div dir="ltr">For these types of one-offs I have patches in the front of my queue. For this I'd just add a pref change so the default is false.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 26, 2017 at 12:48 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, Jul 26, 2017 at 03:35:18PM -0400, Ehsan Akhgari wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Unpopular opinion: I don't think this is a good idea.  The code involved in<br>
checking whether we are the default browser has had performance issues (see<br>
<a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1357146" rel="noreferrer" target="_blank">https://bugzilla.mozilla.org/s<wbr>how_bug.cgi?id=1357146</a>) and is in my humble<br>
opinion not so nice behavior which we've wanted to fix (<br>
<a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1143116" rel="noreferrer" target="_blank">https://bugzilla.mozilla.org/s<wbr>how_bug.cgi?id=1143116</a>) for all users.<br>
Hiding this from our developers seems like a disservice to our users,<br>
especially to the new users who we are hoping to attract to Firefox as we<br>
are working on improving the browser.<br>
</blockquote>
<br></span>
Fair enough, but my issue is that I see and have to deal with this dialog during development much more often than ordinary users do in their daily usage. Sometimes many times in a single day, often in debug builds, where dismissing it takes much longer than it normally would.<br>
<br>
Fixing it for users would be nice, but developer workflows are and often should be different, and I think we should consider changing default behaviors for development runs when they make development significantly easier. And in the particular case of the default browser check... even if we improve the user experience, it doesn't really ever make sense from `mach run` on a development build, unless we explicitly need it in order to test something.<div class="HOEnZb"><div class="h5"><br>
______________________________<wbr>_________________<br>
firefox-dev mailing list<br>
<a href="mailto:firefox-dev@mozilla.org" target="_blank">firefox-dev@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/firefox-dev" rel="noreferrer" target="_blank">https://mail.mozilla.org/listi<wbr>nfo/firefox-dev</a><br>
</div></div></blockquote></div><br></div>