[Webkit-unassigned] [Bug 174670] WebDriver: wait until navigation is complete before running new commands and after a click

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jul 20 10:48:42 PDT 2017


--- Comment #7 from Brian Burg <bburg at apple.com> ---
Comment on attachment 315976
  --> https://bugs.webkit.org/attachment.cgi?id=315976

View in context: https://bugs.webkit.org/attachment.cgi?id=315976&action=review

> Source/WebKit/UIProcess/Automation/WebAutomationSession.cpp:385
> +

I'll hold off on being pedantic about spec conformance until we fix the FIXME. When you implement the page load strategy and the rest of the steps in §9, please include the steps in comments here and in the private methods so it's easy to verify we do the right thing.

> Source/WebKit/UIProcess/Automation/WebAutomationSession.cpp:400
> +    if (auto callback = m_pendingNavigationInBrowsingContextCallbacksPerPage.take(page.pageID()))

Note: per §6.3, substeps 4.3.1–4.3.4, the spec assumes commands are serialized into a queue per session, otherwise one command could cause another one to timeout if they both need to wait for navigation to complete.

I don't think it's the case right now that we queue commands explicitly or implicitly in WebKitDriver, nor in safaridriver. So this could hypothetically be a problem right now, but it seems unlikely in practice.

You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20170720/18f47ae2/attachment.html>

More information about the webkit-unassigned mailing list