[wpt-sync] Sync PR 15267 - Add screen orientation test: fire event before promise is resolved

RESOLVED FIXED in Firefox 67

Status

()

enhancement
P4
normal
RESOLVED FIXED
4 months ago
3 months ago

People

(Reporter: wptsync, Unassigned)

Tracking

unspecified
mozilla67
Points:
---

Firefox Tracking Flags

(firefox67 fixed)

Details

(Whiteboard: [wptsync downstream], )

Assignee

Description

4 months ago

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

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

Johanna-hub <jyherman@gmail.com> wrote:

Add screen orientation test: fire event before promise is resolved

For feedback

@marcoscaceres I'm having issues with going to fullscreen and this is causing the first promise to reject and the test to fail at that point. Will look into it more tomorrow!
https://wpt.fyi/results/screen-orientation/event-before-promise.html?label=pr_head&max-count=1&pr=15267

Assignee

Updated

4 months ago
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Assignee

Updated

4 months ago
Component: web-platform-tests → DOM
Product: Testing → Core
Assignee

Updated

4 months ago
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Assignee

Comment 3

4 months ago
Ran 1 tests and 1 subtests
OK     : 1
FAIL   : 1

New tests that have failures or other problems:
/screen-orientation/event-before-promise.html
    The 'change' event must fire before the [[orientationPendingPromise]] is resolved.: FAIL

Comment 4

4 months ago
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4551719aa859
[wpt PR 15267] - Add screen orientation test: fire event before promise is resolved, a=testonly
https://hg.mozilla.org/integration/mozilla-inbound/rev/09ce1b8d9831
[wpt PR 15267] - Update wpt metadata, a=testonly

Comment 5

4 months ago
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5aad8dd39c85
[wpt PR 15267] - Add screen orientation test: fire event before promise is resolved, a=testonly
https://hg.mozilla.org/integration/mozilla-inbound/rev/ad47847d2f63
[wpt PR 15267] - Update wpt metadata, a=testonly

Comment 6

4 months ago
bugherder
Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.