Closed Bug 1734589 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 31150 - Don't require the presence of `document` in `FileAPI/file/File-constructor.any.js`

Categories

(Core :: DOM: File, task, P4)

task

Tracking

()

RESOLVED FIXED
95 Branch
Tracking Status
firefox95 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Andreu Botella <abb@randomunok.com> wrote:

Don't require the presence of document in FileAPI/file/File-constructor.any.js

FileAPI/file/File-constructor.any.js is a test file that will run in both window and worker environments, but it uses document without testing whether it is present. This results in the worker test failing in all engines.

This change only runs the document.body subtest if document is a property of the global scope.

Component: web-platform-tests → DOM: File
Product: Testing → Core

CI Results

Ran 0 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 2 tests and 50 subtests

Status Summary

Firefox

OK : 2
PASS: 100

Chrome

OK : 2
PASS: 100

Safari

OK : 2
PASS: 100

Links

GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e0df1d757f50 [wpt PR 31150] - Don't require the presence of `document` in `FileAPI/file/File-constructor.any.js`, a=testonly https://hg.mozilla.org/integration/autoland/rev/c2a38906ebe2 [wpt PR 31150] - Update wpt metadata, a=testonly
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 95 Branch
You need to log in before you can comment on or make changes to this bug.