WebDriver/Meetings/2018-08-20

From MozillaWiki
Jump to: navigation, search

Agenda

  • Action items:
    • dburns to take care of removing extension prefs from Selenium or the whole file
    • dburns to talk to Mandy about work week in London
  • Frame scripts are going away
  • Get cross-compilation work for geckodriver
  • Intermittents update
    • Perma ccov failure: Content crash sometimes causes parent to crash due to breakpad assertion (Bug 1483862)
    • Perma ccov failure: mozprocess conversation to Python3 caused ASCII decoding errors (Bug 1483850)
    • Lots of EdgeNeedsSweepUnbarrieredSlow assertions and crashes lately (Bug 1482029)
      • Offending patch got backed-out from firefox-62 beta
    • Page load regression for Firefox UI SSL tests by move from mozqa.com -> badssl.com (Bug 1478848)
      • Maybe an issue because we don't correctly handle the events correctly

Minutes

Action items

  • Extension prefs: Outstanding
  • Work week: Outstanding

Frame scripts are going away

ato
Context: https://mail.mozilla.org/pipermail/firefox-dev/2018-August/006719.html
To reduce memory consuption to have more content processes there is project Fission.
Their goal is to move code into JSMs and, in the long term, remove frame scripts.
It will be a sizable task to convert Marionette to the new model.
whimboo
I think this is going to impact us at some point in the future, but not necessarily something we have to take care of right now.
As long as frame scripts are not marked as deprecated, there shouldn’t be need to change anything in Marionette.
Even if everything else has been ported to ActorChilds, Firefox will run fast.
Marionette is disabled by default, so it shouldn’t affect regular users.

Get cross-compilation work for geckodriver

whimboo
ted got everything mostly working, thank you!
One thing remaining for getting geckodriver to compile, which was fixing the outstanding code review issue from glandium.
ted doesn’t have the time right now.
ato
I will reach out to chmanchester to see if he has the time.

ACTION: ato to talk to chmanchester

Intermittents update

whimboo
My proposal would be to highlight the most annoying ones from the past week.
Go over these in the meeting.
I am happy to explain how I went about figuring out the problems.
ato
For my part, it would be useful to keep tag on intermittents from week to week so that we don’t drop the ball on an important one.
whimboo
I will go over the list of intermittents every Monday.
But I’m happy to teach you how to do this.
The quality of the intermittent reports is questionable.

[talk about the intermittent from when whimboo was on PTO]

Status updates

  • ato
    • Fixed excessive log output from Marionette’s frame scripts (1482829)
      • This was considered an urgent priority to WPT (WPT issue 12166)
      • Means we no longer have process specific code in our log module, but instead rely on Log.jsm’s Logger#managerLevelFromPref which is better than rolling our own implementation as part of Services.ppmm.initialProcessData
    • Continued work on marionette-window-tracking
      • Now passes Mn tests, but still some trouble understanding what the FxUI tests are doing
      • Android still completely broken
      • Good news is that it will make hundreds of WPT reftests pass
    • Provided more number conversions for mozprofile::preferences::PrefValue (1483288)
    • Fixed a bug reported by roc about wptrunner not using the timeout multiplier for reftests run in Marionette (1483766)
    • Did some research into getting geckodriver/Marionette working in an IPv6 environment (1483996)
      • Turns out nsIServerSocket provides a way to define the interface and a Unix domain socket
    • Replaced an instance of nsITimer use in the action module with Sleep that I recently introduced (1484161)
    • Dropped duplicated outerWindowID logging in listener.js (1484153)
    • Fixed metadata and ‘File a bug’ link in the WebDriver standard (PR 1289), fixing issue 1288
    • Updated links to tools-marionette@ mailing list archive (1483805)
    • Wrote MDN documentation for the WebDriver Timeouts object
  • whimboo
    • [geckodriver] Serde work completed for WebDriver commands. Custom commands are left to do (Bug 1396821)
    • [wdspec] Add tests for POST commands which make use of a Null body (Bug 1469601)
    • [wdspec] Create basic tests for Get Window Handles and Get Window Handle (Bug 1483801)

(Spoken status updates in bold.)

PTO/travel (🏖)

  • Monday 27 August is a bank holiday in the UK