Closed Bug 1557423 Opened 6 years ago Closed 6 years ago

[wpt-sync] Sync PR 17208 - [UserTimingL3] Replace SyntaxError with TypeError.

Categories

(Core :: Performance: General, defect, P4)

defect

Tracking

()

RESOLVED FIXED
mozilla70
Tracking Status
firefox70 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync upstream error])

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

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

Tom McKee <tommckee@chromium.org> wrote:

[UserTimingL3] Replace SyntaxError with TypeError.

The User Timing L3 specification requires TypeErrors to be thrown when
calls to performance.measure() are malformed. The User Timing L2
specification used SyntaxErrors for similar error conditions.

This CL changes our use of SyntaxError to TypeError when L3 conformance
is active.

Bug: 953960
Change-Id: I87bb0717687f4a3b149eaeb6fc190fa9a98d1f9b
Reviewed-on: https://chromium-review.googlesource.com/1647342
WPT-Export-Revision: 7b5ab7cef1efb6f0a75aa6614a663beecb0fba5c

PR 17208 applied with additional changes from upstream: b09bca899835b03d0f90e2903941aca13223f075
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Component: web-platform-tests → Performance
Product: Testing → Core
Failed to get results from try push
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/df780d24c910 [wpt PR 17208] - [UserTimingL3] Replace SyntaxError with TypeError., a=testonly
Whiteboard: [wptsync downstream] → [wptsync upstream error]
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
You need to log in before you can comment on or make changes to this bug.