Selenium atom "Get Visible Text" ignores slot value for web component without custom text
Categories
(Remote Protocol :: Marionette, defect, P3)
Tracking
(firefox124 fixed)
| Tracking | Status | |
|---|---|---|
| firefox124 | --- | fixed |
People
(Reporter: whimboo, Assigned: whimboo)
References
(Blocks 2 open bugs, )
Details
(Whiteboard: [webdriver:m10][wptsync upstream][webdriver:relnote])
Attachments
(2 files)
In the following case the Selenium atom Get Visible Text has to return the value foobar because the default slot value is shown as well, but it only returns bar.
<script>
class TestContainer extends HTMLElement {
connectedCallback() {
this.attachShadow({ mode: 'open' });
this.shadowRoot.innerHTML = `<slot><span>foo</span></slot>bar`;
}
}
customElements.define('test-container', TestContainer);
</script>
</head>
<body>
<test-container></test-container>
</body>
|
Assignee | |
Updated•7 months ago
|
|
|
Assignee | |
Comment 1•7 months ago
|
||
I've filed the Selenium issue as: https://github.com/SeleniumHQ/selenium/issues/13206
At the moment it's not blocking us, so as agreed on it's P3 for now.
|
|
Assignee | |
Comment 2•7 months ago
|
||
The work on bug 1865359 will fix that bug as well.
|
|
Assignee | |
Comment 3•5 months ago
|
||
(In reply to Henrik Skupin [:whimboo][⌚️UTC+1] from comment #1)
I've filed the Selenium issue as: https://github.com/SeleniumHQ/selenium/issues/13206
This is now fixed. Based on that we need an update of the Selenium atoms again. Therefore I created https://github.com/w3c/webdriver/issues/1783.
|
|
Assignee | |
Comment 4•5 months ago
|
||
I've opened a PR for the webdriver repository to bump the Selenium atoms revision: https://github.com/w3c/webdriver/pull/1784
|
|
Assignee | |
Comment 5•5 months ago
|
||
The WebDriver classic PR got merged, as such we can land a patch to update the atoms.
|
|
Assignee | |
Comment 6•4 months ago
|
||
|
||
Updated•4 months ago
|
|
|
Assignee | |
Comment 7•4 months ago
|
||
Depends on D201582
|
|
Assignee | |
Updated•4 months ago
|
Pushed by hskupin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/93a90f93c765 [wdspec] Fix escaping in test_shadow_root_slot[default hidden]. r=webdriver-reviewers,jdescottes https://hg.mozilla.org/integration/autoland/rev/4841b6b6e4f2 [marionette] Update Selenium atoms to 21560a497470f0c769931c17fa9803c6cddf6f1c. r=webdriver-reviewers,jdescottes
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/44549 for changes under testing/web-platform/tests
|
||
Comment 10•4 months ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/93a90f93c765
https://hg.mozilla.org/mozilla-central/rev/4841b6b6e4f2
Upstream PR merged by moz-wptsync-bot
|
|
Assignee | |
Updated•4 months ago
|
Description
•