Closed Bug 901776 Opened 6 years ago Closed 6 years ago

Update isElementDisplayed atom to handle hidden attribute

Categories

(Testing :: Marionette, defect)

x86
macOS
defect
Not set

Tracking

(firefox24 wontfix, firefox25 wontfix, firefox26 fixed, b2g18 fixed, b2g18-v1.0.0 wontfix, b2g18-v1.0.1 wontfix, b2g-v1.1hd fixed)

RESOLVED FIXED
mozilla26
Tracking Status
firefox24 --- wontfix
firefox25 --- wontfix
firefox26 --- fixed
b2g18 --- fixed
b2g18-v1.0.0 --- wontfix
b2g18-v1.0.1 --- wontfix
b2g-v1.1hd --- fixed

People

(Reporter: automatedtester, Assigned: automatedtester)

References

Details

Attachments

(1 file)

The hidden attribute in html5 affects the visibility, this has been updated in the webdriver spec and in the atoms
Depends on: 901781
Assignee: nobody → dburns
Attached patch bug901776.patchSplinter Review
Attachment #792812 - Flags: review?(mdas)
Attachment #792812 - Flags: review?(mdas) → review+
Whiteboard: [checkin-needed-b2g18]
https://hg.mozilla.org/mozilla-central/rev/3c3df71ac20a
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
testing/marionette/atoms/atoms.js is almost an unreadable blob of javascript. I wonder how you are able to patch it.
I wonder because I wonder if opacity:0 is considered isElementDisplayed or not.
The code is "compiled" from the Selenium project. Elements with Opacity:0 should not be visible.
You need to log in before you can comment on or make changes to this bug.