1826886 - The "name" of an OPFS root should be the empty string

Status: RESOLVED FIXED

(Core :: DOM: File, defect)

Description

[Austin Sullivan]

Steps to reproduce:

let opfsRoot = await navigator.storage.getDirectory();
console.log(opfsRoot.name);

Actual results:

logs "root"

Expected results:

logs the empty string

See step 3.2 of https://fs.spec.whatwg.org/#dom-storagemanager-getdirectory

Tested on Firefox 111 - haven't tested more recent builds

Confirmed that stable Chrome and Safari both use the empty string

Comment 1

[Kagami Rosylight [:saschanaz] (they/them)]

Confirmed. Jari, could you take a look?

Comment 2

[Jari Jalkanen]

Attachment: Bug 1826886 - Use empty OPFS root name. r=#dom-storage

Comment 3

[Jari Jalkanen]

Attachment: Bug 1826886 - Check that OPFS root name is empty. r=#dom-storage

The test checks that name is empty also after deserialization.

Depends on D175221

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/85cb12cef6b7
Use empty OPFS root name. r=dom-storage-reviewers,janv
https://hg.mozilla.org/integration/autoland/rev/4bd6beb5ccc4
Check that OPFS root name is empty. r=dom-storage-reviewers,janv

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/39491 for changes under testing/web-platform/tests

Comment 6

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/85cb12cef6b7
https://hg.mozilla.org/mozilla-central/rev/4bd6beb5ccc4

Comment 7

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot