[e10s] fix browser_social_activation test

RESOLVED FIXED in Firefox 41

Status

()

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: mixedpuppy, Assigned: mixedpuppy)

Tracking

(Blocks: 2 bugs)

Trunk
Firefox 41
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(e10s+, firefox41 fixed)

Details

Attachments

(1 attachment, 3 obsolete attachments)

(Assignee)

Description

3 years ago
disabled in browser.ini for mouse synthisize
(Assignee)

Comment 1

3 years ago
Created attachment 8605996 [details] [diff] [review]
replace mouse synthisize
Assignee: nobody → mixedpuppy
Attachment #8605996 - Flags: review?(felipc)
(Assignee)

Comment 2

3 years ago
Created attachment 8606003 [details] [diff] [review]
replace mouse synthisize

minor non-e10s fix
Attachment #8605996 - Attachment is obsolete: true
Attachment #8605996 - Flags: review?(felipc)
Attachment #8606003 - Flags: review?(felipc)
Comment on attachment 8606003 [details] [diff] [review]
replace mouse synthisize

Review of attachment 8606003 [details] [diff] [review]:
-----------------------------------------------------------------

Did you try BrowserTestUtils.synthesizeMouseAtCenter ? That version should work with e10s.
Blocks: 984139
tracking-e10s: --- → +
(Assignee)

Comment 4

3 years ago
Created attachment 8608403 [details] [diff] [review]
replace mouse synthisize
Attachment #8606003 - Attachment is obsolete: true
Attachment #8606003 - Flags: review?(felipc)
Attachment #8608403 - Flags: review?(felipc)
(Assignee)

Comment 5

3 years ago
Comment on attachment 8608403 [details] [diff] [review]
replace mouse synthisize

noticed an issue, removing r?
Attachment #8608403 - Flags: review?(felipc)
(Assignee)

Comment 6

3 years ago
Created attachment 8608994 [details] [diff] [review]
replace mouse synthisize

leaving this disabled on e10s+debug since it hit bug 1150147 on linux debug.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=5a02b82e05da
Attachment #8608403 - Attachment is obsolete: true
Attachment #8608994 - Flags: review?(felipc)
Comment on attachment 8608994 [details] [diff] [review]
replace mouse synthisize

Review of attachment 8608994 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/base/content/test/social/browser.ini
@@ +33,5 @@
>  skip-if = e10s && debug # Leaking docshells (bug 1150147)
>  [browser_share.js]
>  skip-if = true # bug 1115131
>  [browser_social_activation.js]
> +skip-if = e10s && debug # e10s/Linux/Debug Leaking docshells (bug 1150147)

make a note in bug 1150147 about this test
Attachment #8608994 - Flags: review?(felipc) → review+
(Assignee)

Updated

3 years ago
Blocks: 1150147
https://hg.mozilla.org/mozilla-central/rev/31c58b01300a
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox41: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 41
You need to log in before you can comment on or make changes to this bug.