canmove, Confirmed users, Bureaucrats and Sysops emeriti
4,714
edits
WebDriver/RemoteProtocol/Meetings/2020/02/14: Difference between revisions
WebDriver/RemoteProtocol/Meetings/2020/02/14 (view source)
Revision as of 20:09, 26 February 2021
, 26 February 2021Whimboo moved page Remote/Meetings/2020/02/14 to WebDriver/RemoteProtocol/Meetings/2020/02/14: Reorganization for WebDriver BiDi project documenation
m (-) |
m (Whimboo moved page Remote/Meetings/2020/02/14 to WebDriver/RemoteProtocol/Meetings/2020/02/14: Reorganization for WebDriver BiDi project documenation) |
||
| (8 intermediate revisions by 3 users not shown) | |||
| Line 1: | Line 1: | ||
== Agenda == | == Agenda == | ||
* | * Scope needs to be reduced due to a slow down in the team's completion rate (mmucci) | ||
* New meeting time? | * New meeting time - possible days/times: | ||
** Monday at 10 AM Eastern (''4 PM Henrik''). | |||
** Wednesday at 9 AM or 10 AM Eastern (''3 PM or 4 PM Henrik''). | |||
** Other Options? | |||
* Confirm puppeteer-firefox deprecation warning (:maja_zf) | |||
* Move discussion from DevTools slack to chat.mozilla.org ([[Matrix]])? | |||
* Reviews while Henrik is out next week? | |||
* puppeteer triage for firefox issues (:digitarald) | |||
== Roster == | == Roster == | ||
| Line 7: | Line 14: | ||
; Present | ; Present | ||
whimboo, maja_zf, digitarald, mmucci, jgraham, Joel Einbinder (Microsoft) | |||
; | == Minutes == | ||
=== Scope === | |||
Scope needs to be reduced due to a slow down in the team's completion rate. | |||
* Reduce by 25 bugs. | |||
* Dashboard: https://docs.google.com/spreadsheets/d/1CAMaBlX6n5nKvKuBNUcmTZSaPaiGVvIT07u39lkirAE/edit?usp=sharing | |||
* Bugzilla Query for Unassigned MVP Bugs: https://mzl.la/2LR9ufS | |||
* (done) Maja and Henrik will discuss and change Bugzilla whiteboard tags (change to [puppeteer-beta-reserve]); email mmucci if contingency is removed | |||
* MVP Backlog following the scope reduction is as follows: 15 MVP in Total | |||
** 8 MVP unassigned and available | |||
** 1 MVP as contingency | |||
** 6 MVP assigned and in development | |||
=== Admin Topics === | |||
* New meeting time should be Wednesday 7:30 UTC or 4PM UTC? Probably 4UTC, Maja will send announcement. | |||
* Yes, move chat from Slack to Matrix - see [https://riot.im/app/#/room/#remote-protocol:mozilla.org #remote-protocol] on [[Matrix]] | |||
* jgraham will fill in with code review while whimboo is out next week | |||
=== Deprecation message === | |||
* See first line of https://github.com/puppeteer/puppeteer/issues/5397 | |||
* [digitarald] Keep it simple, the sooner message is out the better | |||
** (done) Maja will propose a message and update the Remote wiki page | |||
=== Update about Playwright === | |||
Joel: | |||
* Playwright will continue to use Juggler Firefox now, but they don't want to maintain a fork for firefox and webkit | |||
** Mainly focused on WebKit lately because it's harder | |||
* Interested in switching to Firefox remote protocol implementation but that's in the long run; want to provide help by answering questions about CDP, Blink, Juggler | |||
** Still concerned by going after CDP as the protocol layer, but will cooperate regardless of what path Firefox chooses | |||
* Fixing bugs in Chrome sometimes leaves Puppeteer in a broken state; when Puppeteer team asks questions, Joel tries to be responsive. | |||
* Six people on Playwright, but Joel only offers himself as a point of contact about technical questions. For "official" answers to other questions, Pavel is the right person. | |||
Questions: | |||
== | * Searching through CDP implementation | ||
** [joel] CDP is spread out in Chrome intentionally to be as close as possible to relevant Blink code | |||
** [maja] [https://docs.google.com/document/d/1c-COD2kaK__5iMM5SEx-PzNA7HFmgttcYfOHHX0HaOM/edit Guide to CDP contributions] shows where code is -- see also [[Remote/Developer_Resources]] | |||
* [whimboo] Do you have metrics that indicate what Playwright methods are most used? | |||
** [joel] Everything works and is prioritized equally, priority is feature completeness, large test suite. More unit tests in an area might indicate more bug reports from users? | |||
=== Puppeteer issue triage === | |||
* Feedback from users after initial 2.1 announcement? | |||
** [maja] A bit of activity, not huge. Mainly confusion about which version of Firefox to use, fetching/launching the browser automatically, feature-completeness in comparison with puppeteer-firefox (i.e. Should puppeteer-firefox still be used for now?) | |||
** A user contributed improvements to the Puppeteer docs about cross-browser support, https://github.com/puppeteer/puppeteer/pull/5360 | |||
* Browser fetching is in progress by Maja | |||
=== Other Business === | |||
* [joel] Do puppeteer unit tests run in Mozilla CI? Yes | |||
** James has a patch for setting test metadata -- this will allow running them as Tier 2 | |||
* [joel] Seems like users are confused by which Firefox to run; Joel is doing is best to clarify on which browser is being used implicitly by Playwright | |||
* [whimboo] - Should we enable remote agent on Beta? | |||
** [jgraham] Not yet, too much change. Once more stable, enabling on DevEdition (beta) makes the most sense given the audience. | |||
== Progress == | == Progress == | ||
| Line 20: | Line 80: | ||
=== Milestone Status === | === Milestone Status === | ||
* '''Last week:''' | * '''Last week:''' 70 Total; 49 Open (70%); 21 Resolved (30%); [January 24th] | ||
* '''This week:''' | * '''This week:''' 78 Total; 47 Open (60.26%); 31 Resolved (39.74%) | ||
=== Other indicators === | === Other indicators === | ||
| Line 30: | Line 90: | ||
<pre> | <pre> | ||
$ hg log -M -r | $ hg log -M -r 30ff42ad6a0d::central --template "{node|short}\t{author|person}\t{desc|firstline}\n" remote | ||
30ff42ad6a0d Henrik Skupin Bug 1587845 - [remote] ESLint fix for browser_captureScreenshot.js. | |||
abfe9c13a2f4 Coroiu Cristina Bug 1610374 - Disable browser_navigateToHistoryEntry.js on linux1804 and win for frequent failures r=remote-protocol-reviewers | |||
537d9cea1acd Noemi Erli Bug 1610382 - Disable browser_setCacheDisabled.js for frequent failures r=remote-protocol-reviewers,gbrown,whimboo | |||
d627299e3dd8 Maja Frydrychowicz Bug 1611274 - [puppeteer] Vendor in fixes for Launcher tests | |||
686af4c3274d David Burns Bug 1588114 - [remote] Implement Network.setCookie and Network.setCookies. r=remote-protocol-reviewers,maja_zf | |||
6f15878925ca Henrik Skupin Bug 1610377 - [remote] Move session history navigation code into the parent process. r=remote-protocol-reviewers,maja_zf | |||
d45350ccf8aa Henrik Skupin Bug 1610377 - [remote] Wait for history index selected when using gBrowser.gotoIndex(). r=remote-protocol-reviewers,maja_zf | |||
bf5ac2c3466b Mihai Alexandru Michis Bug 1610377 - Fix lint error in remote/test/browser/page/head.js a=lint-fix | |||
b9b47103dff1 Mark Banner Bug 1614891 - Clarify the ESLint situation around some remote/ files. Remove unnecessary remote/test/demo.js. r=remote-protoco | |||
</pre> | </pre> | ||
| Line 51: | Line 120: | ||
== PTO == | == PTO == | ||
* Henrik: PTO Feb 17th - 21st | |||