Closed Bug 1250388 Opened 8 years ago Closed 8 years ago

Enable some tests in dom/html/test/ that work with e10s

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla47
Tracking Status
e10s + ---
firefox47 --- fixed

People

(Reporter: mccr8, Assigned: mccr8)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

test_iframe_sandbox_popups_inheritance.html and test_bug741266.html seem to work now. Looks like the latter works because I fixed it in bug 1229117 but forgot to actually enable it.

try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=7996ab5e81e6

test_bug369370.html works for me locally on OSX, but fails on L64 on try:
TEST-UNEXPECTED-FAIL | dom/html/test/test_bug369370.html | Checking scrollLeft - got 413, expected 400
TEST-UNEXPECTED-FAIL | dom/html/test/test_bug369370.html | Checking scrollTop - got 313, expected 300
TEST-UNEXPECTED-FAIL | dom/html/test/test_bug369370.html | Image is vertically centered - got 18.5, expected 25
So I'll leave that disabled.
bz gave me a blanket rs to enable tests that work without changes.
Attachment #8722345 - Flags: review+
Blocks: e10s-tests
tracking-e10s: --- → +
https://hg.mozilla.org/mozilla-central/rev/88cbc2c8cea6
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: