Firefox/Meeting/07-Mar-2023: Difference between revisions

From MozillaWiki
Jump to navigation Jump to search
(Added content for Mar 7 2023)
 
(Updated Mar 7 2023)
 
Line 1: Line 1:
Today’s meeting leader is: '''kpatenio'''
Today’s meeting leader is: '''kpatenio'''
Today’s meeting leader is: '''kpatenio'''


Line 34: Line 36:
<ul>
<ul>
<li><blockquote><p>We need people to sign-up for this - slot your name in!</p></blockquote></li></ul>
<li><blockquote><p>We need people to sign-up for this - slot your name in!</p></blockquote></li></ul>
</li></ul>
== Project Updates ==
=== Add-ons / Web Extensions ===
Addon Manager &amp; about:addons
* Starting from Firefox 112, users can now search for text inside the about:addons page (through the usual associated keyboard shortcuts, e.g. Ctrl-F and ‘/’) - [https://bugzilla.mozilla.org/show_bug.cgi?id=1499500 ''Bug 1499500'']
WebExtensions Framework
<ul>
<li>Thanks to Itiel, the default extension icons in the extensions button panel are now filled with the current theme color - [https://bugzilla.mozilla.org/show_bug.cgi?id=1817865 ''Bug 1817865'']</li>
<li><p>In Nightly 112 we migrated the kvstore path used by ExtensionPermissions, to be sure that the kvstore path isn’t shared with the one used internally by ExtensionScriptingStore - [https://bugzilla.mozilla.org/show_bug.cgi?id=1807010 ''Bug 1807010'']</p>
<ul>
<li><blockquote><p>'''NOTE: '''the underlying race issue was investigated and confirmed as part of [https://bugzilla.mozilla.org/show_bug.cgi?id=1805427 ''Bug 1805427''], and so be aware that despites the inline comment associated to nsIKeyValueService.getOrCreate in nsIKeyValue.idl states that it can be used for multiple database names stored in the same kvstore path, that isn’t thread safe with the SafeMode (and old LMDB mode is deprecated).</p></blockquote></li></ul>
</li></ul>
WebExtension APIs
* Starting from Firefox 112, downloads.removeFile API calls that fail will be logging more detailed info about the actual underlying issue in the browser console - [https://bugzilla.mozilla.org/show_bug.cgi?id=1807815 ''Bug 1807815'']
=== Developer Tools ===
=== DevTools ===
<ul>
<li><p>External contributors:</p>
<ul>
<li><blockquote><p>khadija fixed a visual issue in the netmonitor ([https://bugzilla.mozilla.org/show_bug.cgi?id=1809912 ''bug''])</p></blockquote></li>
<li><blockquote><p>marlene replaced wrong parameter value in calls to CodeMirror method from the debugger ([https://bugzilla.mozilla.org/show_bug.cgi?id=1819355 ''bug''])</p></blockquote></li></ul>
</li>
<li><p>Contributions from other teams:</p>
<ul>
<li><blockquote><p>bradwerth improved performance of the grid highlighter when there are a lot of columns in the grid ([https://bugzilla.mozilla.org/show_bug.cgi?id=1815140 ''bug''])</p></blockquote></li>
<li><blockquote><p>Emilio fixed a scaling issue in the inspector tooltip ([https://bugzilla.mozilla.org/show_bug.cgi?id=1817381 ''bug''])</p></blockquote></li></ul>
</li>
<li><p>Hubert added search modifier in project search in the debugger ([https://bugzilla.mozilla.org/show_bug.cgi?id=1817984 ''bug''])</p>
<ul>
<li><blockquote><p>[https://snipboard.io/trvkWu.jpg ''screenshot''] </p></blockquote></li></ul>
</li>
<li>Hubert also fixed an issue with Navigator.sendBeacon requests that were shown as Blocked in the Netmonitor even if they went through ([https://bugzilla.mozilla.org/show_bug.cgi?id=1819279 ''bug''])</li>
<li>Julian recovered a regression that broke network throttling in Netmonitor ([https://bugzilla.mozilla.org/show_bug.cgi?id=1817659 ''bug''])</li>
<li>Nicolas removed duplicated ChromeWorker messages in Browser Console ([https://bugzilla.mozilla.org/show_bug.cgi?id=1818228 ''bug''])</li>
<li>Nicolas improved performance of pretty printing in debugger ([https://bugzilla.mozilla.org/show_bug.cgi?id=1819350 ''bug''], [https://bugzilla.mozilla.org/show_bug.cgi?id=1819269 ''bug''])</li>
<li>Alex made it possible to copy text from Netmonitor HTML preview again ([https://bugzilla.mozilla.org/show_bug.cgi?id=1817857 ''bug''])</li>
<li>Alex landed a patch that does some groundwork ([https://bugzilla.mozilla.org/show_bug.cgi?id=1704690 ''bug'']) for a new feature in the debugger that will let the user override HTTP responses with a local file ([https://bugzilla.mozilla.org/show_bug.cgi?id=1818806 ''bug'']), which is in progress.</li></ul>
=== WebDriver BiDi ===
* Sasha added the browsingContext.print command, which produces a base64-encoded PDF representation of the document ([https://bugzilla.mozilla.org/show_bug.cgi?id=1806810 ''bug''])
* Henrik fixed an issue where TabSession.executeInChild would hang the message manager ([https://bugzilla.mozilla.org/show_bug.cgi?id=1819965 ''bug''])
* Henrik vendored the latest version of puppeteer (19.7.2) which allows to run more tests in CI and have a better coverage of CDP ([https://bugzilla.mozilla.org/show_bug.cgi?id=1817934 ''bug''])
* Henrik also fixed an issue where httpd.js listener wouldn't stop, which could lead to unexpected side effects ([https://bugzilla.mozilla.org/show_bug.cgi?id=1820487 ''bug''])
* Julian created a shared module to generate HAR files from WebDriver BiDi network events, which could be useful for libraries that are using BiDi like Browsertime and Selenium ([https://bugzilla.mozilla.org/show_bug.cgi?id=1817818 ''bug''])
=== ESMification status ===
<ul>
<li>Support for ES modules in workers has landed on nightly, but [https://bugzilla.mozilla.org/show_bug.cgi?id=1812591 ''not yet been enabled for non-nightly''].</li>
<li><p>[https://spidermonkey.dev/areweesmifiedyet/ ''ESMified''] status:</p>
<ul>
<li><blockquote><p>browser: 52.4%</p></blockquote></li>
<li><blockquote><p>toolkit: 39.2%</p></blockquote></li>
<li><blockquote><p>Total: 55.1%</p></blockquote></li></ul>
</li>
<li>#esmification on Matrix</li>
<li>[https://docs.google.com/document/d/1cpzIK-BdP7u6RJSar-Z955GV--2Rj8V4x2vl34m36Go/edit?usp=sharing ''Migration Document''] (with a walkthrough!)</li></ul>
=== Migration Improvements (CalState LA Project) ===
<ul>
<li>Ani [https://bugzilla.mozilla.org/show_bug.cgi?id=1467094 ''made it possible to import from Chromium on Ubuntu when Chromium is installed as a Snap'']!</li>
<li>Steven made it so that [https://bugzilla.mozilla.org/show_bug.cgi?id=1816690 ''the resource type checkboxes default to checked''], and [https://bugzilla.mozilla.org/show_bug.cgi?id=1815444 ''fixed some styling issues''] in the new Migration Wizard</li>
<li>Brian [https://bugzilla.mozilla.org/show_bug.cgi?id=1809347 ''added a page''] to the new Migration Wizard that shows an error message if no valid browser / profile pairs can be found to import from.</li>
<li><p>Some changes that are landing after the soft-freeze lifts:</p>
<ul>
<li><blockquote><p>Alvin [https://bugzilla.mozilla.org/show_bug.cgi?id=1815320 ''updated the mechanism for getting permission to import from Safari''] to read from the entire Safari directory, rather than just the .plist files.</p></blockquote></li>
<li><blockquote><p>Alvin finished the [https://bugzilla.mozilla.org/show_bug.cgi?id=326701 ''Safari bookmark favicon import mechanism'']</p></blockquote></li>
<li><blockquote><p>Bryan finished the [https://bugzilla.mozilla.org/show_bug.cgi?id=711995 ''Chrome (and Chrome-based browser) bookmark favicon import mechanism'']</p></blockquote></li></ul>
</li>
<li>[https://bugzilla.mozilla.org/show_bug.cgi?id=1801313 ''The new Migration Wizard''] is somewhat functional but still very much a work-in-progress. It can be enabled by setting browser.migrate.content-modal.enabled to true. [https://firefox-source-docs.mozilla.org/browser/components/migration/docs/migration-wizard.html ''Documentation can be found here.'']</li></ul>
=== Performance ===
<ul>
<li><p>[https://treeherder.mozilla.org/perfherder/graphs?highlightAlerts=1&highlightChangelogData=1&highlightCommonAlerts=0&series=autoland,3912818,1,13&timerange=5184000 ''Some recent wins''] on the Speedometer performance benchmark on Windows:</p>
<ul>
<li><blockquote><p></p></blockquote>
<ul>
<li>(Higher is better)</li></ul>
</li>
<li><blockquote><p>The two big jumps are from these [https://bugzilla.mozilla.org/show_bug.cgi?id=1444491 ''two''] [https://bugzilla.mozilla.org/show_bug.cgi?id=1815069 ''fixes'']. Great job, Performance Team!</p></blockquote></li></ul>
</li>
<li>Congratulations to Glenn Watson (:gw) [https://bugzilla.mozilla.org/show_bug.cgi?id=1818400 ''for a nice performance improvement''] to WebRender’s picture caching mechanism!</li></ul>
=== Privacy/Security ===
* Cookie Banner Handling can now be enabled in Nightly via about:preferences#privacy. This will enable the feature for both normal and private browsing.
For more granular control you can use the ''cookiebanners.service.mode'' and ''cookiebanners.service.mode.privateBrowsing'' prefs. Supported modes are:<br />
<ul>
<li> 0: Disabled </li>
<li>1: Reject all (this is what the checkbox in the preferences enables) </li>
<li>2. Reject all or fall back to accept all </li>
</ul>
<ul>
<li><p>In addition to our initial list of site-specific [https://github.com/mozilla/cookie-banner-rules-list ''cookie banner handling rules''] we are working on global rules that can handle a list of cookie banner libraries / providers on any site.</p>
<ul>
<li><blockquote><p>You can already test this in Nightly by enabling the pref ''cookiebanners.service.enableGlobalRules ''(beware of potential bugs and performance issues)</p></blockquote></li></ul>
</li></ul>
=== Search and Navigation ===
<ul>
<li>:daleharvey[https://bugzilla.mozilla.org/show_bug.cgi?id=1811931 '' Added support for seeing Google trending suggestions in the address bar''].</li>
<li>:jteow[https://bugzilla.mozilla.org/show_bug.cgi?id=1814773 '' Added an engagement event telemetry on SERP's''], as an ongoing effort to experiment with adding Glean to Search telemetry.</li>
<li>:dao continued work on the Awesomebar results menu, whereby he[https://bugzilla.mozilla.org/show_bug.cgi?id=1816496 '' fixed an issue''] where the urlbar results menu button would continue appear despite using one of the options (e.g. Learn More). Additionally disabling browser.urlbar.resultMenu.keyboardAccessible to prevent the results menu from being selectable using Tab[https://bugzilla.mozilla.org/show_bug.cgi?id=1818455 '' should now work with a mouse''].</li>
<li>:standard8[https://bugzilla.mozilla.org/show_bug.cgi?id=1816939 '' fixed an issue''] where the list of search engines in the preferences panel could be doubled.</li>
<li>:adw[https://bugzilla.mozilla.org/show_bug.cgi?id=1817038 '' worked on a patch to store the weather suggestions keywords in data provided to Nimbus''] rather than using quick suggestion config in remote settings.</li>
<li><p>:daisuke worked on a handful of patches to improve recording user engagement telemetry in the Urlbar. </p>
<ul>
<li><blockquote><p>[https://bugzilla.mozilla.org/show_bug.cgi?id=1817155 ''Bug 1817155''],[https://bugzilla.mozilla.org/show_bug.cgi?id=1817200 '' Bug 1817200''],[https://bugzilla.mozilla.org/show_bug.cgi?id=1817156 '' Bug 1817156'']: Various bugs pertaining to the situation where a user uses paste and go or drop and go feature in the urlbar</p></blockquote></li>
<li><blockquote><p>[https://bugzilla.mozilla.org/show_bug.cgi?id=1817202 ''Bug 1817202'']: Introduce site_specific_contextual_search as a possible result type. This is for contextual search (browser.urlbar.contextualSearch.enabled = true)</p></blockquote></li>
<li><blockquote><p>[https://bugzilla.mozilla.org/show_bug.cgi?id=1817203 ''Bug 1817203'']: Pressing the Go button.</p></blockquote></li></ul>
</li>
<li>Related to urlbar engagement, telemetry wasn't registered properly for weather suggestions, :mcheang and :daisuke worked on patches to address it:[https://bugzilla.mozilla.org/show_bug.cgi?id=1815699 '' Bug 1815699''] and[https://bugzilla.mozilla.org/show_bug.cgi?id=1817899 '' Bug 1817899'']</li>
<li>:mak[https://bugzilla.mozilla.org/show_bug.cgi?id=1817248 '' fixed an issue where the bookmarks could be out of order''] for those with a disabled history, due to unvisited bookmarks all receiving the same ranking, whereas now they should be bucketed by bookmark creation date, resulting in older bookmarks getting a lower ranking.[https://bugzilla.mozilla.org/show_bug.cgi?id=1818549 '' He also worked on updating the documentation''] on Frecency which can be viewed[https://firefox-source-docs.mozilla.org/browser/urlbar/ranking.html '' here''].<br />
</li></ul>
=== Storybook / Reusable components ===
* Thanks to :scoville for [https://bugzilla.mozilla.org/show_bug.cgi?id=1809715 ''cleaning up some unused code in the moz-support-link component'']
* Thanks to :mconley for [https://bugzilla.mozilla.org/show_bug.cgi?id=1819750 ''enabling the use of brand.ftl Fluent strings in our Storybook'']
* :tgiles [https://bugzilla.mozilla.org/show_bug.cgi?id=1813203 ''fixed an issue where we could use moz-support-link in about:plugins and a contrast issue in about:plugins'']
=== Community ===
<ul>
<li><p>[https://www.outreachy.org/apply/project-selection/#firefox ''We have gotten slots for 2 interns through the Outreachy program'']! Now is a good time to groom your good-first-bugs, as candidates are going to be looking to close out some starter bugs as part of their application process.</p>
<ul>
<li><blockquote><p>[standard8] Help would be appreciated in the #introductions and #outreachy Matrix channels!</p></blockquote></li></ul>
</li></ul>
</li></ul>



Latest revision as of 22:45, 7 March 2023

Today’s meeting leader is: kpatenio

Today’s meeting leader is: kpatenio

Friends of the Firefox team

Resolved bugs (excluding employees)

Script to find new contributors from bug list

Volunteers that fixed more than one bug

  • CanadaHonk [:canadahonk]
  • Itiel
  • Mathew Hodson
  • steven w

New contributors (🌟 = first patch)

General triage

Project Updates

Add-ons / Web Extensions

Addon Manager & about:addons

  • Starting from Firefox 112, users can now search for text inside the about:addons page (through the usual associated keyboard shortcuts, e.g. Ctrl-F and ‘/’) - Bug 1499500

WebExtensions Framework

  • Thanks to Itiel, the default extension icons in the extensions button panel are now filled with the current theme color - Bug 1817865
  • In Nightly 112 we migrated the kvstore path used by ExtensionPermissions, to be sure that the kvstore path isn’t shared with the one used internally by ExtensionScriptingStore - Bug 1807010

    • NOTE: the underlying race issue was investigated and confirmed as part of Bug 1805427, and so be aware that despites the inline comment associated to nsIKeyValueService.getOrCreate in nsIKeyValue.idl states that it can be used for multiple database names stored in the same kvstore path, that isn’t thread safe with the SafeMode (and old LMDB mode is deprecated).

WebExtension APIs

  • Starting from Firefox 112, downloads.removeFile API calls that fail will be logging more detailed info about the actual underlying issue in the browser console - Bug 1807815

Developer Tools

DevTools

  • External contributors:

    • khadija fixed a visual issue in the netmonitor (bug)

    • marlene replaced wrong parameter value in calls to CodeMirror method from the debugger (bug)

  • Contributions from other teams:

    • bradwerth improved performance of the grid highlighter when there are a lot of columns in the grid (bug)

    • Emilio fixed a scaling issue in the inspector tooltip (bug)

  • Hubert added search modifier in project search in the debugger (bug)

  • Hubert also fixed an issue with Navigator.sendBeacon requests that were shown as Blocked in the Netmonitor even if they went through (bug)
  • Julian recovered a regression that broke network throttling in Netmonitor (bug)
  • Nicolas removed duplicated ChromeWorker messages in Browser Console (bug)
  • Nicolas improved performance of pretty printing in debugger (bug, bug)
  • Alex made it possible to copy text from Netmonitor HTML preview again (bug)
  • Alex landed a patch that does some groundwork (bug) for a new feature in the debugger that will let the user override HTTP responses with a local file (bug), which is in progress.

WebDriver BiDi

  • Sasha added the browsingContext.print command, which produces a base64-encoded PDF representation of the document (bug)
  • Henrik fixed an issue where TabSession.executeInChild would hang the message manager (bug)
  • Henrik vendored the latest version of puppeteer (19.7.2) which allows to run more tests in CI and have a better coverage of CDP (bug)
  • Henrik also fixed an issue where httpd.js listener wouldn't stop, which could lead to unexpected side effects (bug)
  • Julian created a shared module to generate HAR files from WebDriver BiDi network events, which could be useful for libraries that are using BiDi like Browsertime and Selenium (bug)


ESMification status


Migration Improvements (CalState LA Project)


Performance

Privacy/Security

  • Cookie Banner Handling can now be enabled in Nightly via about:preferences#privacy. This will enable the feature for both normal and private browsing.


For more granular control you can use the cookiebanners.service.mode and cookiebanners.service.mode.privateBrowsing prefs. Supported modes are:

  • 0: Disabled
  • 1: Reject all (this is what the checkbox in the preferences enables)
  • 2. Reject all or fall back to accept all
  • In addition to our initial list of site-specific cookie banner handling rules we are working on global rules that can handle a list of cookie banner libraries / providers on any site.

    • You can already test this in Nightly by enabling the pref cookiebanners.service.enableGlobalRules (beware of potential bugs and performance issues)

Search and Navigation

Storybook / Reusable components

Community

Project Updates

Add-ons / Web Extensions

Addon Manager & about:addons

  • Starting from Firefox 112, users can now search for text inside the about:addons page (through the usual associated keyboard shortcuts, e.g. Ctrl-F and ‘/’) - Bug 1499500

WebExtensions Framework

  • Thanks to Itiel, the default extension icons in the extensions button panel are now filled with the current theme color - Bug 1817865
  • In Nightly 112 we migrated the kvstore path used by ExtensionPermissions, to be sure that the kvstore path isn’t shared with the one used internally by ExtensionScriptingStore - Bug 1807010

    • NOTE: the underlying race issue was investigated and confirmed as part of Bug 1805427, and so be aware that despites the inline comment associated to nsIKeyValueService.getOrCreate in nsIKeyValue.idl states that it can be used for multiple database names stored in the same kvstore path, that isn’t thread safe with the SafeMode (and old LMDB mode is deprecated).

WebExtension APIs

  • Starting from Firefox 112, downloads.removeFile API calls that fail will be logging more detailed info about the actual underlying issue in the browser console - Bug 1807815

Developer Tools

DevTools

  • External contributors:

    • khadija fixed a visual issue in the netmonitor (bug)

    • marlene replaced wrong parameter value in calls to CodeMirror method from the debugger (bug)

  • Contributions from other teams:

    • bradwerth improved performance of the grid highlighter when there are a lot of columns in the grid (bug)

    • Emilio fixed a scaling issue in the inspector tooltip (bug)

  • Hubert added search modifier in project search in the debugger (bug)

  • Hubert also fixed an issue with Navigator.sendBeacon requests that were shown as Blocked in the Netmonitor even if they went through (bug)
  • Julian recovered a regression that broke network throttling in Netmonitor (bug)
  • Nicolas removed duplicated ChromeWorker messages in Browser Console (bug)
  • Nicolas improved performance of pretty printing in debugger (bug, bug)
  • Alex made it possible to copy text from Netmonitor HTML preview again (bug)
  • Alex landed a patch that does some groundwork (bug) for a new feature in the debugger that will let the user override HTTP responses with a local file (bug), which is in progress.

WebDriver BiDi

  • Sasha added the browsingContext.print command, which produces a base64-encoded PDF representation of the document (bug)
  • Henrik fixed an issue where TabSession.executeInChild would hang the message manager (bug)
  • Henrik vendored the latest version of puppeteer (19.7.2) which allows to run more tests in CI and have a better coverage of CDP (bug)
  • Henrik also fixed an issue where httpd.js listener wouldn't stop, which could lead to unexpected side effects (bug)
  • Julian created a shared module to generate HAR files from WebDriver BiDi network events, which could be useful for libraries that are using BiDi like Browsertime and Selenium (bug)


ESMification status


Migration Improvements (CalState LA Project)


Performance

Performance Tools (aka Firefox Profiler)

Privacy/Security

  • Cookie Banner Handling can now be enabled in Nightly via about:preferences#privacy. This will enable the feature for both normal and private browsing.


For more granular control you can use the cookiebanners.service.mode and cookiebanners.service.mode.privateBrowsing prefs. Supported modes are:

  • 0: Disabled
  • 1: Reject all (this is what the checkbox in the preferences enables)
  • 2. Reject all or fall back to accept all
  • In addition to our initial list of site-specific cookie banner handling rules we are working on global rules that can handle a list of cookie banner libraries / providers on any site.

    • You can already test this in Nightly by enabling the pref cookiebanners.service.enableGlobalRules (beware of potential bugs and performance issues)

Search and Navigation

Storybook / Reusable components

Community