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

Out of Memory when BiDi is enabled #1839

Closed
joerg1985 opened this issue Feb 6, 2024 · 1 comment · Fixed by #1848
Closed

Out of Memory when BiDi is enabled #1839

joerg1985 opened this issue Feb 6, 2024 · 1 comment · Fixed by #1848
Assignees

Comments

@joerg1985
Copy link

The DOM of the BiDi-CDP Mapper tab does grow unlimited, this results in a out of memory issue in this tab.
Depending on the available RAM it might take some time to run into this issue.

Minimal code to reproduce the issue using selenium is listed below.
When running real code this usually happens faster, as more things are logged.

        var options = new org.openqa.selenium.chrome.ChromeOptions();
        options.setCapability("webSocketUrl", true);

        WebDriver driver = new org.openqa.selenium.chrome.ChromeDriver(options);

        for (int i = 0; i < 1_000_000_000; i++) {
            driver.navigate().refresh();
        }
OrKoN pushed a commit that referenced this issue Feb 22, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.5.10](chromium-bidi-v0.5.9...chromium-bidi-v0.5.10)
(2024-02-22)


### Features

* support user context in permissions
([#1805](#1805))
([a623dc7](a623dc7))


### Bug Fixes

* limit the mapper tab to 200 lines of logs
([#1846](#1846))
([9c60d0a](9c60d0a)),
closes
[#1839](#1839)

---
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>
@sadym-chromium
Copy link
Collaborator

sadym-chromium commented Mar 7, 2024

Completed in #1846.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants