Closed
Bug 1262055
Opened 9 years ago
Closed 9 years ago
Fix browser_bug562797.js to work under e10s
Categories
(Toolkit :: Add-ons Manager, defect)
Toolkit
Add-ons Manager
Tracking
()
RESOLVED
FIXED
mozilla48
People
(Reporter: Unfocused, Assigned: Unfocused)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
|
5.78 KB,
patch
|
mossop
:
review+
|
Details | Diff | Splinter Review |
browser_bug562797.js is currently disabled for e10s-mode, as one of the test parts needs rewritten. Which is extra-fun because it deals with the browser frame switching between being a child/parent process.
|
Assignee | |
Comment 1•9 years ago
|
||
Ended up rewriting that troublesome part of the test to use promises & Task.spawn(), so BrowserTestUtils could be used without it becoming unreadable spaghetti. Would be nice to convert the rest sometime in the future.
Note that the comment in browser-common.ini about why this was disabled is a red-herring - EventUtils.synthesizeMouse() is used, but it's used when the page loaded is in the parent process.
|
|
Assignee | |
Comment 2•9 years ago
|
||
Oh, and Try results:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=ac0184cdc836
(I fixed the linter issues)
|
||
Updated•9 years ago
|
|
||
Updated•9 years ago
|
Attachment #8738032 -
Flags: review?(dtownsend) → review+
|
||
Comment 4•9 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in
before you can comment on or make changes to this bug.
Description
•