-
Notifications
You must be signed in to change notification settings - Fork 45
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
Navigate and reload calls are out of sync with the HTML spec? #858
Comments
I suspect the key thing we want is to use the |
Also, I think a navigation ID is not a parameter accepted by the HTML spec. We should probably use https://html.spec.whatwg.org/multipage/browsing-the-web.html#ongoing-navigation. |
Yes, it looks like HTML wants to handle creating the ID now. |
I apologize that we messed up this integration. If you need something from the HTML spec to let you generate navigation IDs and pass them in, that should not be a problem. |
I think a bigger problem is that WebDriver BiDi spec currently assumes that page reloads are behaving like regular spec navigations resulting "navigation started" and other events as instrumented in https://html.spec.whatwg.org/#navigate. So what is considered to be a navigation has different meanings in the HTML and WebDriver BiDi specs, I think WebDriver BiDi is closer to what is documented in this note https://html.spec.whatwg.org/#navigating-across-documents:navigate-2 (only in terms of reloads and traversals, the history updates are also separate in WebDriver) |
"default"
as history handling. HTML spec has auto/replace/push only."reload"
as history handling. HTML spec does not support it.Should we use "auto" for history handling in navigate and call https://html.spec.whatwg.org/multipage/browsing-the-web.html#reload to specify the reload command?
cc @jgraham @sadym-chromium
The text was updated successfully, but these errors were encountered: