Bug 311303
| Summary: | [Site Isolation] Extra result for http/tests/security/file-system-access-via-dataTransfer.html | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Anthony Tarbinian <a.tarbinian> |
| Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Anthony Tarbinian
With Site Isolation enabled http/tests/security/file-system-access-via-dataTransfer.html
prints out 4 instances of "Should not receive a file" while the -expected.txt only has 3
logs of that message https://github.com/WebKit/WebKit/blob/b3d35bb8ab6482329d4d5e6c4737548156a49279/LayoutTests/http/tests/security/file-system-access-via-dataTransfer-expected.txt
When running the test with site isolation disabled and without the "finishJSTest" calls and letting it timeout,
it does indeed print the message 4 times. I believe that the last message being printed is suppressed and
enabling site isolation changes some timing that allows the 4th one to be observed.
Here is the diff with site isolation enabled vs. the current -expected.txt. Note how it gets printed
a 4th time, but after "TEST COMPLETE" hinting that the message is logged asynchronously after
"finishJSTest" is called.
```
Test that accessing local file system metadata is not allowed
On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
PASS Should not receive file
PASS Should not receive file
PASS Should not receive file
PASS successfullyParsed is true
TEST COMPLETE
+ PASS Should not receive file
```
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/173894915>
Anthony Tarbinian
Pull request: https://github.com/WebKit/WebKit/pull/61865
EWS
Committed 310534@main (5b1ce0d64d86): <https://commits.webkit.org/310534@main>
Reviewed commits have been landed. Closing PR #61865 and removing active labels.