WebDriver/Meetings/2018-09-03

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
  • Marionette enabled by default in Fennec (1484220)
  • Marionette release notes for Firefox 62 release (all fixes)
  • Testing of geckodriver Nightly builds for next release
  • Intermittents update
    • [fixed] Page load regression for Firefox UI SSL tests by move from mozqa.com -> badssl.com (Bug 1478848)
    • [fixed] (Perma ts_paint_heavy) Heavy profile creater script fails for Nightly because of using geckdriver 0.18.0 (Bug 1486993)
    • [fixed] Content CSS loaded at document_start breaks about:blank (Bug 1465388) might help us for some intermittent failures where about:blank just showed an empty content area
    • [fixed] (Perma ccov) Content crash does not always trigger a content crash (Bug 1483862)
    • [needs fix] (Perma ccov) Assertion failure: m_refCount == 1 or pages_.empty() (Bug 1485985)
    • [needs fix] EdgeNeedsSweepUnbarrieredSlow assertions and crashes (Bug 1482029)
    • Re-enabled a series of wdspec tests which don't fail anymore (see status updates for Henrik)

Minutes

Actions

AutomatedTester
Extension prefs appear to have been removed from Selenium.
https://github.com/SeleniumHQ/selenium/commit/75303155030c8a0f5b25bd7724871893caa8817c
And work week sorted.
ato
Cross-compile patch for geckodriver is handled by Hiro now.
Blocks work to build geckodriver by default in local developer builds.
Expect this combined work to land in central over the next week or so.

Marionette enabled by default in Fennec

ato
Marionette is now enabled in Fennec for all builds.
Note that we only support local connections.
And that support in geckodriver is not there, which means we can’t officially say we support WebDriver on Android.
AutomatedTester
Might we need a PI request in regards of security for Marionette?
ato
There were no changes to the Marionette security model, so no new security review necessary.
Before we add Android support to geckodriver, we have to make sure we have the right socket communication model to Marionette.
We would need still a proxy, maybe adb supports a reverse proxy nowadays.
Android work on geckodriver will also have to be done for GeckoView.

Marionette release notes for Firefox 62 release

whimboo
62 is being released tomorrow.
Do we want to publish release notes?
ato
Yes, thanks for reminding us.
I’m fine for you to prepare a list.
Since Marionette changes aren’t going into the published release notes on mozilla.org, we don’t have to synchronise.
We can also add any items we forget to the wiki later.

ACTION whimboo: Setup a list of changes and get those landed

Testing of geckodriver Nightly builds for next release

whimboo
We want to reach out to real users to get them to run a nightly build of geckodriver before we release the new version with serde changes.
Internally: Dave Hunt
Selenium: barancev
Google: jjuang

ACTION whimboo: Reach out to Dave and barancev

Intermittents

whimboo
OF is broken again, so we have no weekly comments on Bugzilla for the last week.
AutomatedTester
There’s a weird timeout bug that Sarah is having trouble reproducing.
Trying to get better errors from OF.
whimboo
For now I can say that last week was a good week, based on bugs which got fixed.
There was a high frequent failure related to page load for a security page, which didn't check for a stale element first.
This unblocked us to decommission mozqa.com, and as such we don’t own any hosts in SCL3 anymore.
about:blank regression sometimes produced gray pages, but this is fixed in Nightly and Beta.
ato
https://hsivonen.fi/about-blank/
whimboo
Assertion failure: https://bugzilla.mozilla.org/show_bug.cgi?id=1485985
Requires kmag to have a look.
There has a patch being landed for a security bug, which might also fix the assertion https://bugzilla.mozilla.org/show_bug.cgi?id=1482029, and will make our wdspec tests mostly green again.
I also enabled some wdspec tests that no longer fail.

Status updates

(Spoken status updates in bold.)

  • ato
    • Changed timeout and script timeout error HTTP codes from 408 to 500 (1484941)
    • Continued work on including geckodriver in default built (1471281)
    • Tried to get help with test failures on WebDriver MDN sidebar change to no avail
    • Removed lazy_static dependency from webdriver library (1485414)
    • Various lint fixes for webdriver, mozprofile, and mozversion
  • Henrik
    • [wdspec] Re-enabled /webdriver/tests/actions/modifier_click.py (Bug 1447844)
    • [wdspec] Re-enabled /webdriver/tests/minimize_window/user_prompts.py (Bug 1485564)
    • [wdspec] Re-enabled /webdriver/tests/maximize_window/user_prompts.py (Bug 1399633)
    • [wdspec] Parsing timeout values given as floats fail when equal to result of toFixed() (Bug 1485580)
    • [wdspec] Create basic tests for "Get Element Rect" (Bug 1486456)
    • [wdspec] Create basic tests for "Get Element CSS Value" command (Bug 1486495)
    • [wdspec] Create tests for "Is Element Enabled" command (Bug 1486572)
    • [wdspec] Create basic tests for "Take Screenshot" and "Take Element Screenshot" (Bug 1486793)
    • [geckodriver] Finally landed the patch to transition over to Serde (Bug 1396821)
    • [geckodriver] Refactor Marionette code to it's own module (Bug 1481776)
    • [webdriver] Get rid of camel-case for pageLoad and sessionId (Bug 1486467)
    • [marionette] Repeated pause action hangs after a few seconds (Bug 1447449)
    • [marionette] "WebDriver:AcceptAlert" and "WebDriver:DismissAlert" have to wait until the dialog has been closed (Bug 1479368)
    • [marionette] "Accept Alert" and "Dismiss Alert" should check for remaining user prompts (Bug 1487358) - Selenium regression with form submission
    • [marionette] Exception NS_ERROR_FAILURE in ctx.scale() if width or height is greater than 32767 (Bug 1485730)
    • [marionette] Double-click tracker is not reset in "Release Actions" (Bug 1422583)
    • [marionette/geckodriver/webdriver] Fix /webdriver/tests/delete_session/delete.py (Bug 1484909)
    • [fxui] Intermittent testing/firefox-ui/tests/functional/security/test_ssl_disabled_error_page.py TestSSLDisabledErrorPage.test_ssl_disabled_error_page (Bug 1478848)


PTO/travel (🏖)

  • ato away Friday 7 and Monday 10 September
  • Henrik away Friday 7th September