Closed Bug 1573839 Opened 3 months ago Closed 2 months ago

Change BrowserTestUtils synthesizeMouse/etc to handle out of process frames

Categories

(Testing :: Mochitest, task)

Version 3
task
Not set

Tracking

(Fission Milestone:M4, firefox71 fixed)

RESOLVED FIXED
mozilla71
Fission Milestone M4
Tracking Status
firefox71 --- fixed

People

(Reporter: enndeakin, Assigned: enndeakin)

References

(Blocks 1 open bug, Regressed 1 open bug)

Details

Attachments

(2 files)

This involves waiting for child iframes to load as well as being able to synthesize mouse/key events in child iframes.

Summary: Change BrowserTestUtils to handle out of process frames → Change BrowserTestUtils browserLoaded/synthesizeMouse/etc to handle out of process frames
Depends on: 1559841

Note that Kashav is currently working on some of these helper methods

See Also: → 1578465

Note that bug 1559841 is needed to run some of the tests that use this.

Fission Milestone: --- → M4

I had this extra test_context_menu_iframe.js change in my password manager set of patches but it actually belongs here as it would otherwise fail.

Pushed by neil@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4b4202682c55
modify BrowserTestUtils.synthesizeMouse/Key/etc to handle fission. Remove support for passing arrays of selectors and instead modify the last argument to be a browsing context, or browser. Fix up tests that use this obsolete form, r=kmag
https://hg.mozilla.org/integration/autoland/rev/d79153e62c0c
change browser_context_menu_iframe.js to pass a browsing context for synthesizeMouse calls, also fixes test_context_menu_iframe_sandbox_same_origin test that was incorrectly using the wrong frame, r=MattN

The browser_625016.js test fails for me locally every single time whether I have the changes in this bug applied or not. As my changes here are test-only changes unrelated to this failing test, I would instead suggest that that browser_625016.js is just broken entirely.

Flags: needinfo?(enndeakin)
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
Summary: Change BrowserTestUtils browserLoaded/synthesizeMouse/etc to handle out of process frames → Change BrowserTestUtils synthesizeMouse/etc to handle out of process frames
You need to log in before you can comment on or make changes to this bug.