WebDriver/Meetings/2018-08-06

Agenda

ato: Problem with a rebase caused review status to be lost. Sorry about that.; [talks about issues with different review tools]; Lots of extra dependencies for those Phabricator extensions which are just a wrapper around arc/UI stuff.; As I understand it, Mark’s team is working on something built in to mach that will not require these dependencies but which instead will talk directly with Phabricator using its API.; Data loss for using Phabricator because it doesn't post back to Bugzilla. Could be potential problem if we move away from Phabricator in the future.
dburns: We should find a concensus for our team to make use of a single review system.
ato: Finding concensus can be hard when all the tools have problems. I think the best we can do is to suffer through a few weeks and then re-evaluate when support for our workflow is better with Phabricator.
whimboo: Splinter will also go away.
ato: Yes, but Splinter is essentially just comments on patchfiles. There’s no data loss because all comments are reflected in a single thread in Bugzilla.; Splinter is not great—mostly for UI reasons—but it’s simple and easy to understand.

dburns: Finishing up what we did in H1.; There's one OKR about using wpt.fyi data, helping other teams seeing where we have a low interop score.; Show Firefox vs. any browser and see how interoperable we are.
ato: This is so important, good effort.; But maybe difficult to get buy-in?
dburns: People want more web platform tests on mobile.
ato: People seem to think WPT tests and interop testing is important, but it's so far been hard to get engineering managers to insist on it.
dburns: Some of our harnesses have features that WPT doesn’t, and this is a problem (leak checks, ASAN); Long list of reasons why developers are using mochitest instead of wpt, and we need to cut that list.

[discussion about GeckoView]

ato: We have put in an intern application for next year for a “Gecko developer experience” project related to writing a W3C conforming WebDriver client and improving infrastructure.

(Spoken status updates in bold.)

ato
- Added debugging shorthand Sleep to sync module to pause current async task (1473614)
- Removed PREF_COMPLEX from prefs module (1480810)
- Dropped references to mozreview in documentation (1479772, 1479774)