Firefox/Meeting/08-Nov-2016

Actions from last meeting

• None

General Topics / Roundtable (tips/tricks/FYI, productivity improvements, industry news)

• [mconley] This Etherpad is starting to bug out due to its age / history. #moc has requested that we try using something else, like a public GDoc. Objections? Sounds great.
• [mconley] async / await! https://blog.nightly.mozilla.org/2016/11/01/async-await-support-in-firefox/
  • #jsapi has requested we wait a cycle before considering swapping out Task.* for async/await to avoid String.prototype.contains-like snafu
• [mconley] Network throttling tool: https://blog.nightly.mozilla.org/2016/11/07/simulate-slow-connections-with-the-network-throttling-tool/
  • Also, for OS X, there's Network Link Conditioner: http://nshipster.com/network-link-conditioner/
  • Not everybody has epic silicon valley fiber. If you're writing stuff that hits the network, good idea to test with the above.
• Next weeks meeting master is: kitcambridge!

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/2fVJilZ
  • More than one bug fixed:
    • André Bargull
    • Kaffeekaethe
    • Sourav Garg [:jordan]
    • Stefan [:stefanh]
    • Tomislav Jovanovic :zombie
  • New Contributors
    • Kaffeekaethe
    • Sourav Garg [:jordan]

Project Updates

Add-ons

• Landed:
  • Container API for WebExtensions (behind a flag)
    • https://bugzilla.mozilla.org/show_bug.cgi?id=1302697
  • Data for the SDK fixes made in the last week shows that they are performing well
    • https://metrics.mozilla.com/protected/dzeber/reports/addon-sdk-fix-analysis.html

E10s

• More patches have landed which should, in theory, help with tab spinners. Specifically:
  • Bug 1313125 - Get rid of synchronous message from AppsServiceChild.jsm in DOMApplicationRegistry.init ( https://bugzilla.mozilla.org/show_bug.cgi?id=1313125 )
  • Bug 1312880 - Use initialProcessData for RemoteAddonsChild rather than a sync message ( https://bugzilla.mozilla.org/show_bug.cgi?id=1312880 )
  • Bug 1312540 - PContent::Msg_GetGMPPluginVersionForAPI sends a sync IPC message to the parent and causes main thread IO ( https://bugzilla.mozilla.org/show_bug.cgi?id=1312540 )
  • Bug 1314772 - Add test for opt-out TABCHILD_PAINT_TIME probe ( https://bugzilla.mozilla.org/show_bug.cgi?id=1314772 )
• Firefox 51 Beta will run an experiment that will enable e10s for users that have any add-on that has not explicitly opted out of e10s.
  • As usual, will be measuring responsiveness, crash rates, to determine if we're in a state where we could possibly ship this to release
  • If and when we start rolling out in this state, we're going to do it nice and slow
  • Details at https://wiki.mozilla.org/Firefox/AddOns/Status/current#Add-ons.2Fe10s_Program_Status_Report

Firefox Core Engineering [</nowiki>https://wiki.mozilla.org/Firefox_Core_Engineering]

• We are going to begin prompting users of FF45 to upgrade (as part of orphan remediation). Two weeks after 50 is released, we will do the same with FF46. Currently, our orphans (> 3 versions back) account for 4.9% of our audience.
• Looking into including frame pointers on release. Initial stages.
• (everything else is old news, covered in previous meetings)
• Telemetry crash rates: https://chutten.github.io/telemetry_crashes/

Form Autofill

• Landed
  • Bug 1016733 - Implement form auto-fill profile storage ( https://bugzilla.mozilla.org/show_bug.cgi?id=1016733 )
• In Progress
  • Bug 1300990 - Implement an API in the parent process to decide which values of an autofill profile would be filled in which field ( https://bugzilla.mozilla.org/show_bug.cgi?id=1300990 )
  • Bug 1304634 - Support populating autocomplete results from form autofill code ( https://bugzilla.mozilla.org/show_bug.cgi?id=1304634 )

Platform UI and other Platform Audibles

• lsalzman has been making some perf improvements to box-shadow ( https://bugzilla.mozilla.org/show_bug.cgi?id=1250037 )
• <select> improvements by MSU students
  • [beachjar] Vertical popup centering for macosx-e10s autolanded! ( https://bugzilla.mozilla.org/show_bug.cgi?id=430745 )
  • [Mark] More styling improvements in the pipe: https://bugzilla.mozilla.org/show_bug.cgi?id=1309938
  • [Fred_, miguel] Combining e10s and non-e10s <select> mechanisms in final rounds of review: https://bugzilla.mozilla.org/show_bug.cgi?id=1300784
  • [tmaklebust, beachjar] Ability to filter lists > 40 elements long being reviewed: https://bugzilla.mozilla.org/show_bug.cgi?id=1309935
  • [beachjar] Now looking at converging with Edge / Chrome behaviour on <option> styling with e10s popup: https://bugzilla.mozilla.org/show_bug.cgi?id=910022
• In progress:
  • [mconley] Bug 418833 - can't define the style of input when the type is set to "checkbox" or "radio" with CSS - landed but bounced. Hopefully will re-land this week ( https://bugzilla.mozilla.org/show_bug.cgi?id=418833 )
  • [daleharvey] Bug 1307689 - Implement Abandonment Rate telemetry ( https://bugzilla.mozilla.org/show_bug.cgi?id=1307689 )
  • [chunmin] Bug 1289974 - [Presentation WebAPI] provide a trusted UI for device selection on Firefox ( https://bugzilla.mozilla.org/show_bug.cgi?id=1289974 )
  • [scottwu] Bug 1301284 - Update picker style to match the visual spec for <input type="time"> ( https://bugzilla.mozilla.org/show_bug.cgi?id=1301284 )
  • [mconley] Bug 1309316 - Expose preference for user to automatically submit backlogged crash reports in about:tabcrashed ( https://bugzilla.mozilla.org/show_bug.cgi?id=1309316 )
• Landed:
  • [mconley] Bug 1313067 - Text is more truncated with ellipsis in datalist for input ( https://bugzilla.mozilla.org/show_bug.cgi?id=1313067 )
  • [mconley] Bug 1303067 - Enable unsubmitted crash notification for more users ( https://bugzilla.mozilla.org/show_bug.cgi?id=1303067 )
    • Only up to early beta
  • [jessica] Bug 1314542 - [DateTimeInput] input field cleared unexpectedly when using up/down keys ( https://bugzilla.mozilla.org/show_bug.cgi?id=1314542 )
• Quantum Compositor process has been enabled on Nightly for Windows ( https://bugzilla.mozilla.org/show_bug.cgi?id=1314133 )
  • Basic state and controls added to about:support in https://bugzilla.mozilla.org/show_bug.cgi?id=1297792
  • Meta bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1307578
• Rust-based URL parser will be landing soon: https://bugzilla.mozilla.org/show_bug.cgi?id=1151899

Privacy/Security

• The screen sharing whitelist has been removed in 52. Users can now authorize screen sharing for any website from a revamped permission prompt that displays a preview of the shared contents and warns loudly in the more risky cases (1284877, 1284878, 1315737).
• The notification prompt redesign should land in elm some time later this week. Add-on prompts will be fixed in elm before merging to m-c, plugin prompts will be followup work.
• Insecure login field contextual warning
  • Landed:
    • [seanlee] Bug 1217162 - Implement Contextual Feedback on Insecure Passwords
      • Disabled behind security.insecure_field_warning.contextual.enabled for now
  • In Progress:
    • [seanlee] Bug 1289913 - Show autocomplete UI on password fields (https://bugzilla.mozilla.org/show_bug.cgi?id=1289913 )
    • [jkt] Bug 376668 - Show password manager autocomplete UI on first focus instead of on second focus (i.e. when the user types)
    • [timdream] Bug 1314478 - Make Contextual Warnings show up for all username fields
    • [daleharvey] Bug 1120037 and bug 1311301 - autocomplete bugs with autofocus

Quality of Experience

New theming API

• continuing to experiment/explore with some prototypes, demos for MozHawaii
• waiting on a product requirements doc to get plans more concrete by MozHawaii
• patch in progress to allow for changing toolbar icons. authors will just need to set one line in their manifest for each icon, specifying the path to the new icon to use
  • will probably require SVG icons to allow for scaling in toolbars and menupanel, as well as lodpi and hidpi
• Demo: <tuck>
  • Whoa! This is exciting to see. :-)
  • Please tell us what you think, or even more crazy ideas: https://bugzilla.mozilla.org/show_bug.cgi?id=1313325
• On Cedar now.

Onboarding

• Enabling automigration again for beta 51

Reader Mode [evanxd]

• (patch under review) Bug 1173823 - Reader View ignores <base href="...">
• (figuring out the performance issue) Bug 1315490 - 2.33 - 3.82% tps (linux64, windows8-64) regression on push ef3d294a1cf6 (Wed Nov 2 2016)
• (preparing patch for review) Bug 1310075 - Tests for qq.com

Search

• A big number of small patches that optimize our SQLite layer has landed (764573, 835796, 1310152, 1313021, 1313309, 1313310).
• Fixed a number of corner-case bugs around search keywords, one-off searches and result selection with Ctrl-Click (1314643, 1227734, 1310737, 1312018, 1299428, 1296366, 1313969).
• SHIELD study for unified search is in QA's court right now, hoping to ship it soon.
• More ESLint goodness: no-unused-vars is now enabled for browser/ (local scope only).

Sync/FxA

• Sync error dashboards now available in re:dash! https://sql.telemetry.mozilla.org/dashboard/sync-error-dashboard
• Bookmark tracking improvements are coming in 53: https://bugzilla.mozilla.org/show_bug.cgi?id=1258127
• Send tab to device is riding trains now (previously was nightly only) https://bugzilla.mozilla.org/show_bug.cgi?id=1304534

Storage Management

• The V1 UX specs is locked down
• The overall meta bug: Bug 1147820 - [meta] Improve Storage

Front-end Implementations [Fischer]

• (WIP) Bug 1309123 - Show persist-storage permission request notification
• (patch under review) Bug 1313003 - Add Site Data section into Network of Advanced of about:preferences

Content Handling Enhancement

Download Panel [rexlee]

• (patch under review) 1301384 - Unify the spacing and the progress bar styling in the Downloads Panel across platforms
• (checking reviewer's feedback) 1269954 - Pop out download starting notification when a new download request is triggered

Download Dialog Removal [alphan]

• Keep confirming project scope with UX team, PM Peter Dolanjski, and Paolo

RDF Replacement [alphan/eden]

• (patch under review) 1287664 - Revise helperApp.js and(/or) enhance nsIHandlerService API based on the result of bug 1287661
• (preparing patch for review) 1287660 - New implementation of nsIHandlerService for the JSON backend

Project Mortar (PDF)

• (reviewing) bug 1313295, moving codebase from a private repository to m-c
• [Evelyn] study/design event dispatching flow in nested frame architecture
• [Luke] PDF UI integration - jump pages, page view fitting

This week I learned

• [mconley] initialProcessData
• [macOS] mach clobber && caffeinate mach build
  • caffeinate prevents Mac from sleeping until the command finishes. Handy if you're building while on battery.
  • Also handy for presentations! -disumÂ
• (Just finished converting this vimeo thing, so putting it here): https://vimeo.com/190736648
  • For https://bugzilla.mozilla.org/show_bug.cgi?id=941229
  • Question: too whimsy?Â
    • Definitely not l10n friendly
    • (I didn't even notice it until I knew what to look for)
    • More like add-on material then. OK!