Closed Bug 1298133 Opened 8 years ago Closed 7 years ago

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

Categories

(Remote Protocol :: Marionette, defect)

Product:
Remote Protocol
Component:
Marionette
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
mozilla54

People

(Reporter: mconley, Assigned: whimboo)

References

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
Closed: 7 years 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
Product: Testing → Remote Protocol
You need to log in before you can comment on or make changes to this bug.