Closed Bug 1165046 Opened 9 years ago Closed 9 years ago

[e10s] fix browser_social_activation test

Categories

(Firefox Graveyard :: SocialAPI, defect)

defect
Not set
normal

Tracking

(e10s+, firefox41 fixed)

RESOLVED FIXED
Firefox 41
Tracking Status
e10s + ---
firefox41 --- fixed

People

(Reporter: mixedpuppy, Assigned: mixedpuppy)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 3 obsolete files)

disabled in browser.ini for mouse synthisize
Attached patch replace mouse synthisize (obsolete) — Splinter Review
Assignee: nobody → mixedpuppy
Attachment #8605996 - Flags: review?(felipc)
Attached patch replace mouse synthisize (obsolete) — Splinter Review
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.
Attached patch replace mouse synthisize (obsolete) — Splinter Review
Attachment #8606003 - Attachment is obsolete: true
Attachment #8606003 - Flags: review?(felipc)
Attachment #8608403 - Flags: review?(felipc)
Comment on attachment 8608403 [details] [diff] [review]
replace mouse synthisize

noticed an issue, removing r?
Attachment #8608403 - Flags: review?(felipc)
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+
Blocks: 1150147
https://hg.mozilla.org/mozilla-central/rev/31c58b01300a
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 41
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: