Closed Bug 1132688 Opened 9 years ago Closed 9 years ago

Convert Mozmill test 'remote/testSecurity/testMixedScriptContentBlocking.js' to Marionette

Categories

(Testing :: Firefox UI Tests, defect, P1)

Product:
Testing
Component:
Firefox UI Tests
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
mozilla39

People

(Reporter: galgeek, Assigned: chmanchester)

References

Details

Attachments

(1 file)

PR with converted test
52 bytes, text/x-github-pull-request
whimboo
: review+
whimboo
: feedback+
Details | Review 
This bug will cover the necessary work to convert the following test into Marionette.

http://hg.mozilla.org/qa/mozmill-tests/file/default/firefox/tests/remote/testSecurity/testMixedScriptContentBlocking.js
Blocks: 1132629
Priority: -- → P1
I'm going to work on this.
Assignee: nobody → cmanchester
Flags: needinfo?(cmanchester)
Flags: needinfo?(cmanchester)
Comment on attachment 8579036 [details] [review]
PR with converted test

I think the remaining failure was due to xvfb.
Attachment #8579036 - Flags: review?(hskupin)
Comment on attachment 8579036 [details] [review]
PR with converted test

I made my comments on the PR as usual.
Attachment #8579036 - Flags: review?(hskupin) → feedback+
Comment on attachment 8579036 [details] [review]
PR with converted test

I implemented suggested changes in a fixup or made a note of where I didn't.
Attachment #8579036 - Flags: feedback+ → review?(hskupin)
Comment on attachment 8579036 [details] [review]
PR with converted test

r=me with the nits as mentioned on the PR fixed.
Attachment #8579036 - Flags: review?(hskupin) → review+
Target Milestone: --- → Firefox 39
Product: Mozilla QA → Testing
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: