Make test_bug593174.html to pass on Fission
Categories
(Core :: DOM: Navigation, task, P3)
Tracking
()
Fission Milestone | M6c |
People
(Reporter: smaug, Assigned: mbrodesser-Igalia)
References
Details
(Whiteboard: waiting on session history)
Attachments
(1 obsolete file)
Comment 1•4 years ago
|
||
Session history blocks our Nightly experiment so tracking session history test failures for Fission Nightly milestone M6b.
Updated•4 years ago
|
Reporter | ||
Updated•4 years ago
|
Assignee | ||
Comment 2•4 years ago
|
||
Passes locally with ./mach mochitest --setpref="fission.sessionHistoryInParent=true" --enable-xorigin-tests --enable-fission dom/tests/mochitest/bugs/test_bug593174.html
. Fails without fission.sessionHistoryInParent=true
. Hence, it should be tested on Try, once bug 1656208 is fixed.
Comment 3•4 years ago
|
||
Hi Neha, this blocks bug 1652554 which is marked M6b. Should we detach it from there?
Assignee | ||
Updated•4 years ago
|
Updated•4 years ago
|
Comment 4•4 years ago
|
||
(In reply to Jens Stutte [:jstutte] (REO for FF 81) from comment #3)
Hi Neha, this blocks bug 1652554 which is marked M6b. Should we detach it from there?
No, I'll move the meta to M6c to better reflect many of the dependency bugs. Each test should be reviewed in light of the failure cause:
- If it's a broken feature/functionality, fix in M6b (unless it can be argued that it isn't widely used and is acceptable to be broken for some time during the nightly experiment)
- If it's a broken test i.e. test itself needs changes to fix incorrect assumptions that were accidentally met before but are no longer true with fission, or timing/delay changes, then fix in M6c.
Assignee | ||
Comment 5•4 years ago
|
||
I'm waiting for bug 1656208 to be fixed, hence not actively working on this, hence setting Status to "NEW", but keeping it assigned to me.
Assignee | ||
Comment 6•4 years ago
|
||
Requries session history in parent (bug 1656208).
Comment 7•4 years ago
|
||
Mirko has already verified that the test passes with session history pref so enabling the test itself is a trivial task that can be moved to M6c. Nothing to fix here.
Comment 8•4 years ago
|
||
There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:mbrodesser, could you have a look please?
For more information, please visit auto_nag documentation.
Assignee | ||
Updated•4 years ago
|
Comment 9•4 years ago
|
||
Mirko, sessionHistoryInParent is now enabled with Fission (in bug 1668357) so you can re-enable this (and the other tests that were waiting for SHIP) for Fission now. Thanks!
Assignee | ||
Comment 10•4 years ago
|
||
:neha: thanks for letting me know. Then bug 1656208 should be closed, or?
Updated•4 years ago
|
Assignee | ||
Comment 11•4 years ago
|
||
Already enabled in bug 1668357.
Updated•4 years ago
|
Description
•