1671770 - (actor) Perma Fis /webdriver/tests/switch_to_frame/switch.py | test_no_browsing_context[None] - 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=318910603&repo=try
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/X7JyizY-S9OZ2ptEq-NG7w/runs/0/artifacts/public/logs/live_backing.log

---

Comment 1

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

The failure here is when switching to the top-level frame that we have a wrong browsing context reference:

[task 2020-10-17T08:35:36.024Z] 08:35:36     INFO - PID 31210 | 1602923736019	Marionette	DEBUG	0 <- [1,312,null,{"value":{"element-6066-11e4-a52e-4f735466cecf":"ee3d7f29-a766-4495-9ab5-fab9302a94e4"}}]
[task 2020-10-17T08:35:36.026Z] 08:35:36     INFO - PID 31210 | 1602923736020	webdriver::server	DEBUG	<- 200 OK {"value":{"element-6066-11e4-a52e-4f735466cecf":"ee3d7f29-a766-4495-9ab5-fab9302a94e4"}}
[task 2020-10-17T08:35:36.028Z] 08:35:36     INFO - PID 31210 | 1602923736022	webdriver::server	DEBUG	-> POST /session/bb15fe59-f35b-4ec4-8407-1130af4dd1a2/frame {"id": null}
[task 2020-10-17T08:35:36.029Z] 08:35:36     INFO - PID 31210 | 1602923736024	Marionette	DEBUG	0 -> [0,313,"WebDriver:SwitchToFrame",{"id":null}]
[task 2020-10-17T08:35:36.037Z] 08:35:36     INFO - PID 31210 | 1602923736031	Marionette	DEBUG	0 <- [1,313,null,{"value":null}]
[task 2020-10-17T08:35:36.037Z] 08:35:36     INFO - PID 31210 | 1602923736032	webdriver::server	DEBUG	<- 200 OK {"value":null}
[task 2020-10-17T08:35:36.037Z] 08:35:36     INFO - PID 31210 | 1602923736033	webdriver::server	DEBUG	-> GET /session/bb15fe59-f35b-4ec4-8407-1130af4dd1a2/element/ee3d7f29-a766-4495-9ab5-fab9302a94e4/text
[task 2020-10-17T08:35:36.038Z] 08:35:36     INFO - PID 31210 | 1602923736034	Marionette	DEBUG	0 -> [0,314,"WebDriver:GetElementText",{"id":"ee3d7f29-a766-4495-9ab5-fab9302a94e4"}]
[task 2020-10-17T08:35:36.106Z] 08:35:36     INFO - PID 31210 | 1602923736099	Marionette	DEBUG	0 <- [1,314,null,{"value":"bar"}]

Problematic here is the following line:
https://searchfox.org/mozilla-central/rev/40b561b9accbff54387edcba2b8b4a3c3828110e/testing/marionette/driver.js#1800

Instead of using the actor from the top-level browsing context the one from the current browsing context is used.

Comment 2

[Intermittent Failures Robot]

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

Repository breakdown:

• try: 1

Platform breakdown:

• linux1804-64-qr: 1

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

Comment 3

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

Attachment: Bug 1671770 - [marionette] Fix browsing context checks for "WebDriver:SwitchToFrame".

Comment 4

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

Attachment: Bug 1671770 - [wdspec] Improve no browsing context checks for "Switch To Frame".

Depends on D94325

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4a2169905a5e
[marionette] Fix browsing context checks for "WebDriver:SwitchToFrame". r=marionette-reviewers,jdescottes
https://hg.mozilla.org/integration/autoland/rev/614856104da4
[wdspec] Improve no browsing context checks for "Switch To Frame". r=webdriver-reviewers,maja_zf

Comment 6

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

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

Comment 7

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/4a2169905a5e
https://hg.mozilla.org/mozilla-central/rev/614856104da4

Comment 8

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

Upstream PR merged by moz-wptsync-bot

Comment 9

[BMO Automation]

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