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)
Core
Performance: General
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 toperformance.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
| Assignee | ||
Comment 1•6 years ago
|
||
PR 17208 applied with additional changes from upstream: b09bca899835b03d0f90e2903941aca13223f075
| Assignee | ||
Updated•6 years ago
|
Whiteboard: [wptsync downstream] → [wptsync downstream error]
| Assignee | ||
Updated•6 years ago
|
Whiteboard: [wptsync downstream error] → [wptsync downstream]
| Assignee | ||
Comment 2•6 years ago
|
||
| Assignee | ||
Comment 3•6 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=cd1e5af3adfbcea3ff2d741f5ff8138b7b949669
| Assignee | ||
Updated•6 years ago
|
Component: web-platform-tests → Performance
Product: Testing → Core
| Assignee | ||
Comment 4•6 years ago
|
||
| Assignee | ||
Comment 5•6 years ago
|
||
Failed to get results from try push
| Assignee | ||
Comment 6•6 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=f740aa611c0f9884b2eeb349e3c90ac513f89a4e
| Assignee | ||
Comment 7•6 years ago
|
||
| Assignee | ||
Comment 8•6 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=66e97df1a51da1a771e84715f905986bdea164c7
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/df780d24c910
[wpt PR 17208] - [UserTimingL3] Replace SyntaxError with TypeError., a=testonly
| Assignee | ||
Updated•6 years ago
|
Whiteboard: [wptsync downstream] → [wptsync upstream error]
Comment 10•6 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox70:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
You need to log in
before you can comment on or make changes to this bug.
Description
•