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

fix: navigation with wait None #2557

Merged
merged 7 commits into from
Sep 2, 2024
Merged

Conversation

sadym-chromium
Copy link
Collaborator

@sadym-chromium sadym-chromium commented Sep 2, 2024

Do not wait for the CDP command in case BrowsingContext.navigate with wait: None. Provide pendingNavigationId to consistent navigation ID.

@sadym-chromium sadym-chromium added the update-expectations Update WPT expectations label Sep 2, 2024
@sadym-chromium sadym-chromium requested a review from OrKoN September 2, 2024 14:47
@sadym-chromium sadym-chromium marked this pull request as ready for review September 2, 2024 14:47
@sadym-chromium sadym-chromium force-pushed the sadym/navigate-wait-none branch from 28c56cd to 0c395a1 Compare September 2, 2024 15:02
@sadym-chromium sadym-chromium enabled auto-merge (squash) September 2, 2024 15:27
@sadym-chromium sadym-chromium merged commit bf89379 into main Sep 2, 2024
49 checks passed
@sadym-chromium sadym-chromium deleted the sadym/navigate-wait-none branch September 2, 2024 15:30
sadym-chromium pushed a commit that referenced this pull request Sep 5, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.7.0](chromium-bidi-v0.6.5...chromium-bidi-v0.7.0)
(2024-09-05)


### ⚠ BREAKING CHANGES

* emit `browsingContext.contextDestroyed` once
([#2563](#2563))

### Bug Fixes

* always provide url in `browsingContext.navigationStarted`
([#2483](#2483))
([318d621](318d621))
* emit `browsingContext.contextDestroyed` once
([#2563](#2563))
([930d401](930d401))
* fail previous navigation on the next one
([#2569](#2569))
([0cfd51a](0cfd51a))
* navigation with wait `None`
([#2557](#2557))
([bf89379](bf89379))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
update-expectations Update WPT expectations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants