Marionette tries to use this.browser.selectedIndex, which is undefined

RESOLVED FIXED in mozilla54

Status

RESOLVED FIXED
2 years ago
a year ago

People

(Reporter: mconley, Assigned: whimboo)

Tracking

unspecified
mozilla54
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Noticed this while debugging bug 1294456:

http://searchfox.org/mozilla-central/rev/44f6964ba95b8ddd8ebf70c55b34cd2323afeef4/testing/marionette/browser.js#191

this.browser refers to either gBrowser in the Desktop case, or BrowserApp in the Fennec case. Neither of these things has a selectedIndex property available on them, so this is always returning undefined.

I suspect this is not intended.
This has been fixed through other bugs as this file no longer works like that. Closing.
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → INCOMPLETE
This has been fixed as part of bug 1323185 for Firefox 54.
Assignee: nobody → hskupin
Depends on: 1323185
Resolution: INCOMPLETE → FIXED
Target Milestone: --- → mozilla54
You need to log in before you can comment on or make changes to this bug.