Closed
Bug 1159378
Opened 9 years ago
Closed 9 years ago
Set the security info on the service worker cache entries cached from the network
Categories
(Core :: DOM: Workers, defect)
Core
DOM: Workers
Tracking
()
RESOLVED
FIXED
mozilla40
Tracking | Status | |
---|---|---|
firefox40 | --- | fixed |
People
(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)
References
Details
Attachments
(4 files)
6.00 KB,
patch
|
mcmanus
:
review+
|
Details | Diff | Splinter Review |
5.41 KB,
patch
|
nsm
:
review+
|
Details | Diff | Splinter Review |
10.92 KB,
patch
|
nsm
:
review+
|
Details | Diff | Splinter Review |
1.82 KB,
patch
|
nsm
:
review+
|
Details | Diff | Splinter Review |
This is about making https fetches work for such service worker scripts, IOW, doing what bug 1156847 did for the current code to the code being added in bug 1154494.
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8598872 -
Flags: review?(mcmanus)
Assignee | ||
Comment 2•9 years ago
|
||
Attachment #8598873 -
Flags: review?(nsm.nikhil)
Assignee | ||
Comment 3•9 years ago
|
||
This helper script will allow us to run the same test across both HTTP and HTTPS origins. This patch also uses postMessage instead of a direct function call on the parent window, because in the HTTPS case, that window will be cross-origin. It also moves the code to trigger the update and get the corresponding updatefound event to wait_for_update.html, since getting the service worker registration cross-origin is not possible.
Attachment #8598874 -
Flags: review?(nsm.nikhil)
Assignee | ||
Comment 4•9 years ago
|
||
The reason why this test is interesting here is that it tests the code path added in the part 2 of this bug.
Attachment #8598882 -
Flags: review?(nsm.nikhil)
Updated•9 years ago
|
Attachment #8598872 -
Flags: review?(mcmanus) → review+
Attachment #8598873 -
Flags: review?(nsm.nikhil) → review+
Attachment #8598874 -
Flags: review?(nsm.nikhil) → review+
Attachment #8598882 -
Flags: review?(nsm.nikhil) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/47947bd5ebed https://hg.mozilla.org/integration/mozilla-inbound/rev/c532540057dd https://hg.mozilla.org/integration/mozilla-inbound/rev/482c23ebcf04 https://hg.mozilla.org/integration/mozilla-inbound/rev/ca507947df16
https://hg.mozilla.org/mozilla-central/rev/47947bd5ebed https://hg.mozilla.org/mozilla-central/rev/c532540057dd https://hg.mozilla.org/mozilla-central/rev/482c23ebcf04 https://hg.mozilla.org/mozilla-central/rev/ca507947df16
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox40:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in
before you can comment on or make changes to this bug.
Description
•