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

feat: webExtension module #3012

Merged
merged 11 commits into from
Feb 27, 2025
Merged

feat: webExtension module #3012

merged 11 commits into from
Feb 27, 2025

Conversation

LiviuArln
Copy link
Collaborator

@LiviuArln LiviuArln commented Jan 21, 2025

Closes #2849

Copy link
Collaborator

@sadym-chromium sadym-chromium left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job! Please add a couple of e2e tests, and it will be ready to merge:

Install + uninstall successfully.
Innstall invalid extension.

Copy link
Collaborator

@OrKoN OrKoN left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job! I left a few comments. Please add e2e tests as Maksim suggested. There are some WPT tests in progress in the Firefox repository so we do not need to add any for now.

@OrKoN
Copy link
Collaborator

OrKoN commented Feb 20, 2025

@LiviuArln I have pushed an update to enable CDP-fix and verified that the test returns a good CDP error. Could you please add other tests and implement error handling? Currently, only the non-chromedriver test configuration works (CHROMEDRIVER=false VERBOSE=true npm run e2e).

@sadym-chromium I think ChromeDriver is doing exposeDevToolsProtocol elsewhere when it loads mapper. Would you mind updating it? let's chat about it tomorrow.

@OrKoN OrKoN force-pushed the extensions branch 2 times, most recently from 3574a9d to d57019f Compare February 20, 2025 19:43
@LiviuArln LiviuArln marked this pull request as ready for review February 26, 2025 18:12
@OrKoN OrKoN added the update-expectations Update WPT expectations label Feb 27, 2025
browser-automation-bot added a commit to browser-automation-bot/chromium-bidi that referenced this pull request Feb 27, 2025
OrKoN pushed a commit that referenced this pull request Feb 27, 2025
@OrKoN OrKoN enabled auto-merge (squash) February 27, 2025 10:26
@OrKoN OrKoN disabled auto-merge February 27, 2025 10:38
@OrKoN OrKoN enabled auto-merge (squash) February 27, 2025 10:52
@OrKoN OrKoN merged commit 3994b79 into main Feb 27, 2025
48 checks passed
@OrKoN OrKoN deleted the extensions branch February 27, 2025 10:58
sadym-chromium pushed a commit that referenced this pull request Feb 28, 2025
🤖 I have created a release *beep* *boop*
---


##
[2.1.0](chromium-bidi-v2.0.0...chromium-bidi-v2.1.0)
(2025-02-28)


### Features

* webExtension module
([#3012](#3012))
([3994b79](3994b79))


### Bug Fixes

* hide cdp.send after the mapper receives it
([#3137](#3137))
([9ad3d11](9ad3d11))

---
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement the webExtension module
4 participants