[wpt-sync] Sync PR 30261 - FS: Add check for removal of sandbox root dir
Categories
(Testing :: web-platform-tests, task, P4)
Tracking
(firefox94 fixed)
Tracking | Status | |
---|---|---|
firefox94 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 30261 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/30261
Details from upstream follow.
b'Austin Sullivan <asully@chromium.org>' wrote:
FS: Add check for removal of sandbox root dir
The FileSystemHandle::Remove() method was recently added to allow for
the removal of files and directories from the underlying file system.This method now fails when attempting to delete the root of a sandbox
file system.Bug: 1242231
Change-Id: Ibe3ed59b79678ce168d0803c6607abd8ac06594c
Reviewed-on: https://chromium-review.googlesource.com/3123735
WPT-Export-Revision: fdad169e64324495deb28f89090cdf20e2b02a93
Assignee | ||
Comment 1•3 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=0d5e69b70d94bef990cc5e792e1b9c48131ae499
Assignee | ||
Comment 2•3 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=faf8aa8ac72b7a4de50366ed08b13adda0548efe
Assignee | ||
Comment 3•3 years ago
|
||
CI Results
Ran 11 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI
Total 2 tests and 8 subtests
Status Summary
Firefox
OK : 2
FAIL: 16
Chrome
OK : 2
PASS: 14
FAIL: 2
Safari
OK : 2
FAIL: 16
Links
Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base
Details
New Tests That Don't Pass
/file-system-access/sandboxed_FileSystemBaseHandle-remove.https.any.html
remove() to remove a file: FAIL (Chrome: PASS, Safari: FAIL)
remove() on an already removed file should fail: FAIL (Chrome: PASS, Safari: FAIL)
remove() to remove an empty directory: FAIL (Chrome: PASS, Safari: FAIL)
remove() on an already removed directory should fail: FAIL (Chrome: PASS, Safari: FAIL)
remove() on a non-empty directory should fail: FAIL (Chrome: PASS, Safari: FAIL)
remove() on a directory recursively should delete all sub-items: FAIL (Chrome: PASS, Safari: FAIL)
remove() on a file should ignore the recursive option: FAIL (Chrome: PASS, Safari: FAIL)
cannot remove the root of a sandbox file system: FAIL (Chrome: FAIL, Safari: FAIL)
/file-system-access/sandboxed_FileSystemBaseHandle-remove.https.any.worker.html
remove() to remove a file: FAIL (Chrome: PASS, Safari: FAIL)
remove() on an already removed file should fail: FAIL (Chrome: PASS, Safari: FAIL)
remove() to remove an empty directory: FAIL (Chrome: PASS, Safari: FAIL)
remove() on an already removed directory should fail: FAIL (Chrome: PASS, Safari: FAIL)
remove() on a non-empty directory should fail: FAIL (Chrome: PASS, Safari: FAIL)
remove() on a directory recursively should delete all sub-items: FAIL (Chrome: PASS, Safari: FAIL)
remove() on a file should ignore the recursive option: FAIL (Chrome: PASS, Safari: FAIL)
cannot remove the root of a sandbox file system: FAIL (Chrome: FAIL, Safari: FAIL)
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/70637db68ab5 [wpt PR 30261] - FS: Add check for removal of sandbox root dir, a=testonly https://hg.mozilla.org/integration/autoland/rev/9e9cd760423f [wpt PR 30261] - Update wpt metadata, a=testonly
Comment 5•3 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/70637db68ab5
https://hg.mozilla.org/mozilla-central/rev/9e9cd760423f
Description
•