Open Bug 1445227 Opened 7 years ago Updated 2 years ago

getPointerInteractablePaintTree does not return elements in a shadow DOM

Categories

(Remote Protocol :: Marionette, defect, P3)

Version 3
defect
Points:
5

Tracking

(Not tracked)

People

(Reporter: automatedtester, Unassigned)

References

(Blocks 1 open bug)

Details

When looking at shadow dom support in Marionette we are not doing the right things to spot elements in the shadow dom. 1) https://searchfox.org/mozilla-central/source/testing/marionette/element.js#1193-1197 we are using shadowRoot as setter when its only a getter 2)https://github.com/w3c/csswg-drafts/issues/556 says elements from point should return elements in the shadow dom but should return the host instead
Priority: -- → P2
Initial changes were added in bug 1359054
I don’t think we should assume that this is a regression, considering that test_shadow_dom.py was disabled due to https://bugzilla.mozilla.org/show_bug.cgi?id=1293844 and https://bugzilla.mozilla.org/show_bug.cgi?id=1387678. I think it is likely that this is a case of the Shadow DOM implementation in Gecko diverging from the earlier support for it in Marionette and a lack of test coverage/continued support for Shadow DOM in Marionette.
Priority: P2 → P3

If this is still a problem it might block bug 1700095.

Blocks: 1700095
No longer blocks: 1686180

David, do you think that this is still a problem? I wonder if that really blocks bug 1700095 which is for finding elements within a shadow root. I think that I inappropriately set it as blocker and it should only affect element interaction (send keys and click).

Flags: needinfo?(david.burns)

I think it should only be an issue for element interaction not the finding of things

Flags: needinfo?(david.burns)
No longer blocks: 1700095
Severity: normal → S3
Points: --- → 5
Priority: P3 → P2
Whiteboard: [webdriver:backlog]

WebDriver actions do not check if the element is in view and as such doesn't use getPointerInteractablePaintTree. So there is no need to block bug 1690884.

No longer blocks: 1690884
Priority: P2 → P3
Whiteboard: [webdriver:backlog]
Product: Testing → Remote Protocol
You need to log in before you can comment on or make changes to this bug.