Skip to content

Automate file dialog via WebDriver BiDi#452

Merged
a-sully merged 9 commits intoWICG:mainfrom
sadym-chromium:sadym/webdriver-bidi
Mar 17, 2025
Merged

Automate file dialog via WebDriver BiDi#452
a-sully merged 9 commits intoWICG:mainfrom
sadym-chromium:sadym/webdriver-bidi

Conversation

@sadym-chromium
Copy link
Contributor

@sadym-chromium sadym-chromium commented Feb 28, 2025

Required for automating file picker: w3c/webdriver-bidi#883


Preview | Diff

sadym-chromium added a commit to w3c/webdriver-bidi that referenced this pull request Mar 5, 2025
This PR implements `input.fileDialogOpened` event and defines `unhandledPromptBehavior` for `file`. It should be accompanied by the following changes:
HTML spec: whatwg/html#11030
WebDriver Classic spec: w3c/webdriver#1884
File system access: WICG/file-system-access#452
@w3cbot
Copy link

w3cbot commented Mar 13, 2025

sadym-chromium marked as non substantive for IPR from ash-nazg.

@w3cbot
Copy link

w3cbot commented Mar 13, 2025

sadym-chromium marked as substantive for IPR from ash-nazg.

@sadym-chromium sadym-chromium changed the title [WIP] file picker automation via WebDriver BiDi Automate file dialog via WebDriver BiDi Mar 13, 2025
@sadym-chromium sadym-chromium marked this pull request as ready for review March 13, 2025 09:34
Copy link
Collaborator

@a-sully a-sully left a comment

Choose a reason for hiding this comment

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

LGTM once comments are addressed

Eventually it would be nice if WebDriver's filePickerOptions supported more than just "multiple" (e.g. suggestedName) but I'm happy to see this hook being added and we can always add more support to that dictionary later

@sadym-chromium
Copy link
Contributor Author

@a-sully do you know who can merge the PR?

@a-sully
Copy link
Collaborator

a-sully commented Mar 17, 2025

@a-sully do you know who can merge the PR?

I can, but only once the CI is happy. Unfortunately it was failing unrelated to your PR. I just merged 8764848 which should fix it. You'll need to merge/rebase your branch to pull in that change so we can re-run CI with the new config

@sadym-chromium
Copy link
Contributor Author

I can, but only once the CI is happy. Unfortunately it was failing unrelated to your PR. I just merged 8764848 which should fix it. You'll need to merge/rebase your branch to pull in that change so we can re-run CI with the new config

rebased. Please approve the CI

@a-sully a-sully merged commit f57532d into WICG:main Mar 17, 2025
2 checks passed
github-actions bot added a commit that referenced this pull request Mar 17, 2025
* Respect "dismissed" from "WebDriver BiDi file dialog opened"

* showDirectoryPicker

* Pass |File Picker Options| to "WebDriver BiDi file dialog opened"

* remove "directory"

* fix reference to BiDi spec

* Update index.bs

Co-authored-by: Austin Sullivan <asully@google.com>

* Update index.bs

Co-authored-by: Austin Sullivan <asully@google.com>

* Update index.bs

Co-authored-by: Austin Sullivan <asully@google.com>

* filePickerOptions

---------

Co-authored-by: Austin Sullivan <asully@google.com>

SHA: f57532d
Reason: push, by @a-sully

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@sadym-chromium sadym-chromium deleted the sadym/webdriver-bidi branch March 19, 2025 11:02
Copy link

@sonnyquinn24 sonnyquinn24 left a comment

Choose a reason for hiding this comment

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

Done

Copy link

@sonnyquinn24 sonnyquinn24 left a comment

Choose a reason for hiding this comment

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

@sonnyquinn24
Copy link

@w3cbot

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 this pull request may close these issues.

4 participants