Closed Bug 1580070 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 18951 - Initial Storage Access API IDL Changes

Categories

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

task

Tracking

(firefox71 fixed)

RESOLVED FIXED
mozilla71
Tracking Status
firefox71 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Brandon Maslen <brandm@microsoft.com> wrote:

Initial Storage Access API IDL Changes

This change adds the initial surface area for the methods and sandbox token exposed by the Storage Access API behind the storageAccessAPI runtime flag. Initially the promises created will return simple/immediate resolutions or rejections. Future changes will update the logic of each method to ensure it is functioning correctly. Additionally Web Platform Tests have been added to validate the added behaviour.

As the newly added tests rely on running tests within iFrames an exposed bug in the content_shell testrunner JS has been fixed. The change ensures that iframes will not complete the testharness and that only the main frame will trigger completion.

Bug: 989663
Change-Id: I2388fbc25ceb95c49435aa986191b0aca925d7d5

Reviewed-on: https://chromium-review.googlesource.com/1790583
WPT-Export-Revision: 13e767e8ea930a93cddd7bbf24523cb074829b2d

Ran 4 tests and 9 subtests
OK     : 2
PASS   : 9[linux64-asan-opt,linux64-debug,linux64-opt,linux64-qr-debug,linux64-qr-opt,windows10-64-debug,windows10-64-opt,windows10-64-qr-debug,windows10-64-qr-opt,windows7-32-debug,windows7-32-opt], 3[android-em-7.0-x86_64-debug-geckoview,android-em-7.0-x86_64-opt-geckoview]
FAIL   : 6[android-em-7.0-x86_64-debug-geckoview,android-em-7.0-x86_64-opt-geckoview]
TIMEOUT: 2

New tests that have failures or other problems:
/storage-access-api/hasStorageAccess.sub.window.html: TIMEOUT
    [top-level-context] document.hasStorageAccess() should be allowed by default: true: FAIL[android-em-7.0-x86_64-debug-geckoview,android-em-7.0-x86_64-opt-geckoview]
    [top-level-context] document.hasStorageAccess() should be supported on the document interface: FAIL[android-em-7.0-x86_64-debug-geckoview,android-em-7.0-x86_64-opt-geckoview]
/storage-access-api/idl.window.html
    Document interface: operation hasStorageAccess(): FAIL[android-em-7.0-x86_64-debug-geckoview,android-em-7.0-x86_64-opt-geckoview]
    Document interface: operation requestStorageAccess(): FAIL[android-em-7.0-x86_64-debug-geckoview,android-em-7.0-x86_64-opt-geckoview]
/storage-access-api/requestStorageAccess.sub.window.html: TIMEOUT
    [top-level-context] document.requestStorageAccess() should be rejected by default with no user gesture: FAIL[android-em-7.0-x86_64-debug-geckoview,android-em-7.0-x86_64-opt-geckoview]
    [top-level-context] document.requestStorageAccess() should be supported on the document interface: FAIL[android-em-7.0-x86_64-debug-geckoview,android-em-7.0-x86_64-opt-geckoview]
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/08ee58a93f29
[wpt PR 18951] - Initial Storage Access API IDL Changes, a=testonly
https://hg.mozilla.org/integration/mozilla-inbound/rev/2db62af950f5
[wpt PR 18951] - Update wpt metadata, a=testonly
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
You need to log in before you can comment on or make changes to this bug.