Closed
Bug 1270509
Opened 8 years ago
Closed 8 years ago
Enable some tests on e10s which seem to work fine
Categories
(Core :: DOM: Content Processes, defect)
Core
DOM: Content Processes
Tracking
()
RESOLVED
FIXED
mozilla49
People
(Reporter: nika, Assigned: nika)
References
(Blocks 1 open bug)
Details
Attachments
(3 files)
1018 bytes,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
940 bytes,
patch
|
enndeakin
:
review+
|
Details | Diff | Splinter Review |
726 bytes,
patch
|
bzbarsky
:
review+
|
Details | Diff | Splinter Review |
These are a few tests which are currently disabled on e10s, which (at least seem) to work fine when run in e10s.
Assignee | ||
Comment 1•8 years ago
|
||
Attachment #8749193 -
Flags: review?(bugs)
Assignee | ||
Comment 2•8 years ago
|
||
Attachment #8749194 -
Flags: review?(enndeakin)
Assignee | ||
Comment 3•8 years ago
|
||
Attachment #8749195 -
Flags: review?(enndeakin)
Updated•8 years ago
|
Blocks: e10s-tests
tracking-e10s:
--- → +
Updated•8 years ago
|
Attachment #8749194 -
Flags: review?(enndeakin) → review+
Comment 4•8 years ago
|
||
Comment on attachment 8749195 [details] [diff] [review] Enable browser_xhr_sandbox.js on e10s It looks like this test creates a non-remote content iframe in the hidden window then does some sandbox related checking on it. I presume we also want to test this for remote frames. I'm not the right person to review what should be done here. Perhaps one of the test author or reviewers?
Updated•8 years ago
|
Attachment #8749195 -
Flags: review?(enndeakin)
Assignee | ||
Comment 5•8 years ago
|
||
Comment on attachment 8749195 [details] [diff] [review] Enable browser_xhr_sandbox.js on e10s It looks like bz has reviewed changes to this test before. r?-ing him.
Attachment #8749195 -
Flags: review?(bzbarsky)
Comment 6•8 years ago
|
||
Comment on attachment 8749195 [details] [diff] [review] Enable browser_xhr_sandbox.js on e10s r=me if it's green on try
Attachment #8749195 -
Flags: review?(bzbarsky) → review+
Comment 7•8 years ago
|
||
Comment on attachment 8749193 [details] [diff] [review] Enable test_bug482935.html on e10s Assuming the test passes on e10s, r+. Give it couple of try runs.
Attachment #8749193 -
Flags: review?(bugs) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/33bc6f261449 https://hg.mozilla.org/integration/mozilla-inbound/rev/75dd87c07444 https://hg.mozilla.org/integration/mozilla-inbound/rev/200733834a6e
Comment 9•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/33bc6f261449 https://hg.mozilla.org/mozilla-central/rev/75dd87c07444 https://hg.mozilla.org/mozilla-central/rev/200733834a6e
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox49:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in
before you can comment on or make changes to this bug.
Description
•