1678455 - "WebDriver:ElementClick" on links with targets other than "_blank" can cause page load timeout errors

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, defect, P1)

Description

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

While writing tests for bug 1674329 I noticed that using "WebDriver:ElementClick" to click on links with different target values like "_top", "_parent", "_self" can fail, and causing a page load timeout error.

To be better equipped for bug 1674329 I will get this fixed first.

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

Attachment: Bug 1678455 - [wdspec] Add target navigation tests for Element Click.

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

Attachment: Bug 1678455 - [marionette] Add isLoadEventExpected checks for link targets.

Depends on D97758

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

Marking bug as marionette-fission-mvp because the broken behavior is blocking my work on bug 1674329.

Comment 4

[Chris Peterson [:cpeterson]]

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

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/38a53d22dd4e
[wdspec] Add target navigation tests for Element Click. r=webdriver-reviewers,jgraham,maja_zf
https://hg.mozilla.org/integration/autoland/rev/f3c67028d935
[marionette] Add isLoadEventExpected checks for link targets. r=marionette-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/26699 for changes under testing/web-platform/tests

Comment 7

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/38a53d22dd4e
https://hg.mozilla.org/mozilla-central/rev/f3c67028d935

Comment 8

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

(In reply to Web Platform Test Sync Bot (Matrix: #interop:mozilla.org) from comment #6)

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

James, can you please check why that PR doesn't get merged? Thanks.

Comment 9

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

Upstream PR merged by moz-wptsync-bot