Firefox/Meeting/28-Mar-2017

From MozillaWiki
Jump to: navigation, search

Actions from last meeting

  • Select a new meeting leader for April 11th meeting. Today’s meeting leader is: felipe
    • Next meeting leader is: RyanVM!

General Topics / Roundtable

(tips/tricks/FYI, productivity improvements, industry news)

  • [felipe] faststripe to be removed (bug 1351258)
  • [felipe] Using native async/await instead of Task.jsm
    • Task.async => async
    • yield => await
  • [mconley] Should we be adding a section on the re-theme effort?
  • [mconley] In case you didn’t notice, MozReview allows you to comment on commit messages now. 🎉 There was also a big discussion on commit messages on dev-platform, and a number of our high IC engineers expressed a desire for more clear commit messages before requesting review.
  • [florian] bug 1316187: the browser_all_files_referenced.js test landed.
    • It will fail whenever you remove the last reference to a chrome file and forget to remove it, or when pre-landing chrome files that aren't used anywhere yet.
    • It contains a long list of exceptions with bugs on file to cleanup existing unused files. This should be an easy source of good first bugs if you are looking for mentoring opportunities.
  • [florian] getComplexValue/setComplexValue should no longer be used with the nsISupportsString type for preference values with unicode characters. Instead just use the new getStringPref/setStringPref methods (bug 1345294)
    • not to be confused with getCharPref/setCharPref
    • eslint will report code using the old pattern
  • [MattN] "skin" vs. "content" stylesheets after Complete Theme removal (Fx57+)
    • Sounds like removing the concept of “skin” packages will be best

Friends of the Firefox team

(Give a shoutout/thanks to people for helping fix and test bugs. Introductions)

Project Updates

Add-ons

Activity Stream

Electrolysis (e10s)

Firefox Core Engineering

Form Autofill

Go Faster

Photon

  • (for next time)

Platform UI and other Platform Audibles

Privacy/Security

Project Mortar (PDFium)

  • [evelyn] Front-end work is almost done! A few bugs remaining, but it’s pretty polish-y.
  • [evelyn] Dealing with process separation work, and waiting on this bug to land which will allow us to create a special type of JS-implemented plugin
  • [evelyn] Tackling the printing engine as well
  • Blocked on spinning up QA help for manual testing, but we will first add more automation test and compare the result of pdf.js to understand how much improvement we gain. (Thanks to Benjamin’s suggestion!)
  • Talking to release team on release to-dos

Quality of Experience

Search

  • Phase 1 of the hi-res favicons work should land before the next meeting.
  • The last big issue with one-off search buttons in the awesomebar is very close to landing.
  • Various miscellaneous fixes for the search and location bars.

Sync / Firefox Accounts

  • Fixes:
    • Sync will discard folder child order if the local timestamp is newer than the remote. This shows up most frequently on first syncs.
    • First sync for passwords was broken in Aurora and Nightly.
  • Push-driven sign-in confirmation is coming! Design doc in progress; should have more updates in the next meeting.
  • If you’re curious…

Storage Management

  • [fischer] The project target due date is 4/17.
  • [fischer] The implementations are almost done. The remained 3 bugs are expected to be resolved before the target 4/17.
  • [fischer] Bug 1312349: Hide the section of Offline Web Content and User Data in about:preferences
    • Because the Storage management handles appcache as well, after the Storage management completes, the Offline(Appcache) group will be hidden.
    • The pref to control hide the Offline group is browser.preferences.offlinegroup.enabled

Test Pilot

This week I learned