Closed Bug 1255173 Opened 7 years ago Closed 7 years ago

Fix onZoomReset() in test_wheel_default_action.html to work with e10s

Categories

(Core :: DOM: Events, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla48
Tracking Status
e10s + ---
firefox48 --- fixed

People

(Reporter: mccr8, Assigned: mccr8)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

No description provided.
With this fixed, all of the tests except doTestWholeScroll() work with e10s enabled.

Fixing this requires two things. First, "test.events.async.enabled" must be set to true so we generate key events in the parent process. Second, some SpecialPowers infrastructure must be set up to send "browser-fullZoom:zoomReset" to the child process.
"test.events.async.enabled" must be set to true so we generate key events in the parent process, which is what triggers the zoom reset.

In addition, some SpecialPowers infrastructure must be set up to listen for "browser-fullZoom:zoomReset" in the parent and send it to an observer in the child.

try run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=ed14a344e2b2

This does not yet enable the test with e10s because there are more remaining failures.

r?masayuki for the window_wheel_default_action.html changes, r?jmaher for the specialpowersAPI.js changes.
Attachment #8728707 - Flags: review?(masayuki)
Attachment #8728707 - Flags: review?(jmaher)
Attachment #8728707 - Flags: review?(masayuki) → review+
Comment on attachment 8728707 [details] [diff] [review]
Fix onZoomReset() in test_wheel_default_action.html to work with e10s.

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

thanks for fixing this up!
Attachment #8728707 - Flags: review?(jmaher) → review+
Blocks: e10s-tests
tracking-e10s: --- → +
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/3e48de1a3b6c
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in before you can comment on or make changes to this bug.