1671372 - Perma Fis /webdriver/tests/release_actions/release.py | test_no_browsing_context - AssertionError: unknown error (500): TypeError: can't access property "getActor", browsingContext.currentWindowGlobal is null

Status: RESOLVED FIXED

(Testing :: Marionette Client and Harness, defect, P1)

Description

[Treeherder Bug Filer]

Filed by: hskupin [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer.html#?job_id=318701846&repo=try
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/H8js8XaCQHW9rkka6d7Y0A/runs/0/artifacts/public/logs/live_backing.log

---

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Here the exact failure:

[task 2020-10-15T09:59:17.924Z] 09:59:17     INFO - PID 9484 | 1602755957921	Marionette	DEBUG	0 -> [0,55,"WebDriver:ElementClick",{"id":"84aa04ee-de73-48a2-bb70-cf65e9d42814"}]
[task 2020-10-15T09:59:17.944Z] 09:59:17     INFO - PID 9484 | 1602755957933	Marionette	TRACE	[41] Frame with id 12884901890 got removed
[task 2020-10-15T09:59:17.944Z] 09:59:17     INFO - PID 9484 | 1602755957934	Marionette	TRACE	Received DOM event click for [object HTMLInputElement]
[task 2020-10-15T09:59:17.945Z] 09:59:17     INFO - PID 9484 | 1602755957935	Marionette	TRACE	Canceled page load listener because current frame has been removed
[task 2020-10-15T09:59:17.946Z] 09:59:17     INFO - PID 9484 | 1602755957935	Marionette	DEBUG	0 <- [1,55,null,{"value":null}]
[task 2020-10-15T09:59:17.946Z] 09:59:17     INFO - PID 9484 | 1602755957937	webdriver::server	DEBUG	<- 200 OK {"value":null}
[task 2020-10-15T09:59:17.946Z] 09:59:17     INFO - PID 9484 | 1602755957939	webdriver::server	DEBUG	-> DELETE /session/cc8cb509-4130-4306-b8a2-462e2c87e0b3/actions
[task 2020-10-15T09:59:17.947Z] 09:59:17     INFO - PID 9484 | 1602755957940	Marionette	DEBUG	0 -> [0,56,"WebDriver:ReleaseActions",{}]
[task 2020-10-15T09:59:17.948Z] 09:59:17     INFO - PID 9484 | 1602755957940	Marionette	DEBUG	0 <- [1,56,{"error":"unknown error","message":"TypeError: can't access property \"getActor\", browsingContext.currentWindowGlobal  ... ionette/content/driver.js:382:3\nGeckoDriver.prototype.releaseActions@chrome://marionette/content/driver.js:1965:16\n"},null]
[task 2020-10-15T09:59:17.950Z] 09:59:17     INFO - PID 9484 | Marionette threw an error: TypeError: can't access property "getActor", browsingContext.currentWindowGlobal is null
[task 2020-10-15T09:59:17.950Z] 09:59:17     INFO - PID 9484 | GeckoDriver.prototype.getActor@chrome://marionette/content/driver.js:382:3
[task 2020-10-15T09:59:17.950Z] 09:59:17     INFO - PID 9484 | GeckoDriver.prototype.releaseActions@chrome://marionette/content/driver.js:1965:16

As it looks like this needs a fix for bug 1666204.

Comment 2

[Chris Peterson [:cpeterson]]

Tracking marionette-fission-mvp bugs for Fission Beta milestone (M7)

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+2]]

At least for WebDriver:ReleaseActions we have an inappropriate browsing context check:

https://searchfox.org/mozilla-central/rev/e0eb861a187f0bb6d994228f2e0e49b2c9ee455e/testing/marionette/driver.js#1961,1965

While the check in line 1961 operates on the top-level browsing context, the actor used to run the command is the current one.

We will have to update line 1961 to also use the current browsing context.

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Lets keep this bug for release actions.

Comment 5

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Bug 1671372 - [marionette] "WebDriver:ReleaseActions" has to check the current and not top-level browsing context.

Comment 6

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Bug 1671372 - [wdspec] "Release Actions" has to check for current and not top-level browsing context.

Based on the proposed WebDriver spec changes:
https://github.com/w3c/webdriver/pull/1555

Depends on D93751

Comment 7

[Intermittent Failures Robot]

7 failures in 3694 pushes (0.002 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• try: 7

Platform breakdown:

• linux1804-64-qr: 5
• windows10-64-qr: 2

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1671372&startday=2020-10-12&endday=2020-10-18&tree=all

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/693a38392149
[marionette] "WebDriver:ReleaseActions" has to check the current and not top-level browsing context. r=marionette-reviewers,jdescottes
https://hg.mozilla.org/integration/autoland/rev/cb435bbb7984
[wdspec] "Release Actions" has to check for current and not top-level browsing context. r=webdriver-reviewers,maja_zf

Comment 9

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/693a38392149
https://hg.mozilla.org/mozilla-central/rev/cb435bbb7984

Comment 10

[Henrik Skupin [:whimboo][⌚️UTC+2]]

I miss a sync request from the wpt bot. James can you please check?

Comment 11

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/26223 for changes under testing/web-platform/tests

Comment 12

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot

Comment 13

[BMO Automation]

Moving bug to Testing::Marionette Client and Harness component per bug 1815831.