Return "no such window" when listener cannot be reached

RESOLVED FIXED in Firefox 48

Status

Testing
Marionette
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: ato, Assigned: ato)

Tracking

(Blocks: 1 bug, {ateam-marionette-server})

Version 3
mozilla49
ateam-marionette-server
Points:
---

Firefox Tracking Flags

(firefox48 fixed, firefox49 fixed)

Details

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

a year ago
We currently return a "no such frame" error when the listener cannot be reached.  We should return "no such window".

This is going to be a stopgap solution to work with wdspec tests.  The proper solution is to follow the steps as laid out by the specification with regards to when existence of the current browsing context (i.e. listener) should take place.
(Assignee)

Updated

a year ago
Assignee: nobody → ato
Blocks: 721859
Keywords: ateam-marionette-server
(Assignee)

Comment 1

a year ago
Created attachment 8755500 [details]
MozReview Request: Bug 1275043 - Return "no such window" error when listener cannot be reached; r?automatedtester

This is a stopgap solution to work with wdspec tests.  The proper solution
is to follow the steps as laid out by the specification with regards to
when existence of the current browsing context (i.e. listener) should
take place.

Review commit: https://reviewboard.mozilla.org/r/54642/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/54642/
Attachment #8755500 - Flags: review?(dburns)
(Assignee)

Comment 2

a year ago
Comment on attachment 8755500 [details]
MozReview Request: Bug 1275043 - Return "no such window" error when listener cannot be reached; r?automatedtester

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/54642/diff/1-2/
Comment on attachment 8755500 [details]
MozReview Request: Bug 1275043 - Return "no such window" error when listener cannot be reached; r?automatedtester

https://reviewboard.mozilla.org/r/54642/#review51258
Attachment #8755500 - Flags: review?(dburns) → review+

Comment 4

a year ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/8a2859f10ae0
(Assignee)

Updated

a year ago
Status: NEW → ASSIGNED

Comment 5

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/8a2859f10ae0
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
status-firefox49: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49

Comment 6

a year ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-aurora/rev/fb6c47bc3e2d
status-firefox48: --- → fixed

Comment 7

a year ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-aurora/rev/fb6c47bc3e2d
You need to log in before you can comment on or make changes to this bug.