External media tests should use Marionette variables for context names instead of string literals

RESOLVED FIXED in mozilla51

Status

RESOLVED FIXED
2 years ago
11 months ago

People

(Reporter: bryce, Assigned: bryce)

Tracking

Trunk
mozilla51

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
Marionette provides the Marionette.CONTEXT_CONTENT and Marionette.CONTEXT_CHROME variables to be used when setting contexts. These should be used instead of the string literals in the tests.
(Assignee)

Updated

2 years ago
Assignee: nobody → bvandyk
(Assignee)

Comment 1

2 years ago
Created attachment 8778029 [details]
Bug 1292396 - Replace Marionette context string literals with symbolic variables.

Review commit: https://reviewboard.mozilla.org/r/69410/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/69410/
Attachment #8778029 - Flags: review?(dburns)
Attachment #8778029 - Flags: review?(dburns) → review+
Comment on attachment 8778029 [details]
Bug 1292396 - Replace Marionette context string literals with symbolic variables.

https://reviewboard.mozilla.org/r/69410/#review66862

Comment 3

2 years ago
Pushed by bvandyk@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2c1f6bf57d21
Replace Marionette context string literals with symbolic variables. r=automatedtester

Comment 4

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/2c1f6bf57d21
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox51: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51

Updated

11 months ago
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.