Firefox/Meeting/23-Mar-2021: Difference between revisions

From MozillaWiki
Jump to navigation Jump to search
(Add meeting notes for PDFs & Printing)
(Add meeting notes for Performance)
Line 147: Line 147:


==Performance==
==Performance==
<ul>
<li>dthayer improved the [https://bugzilla.mozilla.org/show_bug.cgi?id=1678411 UX of slow script notifications] in Fission</li>
<li>dthayer is also working on various [https://bugzilla.mozilla.org/show_bug.cgi?id=1695674 improvements] and [https://bugzilla.mozilla.org/show_bug.cgi?id=1697091 bug-fixes] for the Pre-XUL Skeleton UI</li>
<li>florian added some very handy profiler markers for CSS animations and transitions. [https://share.firefox.dev/2QaUjTK Here’s an example profile].</li>
<li>The Pre-XUL Skeleton UI experiment will be wrapping up next week, and we hope to have some data on how it impacts new and existing user behaviour</li>
<li>The Process Priority Manager was originally going to be disabled for Fission, but it turns out that for weaker hardware, it seems to really help with pageload time. The Fission team is looking into how to adapt the Process Priority Manager to work for subframes.</li>
<li>mconley spoke with k88hudson about using Nimbus / ExperimentsAPI to run an about:home startup cache experiment
<ul>
<li>Originally, we had a Normandy experiment set up for about:home startup cache, but it turns out Normandy sets its prefs too late. Nimbus seems to do it earlier, which is good for us.</li>
</ul>
</li>
<li>barret [https://bugzilla.mozilla.org/show_bug.cgi?id=1672431 fixed an AsyncShutdown bug for IOUtils], which should be the final blocker to remove the remainder of the OSFile uses in the tree! \o/</li>
</ul>


==Performance Tools==
==Performance Tools==

Revision as of 17:46, 23 March 2021

Today’s meeting leader is: mtigley

General Topics / Roundtable

Friends of the Firefox team

Introductions/Shout-Outs

Resolved bugs (excluding employees)

Fixed more than one bug

  • Andrei Petcu [:petcuandrei]
  • Erica Wright [:ewright]
  • Sebastian Zartner [:sebo]
  • Tim Nguyen :ntim
  • Tom Schuster [:evilpie]

New contributors (🌟 = first patch)

  • 🌟 Karthik Sundar [karthiksundar30092002] fixed a UI issue where expanding objects in the DevTools console made them jump around (Bug 1608571)
  • 🌟 David [heftydav] fixed an issue where the Picture-in-Picture icon will disappear when dragging the tab to a new window (Bug 1669205)
  • 🌟 Henry Vincent [henryvincent33] fixed a typo in an error message created by DevTools’ Front class (Bug 1699146)
  • 🌟 Renuka Bhure [bhurerenuka14] replaced BrowserTestUtils.waitForCondition with TestUtils.waitForCondition in browser_deleteLogin.js (Bug 1698812)
  • 🌟 Sneha sai KNVS [snehaa2296] replaced BrowserTestUtils.waitForCondition with TestUtils.waitForCondition in browser_protectionsUI_report_breakage.js (Bug 1698076)

Project Updates

Add-ons / Web Extensions

Addon Manager & about:addons

  • Mark adjusted addon signature warning messagebar to make it easier to read on hover (Bug 1694428)

WebExtensions Framework

  • Fission-related fixes:

    • Fix DocumentChannel process switching on sidebar and popup extension pages (Bug 1646817)
    • Logging a deprecation warning on extensions using fission-incompatible canvas drawWindow (Bug 1696976)

      • As also described in the warning, extensions should use Use tabs.captureTab extensions API instead (which is fission compatible)
    • Support for OOP iframes in webNavigation.getAllFrames (Bug 1698398)
  • Starting from Firefox 88 Services is exposed by default in all WebExtensions API scripts (included the experimental APIs embedded in privileged and builtin addons) - Bug 1698158:

WebExtension APIs

  • As part of the FTP removal in Firefox 88 (Bug 1626365):
    • browserSettings.ftpProtocolEnable becomes read only
    • β€œftp” has been added to the list of allowed protocols that extension can register as a protocol handler
    • in the proxy WebExtensions API, proxying ftp is deprecated

Developer Tools

Fission

  • Decided not to fix the disabled WebPayments UI for fission.
  • Closed the frontend fission metabug! πŸŽ‰πŸŽ‰πŸŽ‰πŸŽ‰πŸŽ‰πŸŽ‰πŸŽ‰
  • Fission beta experiments will start this week (Firefox 88)

Form Autofill

Installer & Updater

  • Background update agent work continues:
  • Lint

  • If you run `./mach lint` in the top-level directory of a repository, with no paths specified, it will now default to `--outgoing --workdir`, rather than doing a full lint.
  • Messaging System

    Mobile

    New Tab Page

    Nimbus

    NodeJS

    New Tab Page

    Password Manager

    • Bug 1688213 Enable "signon.management.page.fileImport.enabled" for all channels fixed and in 88.
      • Import logins from CSV is in beta! Congrats to :petcuandrei for taking this feature from a UX spec and aspirational bugs to implementing and now shipping a feature that rounds out our whole story around login management.
      • We now import and export logins, allowing you to round-trip from other browsers, other password managers, backed-up profiles or any other exotic scenario for which you can prepare a comma-separated values sheet, with at least password and URL columns for successful import into Firefox.
      • We also have the ability to delete all, so the cost is low to play around with it and file bugs.
    • :dimi landed Bug 1166995 - Run login capture code when a form or a formless password field is removed from the document tree. This fixed some long standing issues with prompting to save passwords on sites e.g bigcommerce.com, and no doubt many many others we were never notified about
    • Several string, style and context menu changes ahead of MR1/Proton, thanks especially to Prathiksha for jumping in to help


    Screenshot of the Import Summary page

    PDFs & Printing

  • Bug 1682162 - Print margin unit should be localized. [sfoster] landed this in 88, so now margin units are displayed according to the unit indicated by your printer. Mms rule.
  • Performance

    • dthayer improved the UX of slow script notifications in Fission
    • dthayer is also working on various improvements and bug-fixes for the Pre-XUL Skeleton UI
    • florian added some very handy profiler markers for CSS animations and transitions. Here’s an example profile.
    • The Pre-XUL Skeleton UI experiment will be wrapping up next week, and we hope to have some data on how it impacts new and existing user behaviour
    • The Process Priority Manager was originally going to be disabled for Fission, but it turns out that for weaker hardware, it seems to really help with pageload time. The Fission team is looking into how to adapt the Process Priority Manager to work for subframes.
    • mconley spoke with k88hudson about using Nimbus / ExperimentsAPI to run an about:home startup cache experiment
      • Originally, we had a Normandy experiment set up for about:home startup cache, but it turns out Normandy sets its prefs too late. Nimbus seems to do it earlier, which is good for us.
    • barret fixed an AsyncShutdown bug for IOUtils, which should be the final blocker to remove the remainder of the OSFile uses in the tree! \o/

    Performance Tools

    Picture-in-Picture

    Pocket

    Privacy/Security

    Push

    Search and Navigation

    Sync

    User Journey

    This week I learned