<div dir="ltr"><div>Hi Cory,<br><br></div><div>It's exciting to see that this feature is moving on to be released!<br></div><div><br></div>Has there been any performance testing on Firefox with the screenshots extension installed?  I'd be curious to know some details about how this new feature impacts the performance of Firefox, especially since this is targeting to be released in Firefox 54, skipping the Nightly channel that is now 55, and WebExtension add-ons do not yet run in their own dedicated process, so I expect that the majority of the code for this add-on will run in the parent process, where performance issues can cause UI jank.  As part of the Quantum Flow project, we are working very hard to improve the performance of Firefox, and a large part of this effort is focused on improving the performance of the browser front-end, and part of this effort needs to be ensuring that new front-end features not regressing the performance before the upcoming Firefox 57 release.  Based on that, I'd be curious to know more about:<br><br><ul><li>Are there some profile links showing what the startup/shutdown of the browser looks like with this extension installed?</li><li>Are there some profile links showing what the impact of the extension to the browser's idle time looks like?  Does the extension need to run code in the background, either lazily or periodically for certain tasks?</li><li>Are there some profile links showing the performance of the extension as the screenshot feature itself is being used?</li><li>Does the extension need to do I/O?  How does it achieve that?  If it uses sqlite, has there been an analysis on the sqlite query performance?  If it uses other types of storage, has there been analysis on the performance of other such mechanisms?</li><li>Does the extension need to do expensive image manipulation on the main thread?</li><li>Are there any other details which would be interesting to know from a performance perspective?</li></ul><p>Thanks,</p><p>Ehsan<br></p></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 29, 2017 at 11:51 AM, Cory Price <span dir="ltr"><<a href="mailto:cprice@mozilla.com" target="_blank">cprice@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">This is a notice of intent to implement the Firefox Screenshots system add-on in desktop Firefox.<br><br>Links:<br><br>- Bug: <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1351424" target="_blank">https://bugzilla.mozilla.org/<wbr>show_bug.cgi?id=1351424</a><br>- Repo: <a href="https://github.com/mozilla-services/screenshots/issues" target="_blank">https://github.com/mozilla-<wbr>services/screenshots/issues</a><br>- Wiki: <a href="https://wiki.mozilla.org/Firefox/Screenshots" target="_blank">https://wiki.mozilla.org/<wbr>Firefox/Screenshots</a> <br>- IRC: #screenshots<br>- Project deliverables / tracking: <a href="https://docs.google.com/spreadsheets/d/1axvrNaDkdA0KaadntiZH3F_rsAew72V4ZhBLqCADUpU/edit#gid=0" target="_blank">https://docs.google.com/<wbr>spreadsheets/d/<wbr>1axvrNaDkdA0KaadntiZH3F_<wbr>rsAew72V4ZhBLqCADUpU/edit#gid=<wbr>0</a> (note the spreadsheet is only accessible to Mozilla employees)<br><br>Overview:<br><br>Firefox Screenshots (previously called Page Shot in Test Pilot) integrates screenshot and share-to-web in Firefox with powerful search features to find past shots.<br><br>It will be shipped as a Go Faster system add-on, and also live in the Firefox tree.  Periodic releases will be imported into the tree.<br><br>Firefox Screenshots will be an Embedded WebExtension, with most of the functionality implemented in a WebExtension and a bootstrap.js wrapper to read Telemetry preferences, manage migration from the Test Pilot Page Shot add-on, and to allow users to disable the WebExtension.<br><br>Errors that occur in the Screenshots code will be reported to Mozilla’s Sentry instance.  Sentry is a server application specifically for collecting and reporting on exceptions. Behavioral events from the add-on are submitted to the Screenshots server. Both kinds of reporting are disabled if Telemetry is disabled.<br><br>Architectural documentation is in docs/ and addon/ in the repository.<br><br>For the purposes of Sentry submission we include the Raven library in the WebExtension <<a href="https://github.com/getsentry/raven-js" target="_blank">https://github.com/getsentry/<wbr>raven-js</a>>.  No other external libraries are used.<br><br>Timeline:<br><br>We are currently targeting Firefox 54, with the hopes of landing a fully functional MVP in advance of Firefox 54 Beta, by April 18th, 2017.<br><br>Team:<br><br>Ian Bicking: Engineering<br>Jared Hirsch: Engineering<br>Danny Coates: Engineering<br>Wil Clouser: Engineering Management<br>John Gruen: Product Management<br>Cory Price: Program Management<br>Michelle Heubusch: Content Strategy<br>Peter DeHaan: QA<br>Morpheus Chen: UX<br>Fang Shih: UX<br>And more: <a href="https://github.com/mozilla-services/screenshots/blob/master/CONTRIBUTORS.md" target="_blank">https://github.com/mozilla-<wbr>services/screenshots/blob/<wbr>master/CONTRIBUTORS.md</a> <br clear="all"><span class="HOEnZb"><font color="#888888"><br>-- <br><div class="m_-3788908438440778884gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Cory Price<br></div>/ckprice<br></div></div></div></div>
</font></span></div>
<br>______________________________<wbr>_________________<br>
firefox-dev mailing list<br>
<a href="mailto:firefox-dev@mozilla.org">firefox-dev@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/firefox-dev" rel="noreferrer" target="_blank">https://mail.mozilla.org/<wbr>listinfo/firefox-dev</a><br>
<br></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Ehsan<br></div></div>
</div>