1669920 - (actors) testing/marionette/harness/marionette_harness/tests/unit/test_switch_frame_chrome.py TestSwitchFrameChrome.test_stack_trace - NoSuchWindowException: No such content frame; perhaps the listener was not registered?

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, defect, P1)

Description

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

With bug 1668846 fixed we call this.listener.setBrowsingContextId() from WebDriver:SwitchToFrame and WebDriver:SwitchToParentFrame:

https://searchfox.org/mozilla-central/rev/f82d5c549f046cb64ce5602bfd894b7ae807c8f8/testing/marionette/driver.js#1755,1807

To avoid failures like the following this only has to be done when staying in Content context.

1602147179320	Marionette	DEBUG	2 -> [0,30,"WebDriver:SwitchToFrame",{"focus":true,"id":0}]
1602147179321	Marionette	TRACE	** switched to context: 37
1602147179321	Marionette	DEBUG	2 <- [1,30,{"error":"no such window","message":"No such content frame; perhaps the listener was not registered?","stacktrace":"Web ... arionette/content/proxy.js:37:29\nGeckoDriver.prototype.switchToFrame@chrome://marionette/content/driver.js:1807:25\n"},null]

Comment 1

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

Affected tests:

test_switch_frame_chrome.py test_switch_frame_chrome.TestSwitchFrameChrome.test_stack_trace
test_switch_frame_chrome.py test_switch_frame_chrome.TestSwitchFrameChrome.test_switch_simple
test_window_status_content.py test_window_status_content.TestNoSuchWindowContent.test_closed_chrome_window_while_in_frame
test_window_status_chrome.py test_window_status_chrome.TestNoSuchWindowChrome.test_closed_chrome_window_while_in_frame
test_window_status_content.py test_window_status_content.TestNoSuchWindowContent.test_closed_chrome_window_while_in_frame

Comment 2

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

Attachment: Bug 1669920 - [marionette] Don't set the framescript's browsing context id when in chrome scope.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3cca52c49b8d
[marionette] Don't set the framescript's browsing context id when in chrome scope. r=marionette-reviewers,maja_zf

Comment 4

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/3cca52c49b8d