"input.#getElementFromElementOrigin()" has to raise "no such element" error for non Elements as element origin.
Categories
(Remote Protocol :: WebDriver BiDi, defect, P1)
Tracking
(firefox115 fixed)
| Tracking | Status | |
|---|---|---|
| firefox115 | --- | fixed |
People
(Reporter: whimboo, Assigned: jdescottes)
References
(Blocks 1 open bug, )
Details
(Whiteboard: [webdriver:m7][wptsync upstream][webdriver:relnote])
Attachments
(2 files)
With the handling of Element Origin via bug 1806788 the newly added method input.#getElementFromElementOrigin() which is part of the windowglobal input module currently retrieves any kind of node and returns it without checking if it is an Element as the BiDi spec describes:
https://w3c.github.io/webdriver-bidi/#get-element-from-inputelementorigin-steps
Let element be the result of trying to deserialize remote reference with reference, realm, and session.
If element doesn’t implement Element return error with error code no such element.
|
Assignee | |
Updated•1 year ago
|
|
|
Assignee | |
Comment 1•1 year ago
|
||
At the moment the actions module would already throw an InvalidArgument error, but for BiDi we want explicitly to throw NoSuchElement in this case.
|
|
Assignee | |
Comment 2•1 year ago
|
||
Depends on D178185
|
||
Updated•1 year ago
|
|
|
Assignee | |
Updated•1 year ago
|
Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6f71f2bcd92f [bidi] Throw NoSuchElement if ElementOrigin's node is not an element r=webdriver-reviewers,whimboo https://hg.mozilla.org/integration/autoland/rev/b0a6c81cf587 [wdspec] Add test for NoSuchElement error with input.ElementOrigin r=webdriver-reviewers,jgraham
|
||
Comment 4•11 months ago
|
||
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/40127 for changes under testing/web-platform/tests
|
|
||
Updated•11 months ago
|
|
||
Comment 5•11 months ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/6f71f2bcd92f
https://hg.mozilla.org/mozilla-central/rev/b0a6c81cf587
|
|
||
Comment 6•11 months ago
|
||
Upstream PR merged by moz-wptsync-bot
|
Reporter | |
Updated•10 months ago
|
Description
•