Closed
Bug 1848388
(webgpu-idl-extra-newobject)
Opened 2 years ago
Closed 2 years ago
Remove `NewObject` annotations that are not in WebGPU spec. IDL
Categories
(Core :: Graphics: WebGPU, defect, P3)
Core
Graphics: WebGPU
Tracking
()
RESOLVED
FIXED
119 Branch
| Tracking | Status | |
|---|---|---|
| firefox119 | --- | fixed |
People
(Reporter: ErichDonGubler, Assigned: ErichDonGubler)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
See attached patch.
| Assignee | ||
Comment 1•2 years ago
|
||
NewObject implies constraints that the WebGPU W3C committee
specifically did not want, according to @jgilbert. We still need some
Throws annotations for fallible allocations, i.e., anything that
constructs a Promise, or which has exceptions thrown in cases of
violated constraints. These aren't part of the IDL spec., but they're
necessary for Firefox's implementation because of how our codegen works.
Updated•2 years ago
|
Assignee: nobody → egubler
Status: NEW → ASSIGNED
| Assignee | ||
Comment 2•2 years ago
|
||
Current state of this bug: working with IDL folks to get consensus on the current course of action. Current disposition seems to be to permit this, but naively doing so would require more work on the IDL side to avoid losing safety.
Flags: needinfo?(egubler)
| Assignee | ||
Updated•2 years ago
|
Flags: needinfo?(egubler)
| Assignee | ||
Updated•2 years ago
|
Flags: needinfo?(egubler)
Pushed by egubler@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/16cc1aa2701e
refactor(webgpu): remove `NewObject` IDL to match spec. r=webgpu-reviewers,webidl,edgar,saschanaz,smaug,jgilbert
Comment 4•2 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox119:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 119 Branch
| Assignee | ||
Updated•2 years ago
|
Flags: needinfo?(egubler)
You need to log in
before you can comment on or make changes to this bug.
Description
•