Mconley: Add notes

2017-08-02T15:00:05Z

Add notes

New page

= General Topics / Roundtable =

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

* [mconley] For-looping in '''hot, performance critical '''code paths? [https://bugzilla.mozilla.org/show_bug.cgi?id=1383362 Don’t use ][https://bugzilla.mozilla.org/show_bug.cgi?id=1383362 forEach]. Even [https://bugzilla.mozilla.org/show_bug.cgi?id=1379900#c11 for-of][https://bugzilla.mozilla.org/show_bug.cgi?id=1379900#c11 is slower than ][https://bugzilla.mozilla.org/show_bug.cgi?id=1379900#c11 for-in][https://bugzilla.mozilla.org/show_bug.cgi?id=1379900#c11 or a C-style ][https://bugzilla.mozilla.org/show_bug.cgi?id=1379900#c11 for][https://bugzilla.mozilla.org/show_bug.cgi?id=1379900#c11 loop].
** Don’t use for-in to iterate over arrays, though.
* [bgrins] `./mach run` and prefs for the scratch_user
** Default browser prompt and about:config warning are hidden by default
** Set arbitrary prefs, like: ./mach run --setpref browser.startup.homepage="about:preferences"
** [https://groups.google.com/d/msg/firefox-dev/kPwA1y-7BpI/gX6rvhEjBQAJ https://groups.google.com/d/msg/firefox-dev/kPwA1y-7BpI/gX6rvhEjBQAJ]
* [standard8] [https://mail.mozilla.org/pipermail/firefox-dev/2017-July/005630.html MozillaBuild 3.0 is out]! (thanks to RyanVM). Includes built-in node support for ESLint
* [mconley] [https://mail.mozilla.org/pipermail/firefox-dev/2017-July/005662.html Stylo is being built by default now], so if you’re wondering why your non-artifact builds have slowed way down, that’s because ccache doesn’t support Rust. Consider switching to [https://groups.google.com/forum/#!msg/mozilla.dev.platform/5srcnP-Wj4E/Uf_2gw2NAAAJ sccache] (if you’re on OS X, you may have to workaround this [https://github.com/mozilla/sccache/issues/163 bug] by running xcode-select --install.If that doesn’t work, consider asking the folks in #developers). One bonus is that sccache, unlike ccache, works on Windows!
** You can disable building Stylo locally with: ac_add_options --disable-stylo in your mozconfig

= Friends of the Firefox team =
(Give a shoutout/thanks to people for helping fix and test bugs. Introductions)
* Resolved bugs (excluding employees): [https://mzl.la/2vpwyMc https://mzl.la/2vpwyMc]
** More than one bug fixed:
*** Alejandro Rodriguez Salamanca
*** Bao Quan [:beekill]
*** Luciano I
*** Tomislav Jovanovic :zombie
*** Tushar Saini (:shatur)
*** flyingrub
** New contributors (🌟 = First Patch!)
*** 🌟Luciano I [https://bugzilla.mozilla.org/show_bug.cgi?id=1380738 tweaked the size] and [https://bugzilla.mozilla.org/show_bug.cgi?id=1382290 the style] of the Firefox Accounts avatar in about:preferences!
*** 🌟Nicolas Ouellet-Payeur made it so that we [https://bugzilla.mozilla.org/show_bug.cgi?id=1375212 always wrap thrown strings with Error objects] in our Sync code!
* jonathanGB’s internship presentation @ 1:15pm PDT today
** Form autofill & WebPayments

= Project Updates =

== Add-ons ==
* Warnings are now [https://bugzilla.mozilla.org/show_bug.cgi?id=1330732 displayed from startup of an extension installed via about:debugging].
* The [https://bugzilla.mozilla.org/show_bug.cgi?id=1341304 devtools.panels.elements.onSelectionChanged event] is now supported.
* We’ve started landing parts of the [https://developer.mozilla.org/en-US/Add-ons/WebExtensions/API/browsingData browsingData API] on Android including [https://bugzilla.mozilla.org/show_bug.cgi?id=1362994 settings] and [https://bugzilla.mozilla.org/show_bug.cgi?id=1362998 removeCookies].
* Improved WebExtensions performance by [https://bugzilla.mozilla.org/show_bug.cgi?id=1381687 storing and transferring schema JSON in structured clone blobs].
* Added a new API to allow extensions to [https://bugzilla.mozilla.org/show_bug.cgi?id=1333376 add a preface to the window title].
* Extensions can now [https://bugzilla.mozilla.org/show_bug.cgi?id=1268020 add an item to the “Tools” menu] in a manner similar to how extensions add items to the context menu.
* Tab objects returned from an API call now [https://bugzilla.mozilla.org/show_bug.cgi?id=1348911 include a lastAccessed property].

== Activity Stream ==
* Activity Stream will be pref’ed off by default in 56 Beta, and will be successively rolled out from 1%, 5% to 10% of the Beta population using Shield pref flips
* Default Top Sites landing for Tier 1 and Tier 2 countries with global fallback
* Pocket will be enabled by geo/locale for Canada and US initially, Germany will be added during 57 Nightly and tested in 56 Beta
* Snippets, Profile Migration, Onboarding Tour recently landed in 56

== Electrolysis (e10s) ==
* Content process preallocator has been disabled while the team coordinates with the Activity Stream team on the preloaded about:newtab browser.

== [https://wiki.mozilla.org/Firefox_Core_Engineering Firefox Core Engineering] ==
* App Updater
** App Updater is now using LZMA compression and SHA384 for signing. The release of builds taking advantage of this should be unthrottled today. So far, complete update files are reduced from 19-21%.
** Reminder: Updates to 64-bit begin in FF56, so you can look for that on Nightly now.
** We have pulled the Update Agent (which continues the download of the update file in the background) from 57. Earliest it would be now is 58.
* Installer
** To support onboarding, we’re [https://bugzilla.mozilla.org/show_bug.cgi?id=1369255 adding an option for users to clean up their profile] if they’ve had an old version of Firefox installed before. This is aimed for 57.
* Crashes/Stability
** Carl Corcoran (ccorcoran) and Gabriele Svelto (gsvelto) are close to landing [https://bugzilla.mozilla.org/show_bug.cgi?id=1333126 CFI-based stackwalking support].
** We’re going to [https://bugzilla.mozilla.org/show_bug.cgi?id=1384106 block 3rd party applications that use our accessibility APIs].
** ...and addressing an issue with early injection (confidential bug due to security).
* Hang Reporting
** Doug Thayer (dthayer) is working with mystor and smaug on how to make the BHR dashboard more useful. This is part of the “BHR Taskforce” with DOM.
* Quantum Flow/Photon Perf
** Just in case anyone isn’t aware, 3-5 people on our team are working on QF and/or Photon Perf bugs as well, trying to pinch hit on Runtime or DOM bugs.
** Namely,those people are Adam Gashlin (agashlin), Kirk Steuber (bytesized), Perry Jiang (perry), and Felipe Gomes (felipe) -- and sometimes Robert Strong (rstrong). Please help them if they reach out with questions.

== [https://wiki.mozilla.org/Firefox/Features/Form_Autofill Form Autofill] ==
* Milestone 3 done
* Fixed
** High level: credit card, sync, l10n, pref refinements, US split telephone fields, performance
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1374960 [Form Autofill] Manage Profiles and Edit Profile dialogs refinements]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1370429 [Form Autofill] Implement the infrastructure of Telephone parser to support the various combination of telephone fields]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1372077 [Form Autofill] Change the label "country" to "country or region" in Preferences]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1375799 [Form Autofill] Move the strings in manageProfiles and editProfile dialog to .properties file]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1377636 Intermittent browser/extensions/formautofill/test/browser/browser_update_doorhanger.js | Uncaught exception - undefined - timed out after 50 tries.]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1378072 [Form Autofill] Update the locale of phishing warning note to align with the latest UX spec]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1363999 Add "sync tracker" support to ProfileStorage]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1379533 [Form Autofill] Support two fieldDetails array for Address and Credit Card feature in FormAutofillHandler.]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=990200 [Form Autofill] Collect information about the count of autofill being applied in different scenario]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1380279 [Form Autofill] Add the ability for "startSearch" to retrieve credit card profiles from storage]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1363995 Implement core reconciliation logic for formautofill engine]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1382189 [Form Autofill] Setting manuallyManagedState bit might trigger frame re-construction and erase the preview text while previewing]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1383065 [Form Autofill] Trigger autofill broken on 3 fields forms]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1371131 [Form Autofill] Extend ProfileAutoCompleteResult to support credit cards]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1378754 Autofill doesn't work when using drag to new window]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1376642 [Form Autofill] Replace the strings "profile" with "address" in preferences]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1362290 Support non-experimental autocomplete pref]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1382529 [Form Autofill] Exception occurs when saving a telephone number like "1234567"]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1377360 [Form Autofill] Refine the warning message in preferences]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1379588 [Form Autofill] Implement credit card profile creation in FormAutofillHandler]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1385753 [Form Autofill] Group tel-related fields when calculating the secondary label]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1382148 Cache the matching result of select elements so we don't need to calculate it every time when previewing and filling]
* Next
** Finishing Credit card support
** Polishing system add-on for the MVP

== Lint ==
* Landed recently:
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1374671 No-whitespace-before-property] (whole tree)
* Upcoming:
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1374620 Dot-notation]
** Enabling more rules in more directories:
*** accessible/
*** mobile/android/
*** toolkit/components/osfile

== Photon ==

[dolske] We’re very close (and on track!) to our feature-complete target date of August 7th. Still a few key things incoming, but starting to shift towards bugfixing and polish instead of adding whole new features.

=== Performance ===
* [https://bugzilla.mozilla.org/show_bug.cgi?id=1356705 A patch to switch the tab strip to using CSS smooth-scroll] (and eliminating some sync reflows in the process), has been reviewed and is waiting for the merge before landing on central.
* There are now some [https://bugzilla.mozilla.org/show_bug.cgi?id=1356271 basic reflow tests] for showing results in the AwesomeBar
* [https://bugzilla.mozilla.org/show_bug.cgi?id=1355426 A patch to switch to the next tab earlier] when removing a tab is waiting for review, and should shave a few milliseconds off of tab switch time.
* mconley is starting to work on [https://bugzilla.mozilla.org/show_bug.cgi?id=1385453 a patch to “warm up” tabs speculatively] when we think we’re going to switch to them.
* Perry is working on removing [https://bugzilla.mozilla.org/show_bug.cgi?id=1360406 the context menu CPOWs] and [https://bugzilla.mozilla.org/show_bug.cgi?id=1353584 making the thumbnailer less aggressive]

=== Structure ===
* Drew worked hard on refactoring the page action panel, and you can now put items that are in [https://bugzilla.mozilla.org/show_bug.cgi?id=1374477 the page action menu in the URL bar permanently!]
* We’ll be adding support for [https://bugzilla.mozilla.org/show_bug.cgi?id=1367927 pocket] and [https://bugzilla.mozilla.org/show_bug.cgi?id=1366041 screenshots] there soon.
* Mike [https://bugzilla.mozilla.org/show_bug.cgi?id=1354533 updated the history portion of the library button] to declutter it and show more history items.
* The overflow panel is now [https://bugzilla.mozilla.org/show_bug.cgi?id=1354086 much prettier]!

=== Animation ===
* The [https://bugzilla.mozilla.org/show_bug.cgi?id=1355922 Save to Pocket] and the [https://bugzilla.mozilla.org/show_bug.cgi?id=1352063 Bookmark] animation landed. Reminder that you do not need to sign up for an account to use Pocket, just click through the tutorial and it will start letting you save pages to Pocket.
* Further refinements on the toolbar button animations [https://bugzilla.mozilla.org/show_bug.cgi?id=1381957 [1]] [https://bugzilla.mozilla.org/show_bug.cgi?id=1380815 [2]]
* New[https://bugzilla.mozilla.org/show_bug.cgi?id=1352075 ][https://bugzilla.mozilla.org/show_bug.cgi?id=1352075 arrow-panel animations] are up for review, looks like all test failures are now fixed.
* The new[https://bugzilla.mozilla.org/show_bug.cgi?id=1352065 ][https://bugzilla.mozilla.org/show_bug.cgi?id=1352065 Download notifications animation] got r+ and should land after the merge
* The new tab[https://bugzilla.mozilla.org/show_bug.cgi?id=1352119 ][https://bugzilla.mozilla.org/show_bug.cgi?id=1352119 loading/progress indicator] is going through review passes

=== Visuals ===
* [https://bugzilla.mozilla.org/show_bug.cgi?id=1377284 Titlebar vibrancy on Mac OS] has just landed!!
* Dao is putting [https://bugzilla.mozilla.org/show_bug.cgi?id=1349555 square ta][https://bugzilla.mozilla.org/show_bug.cgi?id=1349555 bs] in your Nightly, soon…. ☜[ヮﾟ☜]
* Johann [https://bugzilla.mozilla.org/show_bug.cgi?id=1371219 increased the size of all Photon-style menus when accessed through touch].
* Dale updated us to delicious [https://bugzilla.mozilla.org/show_bug.cgi?id=1370929 new dark theme colors].
* Dao [https://bugzilla.mozilla.org/show_bug.cgi?id=1372689 updated the tab strip icons] and threw away a lot of PNGs.
* And a lot of small fixes and tweaks, such as [https://bugzilla.mozilla.org/show_bug.cgi?id=1379266 fine-tuning the Windows accent color text], [https://bugzilla.mozilla.org/show_bug.cgi?id=1377011 sharper sidebar tree icons], [https://bugzilla.mozilla.org/show_bug.cgi?id=1379123 correctly sorting the UI density options], [https://bugzilla.mozilla.org/show_bug.cgi?id=1376413 coloring the titlebar in Mac OS customize mode], [https://bugzilla.mozilla.org/show_bug.cgi?id=1381117 fixing the identity block hover state] and much more.

=== Onboarding ===
* 56: Going through P3 bugs and working on them. Move other things to P5s.
** Accessibility bugs wasn’t an original requirement but we have decided to make them high priority P3s. Will uplift to beta as they being completed.
* 57: Copy has confirmed, waiting for illustration. WIP patches ready to be have illustration put into them.

=== Preferences ===
* 55: Performance section on Preferences was found to have confusing behavior because of how e10s rollout changes user default. Bug 1382649 landed and uplift to beta last week to deal with it. Will ship with minor known issues.
* 56: Search and Preferences Re-org received signed-off w/ YELLOWs. Work started to address them started last week. Most of them are addressed before Aug 2.
* 57: Patches are being worked on right now. Should have everything landed or in review by Aug 7.

== Platform Audibles ==
* Kris Maglione has landed patches that allow you to queue work to occur just after layout and style flushes have occurred! [https://bugzilla.mozilla.org/show_bug.cgi?id=1383367 Consider using them to avoid sync flushes.]

== Privacy/Security ==
* Holding off a couple of days to land [https://bugzilla.mozilla.org/show_bug.cgi?id=1275599 site permission management in preferences] in a [https://bugzilla.mozilla.org/show_bug.cgi?id=1373206 new dialog] in 57, with a few follow-up bugs already filed.
* Working on a new onboarding experiment around better Tracking Protection UI.
* jkt is working on [https://bugzilla.mozilla.org/show_bug.cgi?id=1354601 patches to m-c] that enable Containers to run as a WebExtension

== [https://wiki.mozilla.org/Firefox/Search Search and Navigation] ==
* Location Bar:
** The popup doesn’t show a scrollbar anymore, instead it just [https://bugzilla.mozilla.org/show_bug.cgi?id=1264988 adapts itself to the browser.urlbar.maxRichResults pref] value
** Pref-based Shield studies will now be able to define a [https://bugzilla.mozilla.org/show_bug.cgi?id=1378035 different results mixture based on the search context]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1383749 Fixed a regression in the way we cache one-off buttons] in urlbar and search bar
** Evelyn added [https://bugzilla.mozilla.org/show_bug.cgi?id=1355443 speculative connection to the Current Search Engine]
** We have a patch to [https://bugzilla.mozilla.org/show_bug.cgi?id=1371611 speed up UnifiedComplete startup time] by lazy loading all the urlbar prefs (pending review)
* Jank-free Bookmarking:
** A QE session has been booked to test Async Places Transactions, currently only enabled in Nightly, planned for 57 \o/ (ง°ل͜°)ง
** Fixed a [https://bugzilla.mozilla.org/show_bug.cgi?id=1383138 regression in the Bookmark All Tabs dialog]
** Currently investigating API optimizations when acting on hundreds of bookmarks at once
* Downloads
** Downloads history integration code is [https://bugzilla.mozilla.org/show_bug.cgi?id=1381411 being] [https://bugzilla.mozilla.org/show_bug.cgi?id=830415 refactored] to share it more easily with Photon downloads views
** The [https://bugzilla.mozilla.org/show_bug.cgi?id=1354568 file size is shown again] for completed downloads in the downloads panel (uplifted to 55)
* Misc:
** The Places connection shutdown now [https://bugzilla.mozilla.org/show_bug.cgi?id=1275878 uses an Async Shutdown Blocker instead of a custom notification]. This allows to simplify the History expiration code, that should soon move to idleDispatch.
** [https://bugzilla.mozilla.org/show_bug.cgi?id=1382444 Fixed] [https://bugzilla.mozilla.org/show_bug.cgi?id=1382444 some] [https://bugzilla.mozilla.org/show_bug.cgi?id=1336806 frequent] [https://bugzilla.mozilla.org/show_bug.cgi?id=1382291 intermittent] [https://bugzilla.mozilla.org/show_bug.cgi?id=1382474 failures], [https://bugzilla.mozilla.org/show_bug.cgi?id=1376676 leaks], [https://bugzilla.mozilla.org/show_bug.cgi?id=1383639 tests missing from the manifests] and [https://bugzilla.mozilla.org/show_bug.cgi?id=1347427 useless tests]

== Sync / Firefox Accounts ==
* [https://bugzilla.mozilla.org/show_bug.cgi?id=1385179 Autofill address sync] is coming in 56. Landed, but pref’d off for now.
* Bidirectional bookmark sync is coming to iOS! Please follow the [https://bugzilla.mozilla.org/show_bug.cgi?id=1380056 meta bug] if you’re interested.
* The history engine now [https://bugzilla.mozilla.org/show_bug.cgi?id=1336518 uses Sqlite.jsm to query Places], and [https://bugzilla.mozilla.org/show_bug.cgi?id=1377944 PlacesUtils.history.insertMany], instead of hand-rolled wrappers that spin the event loop. Thanks to contributors Santiago Paez and Luciano Italiani!
* Bookmark validation is now [https://bugzilla.mozilla.org/show_bug.cgi?id=1385127 enabled through early betas]. Please report any perf regressions you see when syncing bookmarks.
* Edouard made[https://bugzilla.mozilla.org/show_bug.cgi?id=1369539 Sync UI startup lazier] and snappier.

== [https://wiki.mozilla.org/Test_Pilot Test Pilot] (text only, have a meeting conflict) ==
* Three new experiments launched '''today. '''Check out [https://testpilot.firefox.com/ https://testpilot.firefox.com/]
* Firefox Screenshots continues to improve in Beta. We will launch in 56 with a small percentage of people and roll out the feature over a few weeks.
* We’ve started blogging at [https://medium.com/firefox-test-pilot https://medium.com/firefox-test-pilot] . Tune in for experiment updates on the new experiments

== Web Payments ==
* [[https://bugzilla.mozilla.org/show_bug.cgi?id=1381179 Bug 1381179] - Resolved] Added “Toolkit :: WebPayments UI” component on Bugzilla
* [[https://bugzilla.mozilla.org/show_bug.cgi?id=1381186 Bug 1381186]] Handles showPayment & abortPayment from the merchant
* [[https://bugzilla.mozilla.org/show_bug.cgi?id=1382388 Bug 1382388]] Handles abort event from the user (other way around)
* [[https://bugzilla.mozilla.org/show_bug.cgi?id=1383300 Bug 1383300]] Started getting the “total” and the “origin” from the API on showPayment and display in the dialog
* Next: integrate autofillStorage to pre-populate “Shipping address” selector

Firefox/Meeting/01-Aug-2017 - Revision history

Mconley: Add notes