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•2 years ago
|
| Assignee | ||
Comment 1•2 years 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•2 years ago
|
||
The work on bug 1865359 will fix that bug as well.
| Assignee | ||
Comment 3•2 years 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•2 years 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•2 years ago
|
||
The WebDriver classic PR got merged, as such we can land a patch to update the atoms.
| Assignee | ||
Comment 6•2 years ago
|
||
Updated•2 years ago
|
| Assignee | ||
Comment 7•2 years ago
|
||
Depends on D201582
| Assignee | ||
Updated•2 years ago
|
Comment 10•2 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/93a90f93c765
https://hg.mozilla.org/mozilla-central/rev/4841b6b6e4f2
| Assignee | ||
Updated•2 years ago
|
Description
•