<div dir="ltr"><div><div><div><div>Hi Wil and the team,<br><br></div>Thanks for the investigations so far!<br><br></div>Did the work in <a href="https://github.com/mozilla-services/screenshots/issues/2554">https://github.com/mozilla-services/screenshots/issues/2554</a> lead to uncovering any issues?<br><br></div>Thanks,<br></div>Ehsan<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Apr 6, 2017 at 2:31 PM, Wil Clouser <span dir="ltr"><<a href="mailto:wclouser@mozilla.com" target="_blank">wclouser@mozilla.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>That issue is closed (Thanks Jared) and I don't know of any other asks here so I'm assuming we're good to go.  Thanks everyone.<span class="HOEnZb"><font color="#888888"><br><br></font></span></div><span class="HOEnZb"><font color="#888888">Wil<br></font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 5, 2017 at 3:28 PM, Wil Clouser <span dir="ltr"><<a href="mailto:wclouser@mozilla.com" target="_blank">wclouser@mozilla.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>Thanks for the discussion and drilling down to actionable items.  This is filed as <a href="https://github.com/mozilla-services/screenshots/issues/2575" target="_blank">https://github.com/mozilla-ser<wbr>vices/screenshots/issues/2575</a><br><br></div>Cheers,<br><br></div>Wil<br></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="m_9192462554159117686h5">On Wed, Apr 5, 2017 at 1:50 PM, Jared Hirsch <span dir="ltr"><<a href="mailto:6a68@mozilla.com" target="_blank">6a68@mozilla.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="m_9192462554159117686h5"><div dir="ltr"><div>Talked this over with Kris and Dave in IRC just now. I'm going to change bootstrap.js to:<br>(1) lazy load all the Cu.imports;<br>(2) delay WebExtension startup until the "<span class="m_9192462554159117686m_-7534841132399234522m_1396738182159357671gmail-message"><span class="m_9192462554159117686m_-7534841132399234522m_1396738182159357671gmail-content">sessionstore-windows-restored<wbr>" event.<br><br></span></span></div><span class="m_9192462554159117686m_-7534841132399234522m_1396738182159357671gmail-message"><span class="m_9192462554159117686m_-7534841132399234522m_1396738182159357671gmail-content">Hopefully this will address the startup performance regression.<br></span></span></div><div class="m_9192462554159117686m_-7534841132399234522HOEnZb"><div class="m_9192462554159117686m_-7534841132399234522h5"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 5, 2017 at 1:02 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>On Wed, Apr 05, 2017 at 12:57:39PM -0700, Dave Townsend wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
AddonManager is already loaded by this point so importing it should cost<br>
nothing. I expect we load Console.jsm elsewhere during startup already too.<br>
</blockquote>
<br></span>
Note that calling Cu.import isn't free, even if the module is already loaded. defineLazyModule getter isn't free easier, but it's much cheaper than calling into XPConnect.<br>
</blockquote></div><br></div>
</div></div><br></div></div><span>______________________________<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>
<br></span></blockquote></div><br></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Ehsan<br></div></div>
</div>