Closed Bug 1630006 Opened 4 months ago Closed 4 months ago

[wpt-sync] Sync PR 22942 - Get rid of SyntaxErrors in PerformanceObserver

Categories

(Core :: Performance, task, P4)

task

Tracking

()

RESOLVED FIXED
mozilla77
Tracking Status
firefox77 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Nicolás Peña Moreno <npm@chromium.org> wrote:

Get rid of SyntaxErrors in PerformanceObserver

This CL changes the SyntaxErrors to TypeErrors when 'type' and
'entryTypes' are both present or not present. It also adds a UseCounter
for when 'entryTypes' is present and 'buffered', the only other member
in the dictionary, is also present. Currently this causes a console
warning whereas it should instead also throw a TypeError, so once this
mistake is proved to be very infrequent, we can proceed to change it to
TypeError. See related PR:

https://github.com/w3c/performance-timeline/pull/164

Change-Id: Ic9399bf54895e3723b0ca2e513161a7e0638b681

Reviewed-on: https://chromium-review.googlesource.com/2144686
WPT-Export-Revision: dd219adb9e79660a2a44971e35972edcc1d1cfa0

Component: web-platform-tests → Performance
Product: Testing → Core
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fbb586b5a4b0
[wpt PR 22942] - Get rid of SyntaxErrors in PerformanceObserver, a=testonly
https://hg.mozilla.org/integration/autoland/rev/da8dc9caa5eb
[wpt PR 22942] - Update wpt metadata, a=testonly
Test result changes from PR not available.
Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla77
You need to log in before you can comment on or make changes to this bug.