Closed
Bug 1159378
Opened 10 years ago
Closed 10 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•10 years ago
|
||
Attachment #8598872 -
Flags: review?(mcmanus)
Assignee | ||
Comment 2•10 years ago
|
||
Attachment #8598873 -
Flags: review?(nsm.nikhil)
Assignee | ||
Comment 3•10 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•10 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•10 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/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: 10 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
•