Open
Bug 1218227
Opened 9 years ago
Updated 2 years ago
Make fetch-request-css-base-url-worker.js pass
Categories
(Core :: DOM: Service Workers, defect, P3)
Core
DOM: Service Workers
Tracking
()
NEW
People
(Reporter: ehsan.akhgari, Unassigned)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
1.90 KB,
patch
|
bkelly
:
review-
|
Details | Diff | Splinter Review |
This test seems to have a bug in it that is causing it to fail!
Reporter | ||
Comment 1•9 years ago
|
||
Attachment #8678620 -
Flags: review?(bkelly)
Comment 2•9 years ago
|
||
This doesn't look right. The point of the test is that the request for the CSS file is rewritten to use the other origin, right?
Comment 3•9 years ago
|
||
Comment on attachment 8678620 [details] [diff] [review] Make fetch-request-css-base-url-worker.js pass Review of attachment 8678620 [details] [diff] [review]: ----------------------------------------------------------------- I agree with Josh, this is not right. Whats happening here is our opaque tainting fake redirect is changing the stylesheet URL. We need bug 1201160 to fix this (as noted in the wpt ini file).
Attachment #8678620 -
Flags: review?(bkelly) → review-
Reporter | ||
Comment 4•9 years ago
|
||
Oh, right, sorry I was misreading the test!
Updated•7 years ago
|
Comment 5•7 years ago
|
||
I think the correct fix here is to implement bug 1222008. We will then propagate back the final URL correctly. Bug 1201160 would instead make us not get intercepted at all, but I don't think that's what chrome is doing today either.
Updated•6 years ago
|
Priority: -- → P3
Updated•6 years ago
|
Assignee: ehsan → nobody
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•