Closed Bug 1173330 Opened 9 years ago Closed 9 years ago

Fix and re-enable browser_bug638292.js in e10s mode

Categories

(Toolkit :: Add-ons Manager, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla44
Tracking Status
e10s + ---
firefox44 --- fixed

People

(Reporter: RyanVM, Assigned: enndeakin)

References

(Blocks 1 open bug)

Details

Attachments

(2 files, 1 obsolete file)

Attached image test screenshot
This test was disabled and annotated at one point with a pointer to bug 1083269. That bug is fixed now, but a Try push from yesterday says it's still failing.

https://treeherder.mozilla.org/logviewer.html#?job_id=8357482&repo=try
That test needs a number of changes:

- can just use BrowserTestUtils.openNewForegroundTab and not use waitForFocus
- not do comparisons on the content document, and instead use a ContentTask.
- use BrowserTestUtils.synthesizeMouseAtCenter instead.
Attached patch browser_bug638292 (obsolete) — Splinter Review
Haven't tested on e10s though.
Assignee: nobody → enndeakin
Status: NEW → ASSIGNED
Attached patch Fix testSplinter Review
Attachment #8621043 - Attachment is obsolete: true
Attachment #8670216 - Flags: review?(felipc)
Attachment #8670216 - Flags: review?(felipc) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/509dfb8dffa103a637e474fab8db61b9ea651072
Bug 1173330, fix to use content tasks and BrowserTestUtils, and reenable browser_bug638292.js, r=felipe
https://hg.mozilla.org/mozilla-central/rev/509dfb8dffa1
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: