Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

regression: geckodriver hung after interact with prompt from beforeunload event #1653

Closed
kovalev007 opened this issue Nov 14, 2019 · 10 comments

Comments

@kovalev007
Copy link

kovalev007 commented Nov 14, 2019

System

  • Version: geckodriver-v0.26.0-win64
  • Platform: windows 10
  • Selenium: java 3.141.59

Testcase

https://output.jsbin.com/fixoxun

  1. click on button with text "open"
  2. wait new window opened
  3. click on button with text "close"
  4. accept user promt
  5. wait new window closed

mozregression

firefox 2019-06-13 - test works without problem
firefox 2019-06-14 - test hung

https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=d72ab884f0394d6e9ab723f03bd43c90a84d7e5b&tochange=b3a69dff7e25f1c1107520c238dd282405afaf92

mozregression_log_from_2019-06-13_to_2019-06-14.txt
test.zip - archive with test on java and jar for execute via mozregression

@kovalev007
Copy link
Author

@whimboo we discussed about it on IRC. Should I provide more information?

@whimboo
Copy link
Collaborator

whimboo commented Nov 14, 2019

@kovalev007 the mozregression log only contains the beginning of the output. Would you mind to paste the last lines where it outputs the exact regressor? That would be important.

As talked about yesterday even we do not support enabling the beforeunload events right now, this is a newer regression in Firefox 69. As such it is not a dupe of issue #1563.

Also please attach that part of the geckodriver log which shows the interaction with the prompt, means when it gets opened, closed, and then failed. Thanks.

@kovalev007
Copy link
Author

@whimboo the mozregression log contains the full output. after last line I wait 5 - 10 minutes and nothing happens. geckodriver hung.

Please see additional example with 2019-06-13 and 2019-06-15.

mozregression_log_2019-06-13_good_and_2019-06-15_bad.txt

@whimboo
Copy link
Collaborator

whimboo commented Nov 19, 2019

In that situation can you just kill the browser process or close it yourself? That should also mark the test as failed, which instructs mozregression that it is a broken build. We really have to get further into that merge commit to figure out what has been causing it. Thanks.

@kovalev007
Copy link
Author

@whimboo you're right.

Full log now available.
full_mozregression_log_from_2019-06-13_to_2019-06-14.txt

Last good revision: 09614cb3a896ab3be71ccc44d7663eb5454cf5cf
First bad revision: 4651b31ffe5c012b5112fc3047c29d0d4a8f9594
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=09614cb3a896ab3be71ccc44d7663eb5454cf5cf&tochange=4651b31ffe5c012b5112fc3047c29d0d4a8f9594

@whimboo
Copy link
Collaborator

whimboo commented Nov 21, 2019

So it's the same patch again. Would you mind doing the hg up -r dance again and check which of the two first changesets are causing that for you?

@kovalev007
Copy link
Author

Second changeset caused this. 5d6397e990cbc374be49baaa4fc6622dfed18119

@whimboo
Copy link
Collaborator

whimboo commented Dec 3, 2019

Ok, that's good to know. Would you mind filing a bug in Bugzilla so that we can continue the investigation over there? As such we can also set the above bug as the regressor. Thanks!

@kovalev007
Copy link
Author

@whimboo
Copy link
Collaborator

whimboo commented Feb 22, 2021

That one got actually fixed already a while ago for Firefox 77.

@whimboo whimboo closed this as completed Feb 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants