Henrik Skupin [:whimboo][⌚️UTC+1] Reporter
	Description • 7 years ago

As shown in https://github.com/mozilla/geckodriver/issues/1026 our click + pageload logic fails if custom event handlers are in-place which delay the form submission. As such the beforeunload timer times out before the page gets loaded, and as such returns too early.

Example test:

    def test_delayed_page_load(self):
        self.marionette.timeout.implicit = 10
        self.marionette.navigate("http://demo.litecart.net/admin/")
        self.marionette.find_element(By.CSS_SELECTOR, "button[name=login]").click()
        logout = self.marionette.find_element(By.CSS_SELECTOR, "a[title=Logout]")
        logout.click()

	Henrik Skupin [:whimboo][⌚️UTC+1] Reporter
	Comment 1 • 7 years ago

Something we have to obey is to also register the onsubmit event listener for the capturing phase in case a custom listener on that page is using preventDefault. Also the registering has to happen before we click the element, and that no race conditions will occur.

I wonder if that is something we would have to add to the spec, because so far it only references the actual click event to wait for, but no other events which might also get fired as result of the click.

	Andreas Tolfsen ❲:ato❳
	Comment 2 • 7 years ago

(In reply to Henrik Skupin (:whimboo) from comment #1)

> I wonder if that is something we would have to add to the spec,
> because so far it only references the actual click event to wait
> for, but no other events which might also get fired as result of
> the click.

I think probably it makes some sense to introduce this feature to
the specification.  The Element Click command is meant to help
the user “do the right thing” when interacting with elements.
Waiting for a form’s submit DOM event is in line with the same
thinking that we have around flushing the event loop, or ensuring
that DOM events resulting from a click, successfully fire before we
return control to the user.

It is worth filing a bug on https://github.com/w3c/webdriver so we
can have the discussion in the proper forum.  I think the hard part
to this discussion will not be whether or not to _do it_, but how
to ensure we escape if the form submission does not mature, e.g. an
event handler prevents the default action.

	Henrik Skupin [:whimboo][⌚️UTC+1] Reporter
	Comment 3 • 7 years ago

This is on file now for the webdriver specification: https://github.com/w3c/webdriver/issues/1166

	David Burns :automatedtester
	Comment 4 • 6 years ago

[Mass Change 2018-01-15] Moving bugs to backlog