Closed Bug 1581072 Opened 5 years ago Closed 5 years ago

window.opener is null on Fission when the opened window origin is redirect to another one

Categories

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

defect

Tracking

()

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

People

(Reporter: edenchuang, Assigned: edenchuang)

References

Details

Attachments

(1 file, 1 obsolete file)

The window.opener is null when the opened window is redirected to another origin by loading a .sjs file. But expect window.opener should not be null.

following service worker mochitests are timeout since window.opener is null.

dom/serviceworkers/test/test_https_origin_after_redirect.html
dom/serviceworkers/test/test_https_origin_after_redirect_cached.html
dom/serviceworkers/test/test_origin_after_redirect.html
dom/serviceworkers/test/test_origin_after_redirect_cached.html
dom/serviceworkers/test/test_origin_after_redirect_to_https.html
dom/serviceworkers/test/test_origin_after_redirect_to_https_cached.html

However, the fail isn't related to ServiceWorker code. Will attached a test case for it.

Testing the window opener after the opened window origin is redirected by loading a sjs.

Nika, since this is not related to ServiceWorker code, I did not dig it deeper. I guess there is something we did not notice when redirecting a window when Fission on. I think this probably is a known issue for Fission team. Do you have any idea about the fail? Thanks.

Flags: needinfo?(nika)

kmag is working on fixing this right now, redirecting ni

Flags: needinfo?(nika) → needinfo?(kmaglione+bmo)
Status: NEW → RESOLVED
Closed: 5 years ago
Flags: needinfo?(kmaglione+bmo)
Resolution: --- → DUPLICATE
Attachment #9092618 - Attachment is obsolete: true

ServiceWorker mochitests fail for Fission is fixed by bug 1542415.
Remove the skip-if for tests

reopen the bug for removing skip-if in mochitest.ini.

Assignee: nobody → echuang
Status: RESOLVED → REOPENED
Priority: -- → P2
Resolution: DUPLICATE → ---
Keywords: checkin-needed

Pushed by btara@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/64f2c69fdaeb
Removing skip-if = fission for testing fission r=perry

Keywords: checkin-needed
Status: REOPENED → RESOLVED
Closed: 5 years ago5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
Regressions: 1582125

Retroactively moving fixed bugs whose summaries mention "Fission" (or other Fission-related keywords) but are not assigned to a Fission Milestone to an appropriate Fission Milestone.

This will generate a lot of bugmail, so you can filter your bugmail for the following UUID and delete them en masse:

0ee3c76a-bc79-4eb2-8d12-05dc0b68e732

Fission Milestone: --- → M4
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: