Closed Bug 1550375 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 16433 - [Native File System] Change at what level we hook into FS backend.

Categories

(Testing :: web-platform-tests, defect, P4)

defect

Tracking

(firefox69 fixed)

RESOLVED FIXED
mozilla69
Tracking Status
firefox69 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 16433 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/16433
Details from upstream follow.

Marijn Kruisselbrink <mek@chromium.org> wrote:

[Native File System] Change at what level we hook into FS backend.

Before this CL the Native File System API hooked into the old FS api
from within blink. This makes implementing several of the features we want
to support in the new API very difficult, so this CL changes things so
the integration point now is in the browser process.

As part of doing this, code specific to the new Native File System API is
also moved to its own directories in content and blink.

Also get rid of getParent() and update tests to not rely on getParent()
as that method is not part of the planned API surface.

 implement a new feature.

Bug: 853326
Binary-Size: Increase is due to large size of mojo bindings needed to
Change-Id: Ia2702f83b6c80bb4f2d014b48e251814677a59bc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1576103
Reviewed-by: Avi Drissman \<avi@chromium.org>
Reviewed-by: Daniel Cheng \<dcheng@chromium.org>
Reviewed-by: Victor Costan \<pwnall@chromium.org>
Commit-Queue: Marijn Kruisselbrink \<mek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657387}

PR 16433 applied with additional changes from upstream: b46cd5fba9471569b3338cca2b54e3c42294a01f
Failed to get results from try push
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/bc293cf1bb44
[wpt PR 16433] - [Native File System] Change at what level we hook into FS backend., a=testonly
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
You need to log in before you can comment on or make changes to this bug.