Closed Bug 1848388 (webgpu-idl-extra-newobject) Opened 11 months ago Closed 10 months ago

Remove `NewObject` annotations that are not in WebGPU spec. IDL

Categories

(Core :: Graphics: WebGPU, defect, P3)

defect

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.

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.

Assignee: nobody → egubler
Status: NEW → ASSIGNED

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)
Blocks: webgpu-phase-2
No longer blocks: webgpu-v1
Flags: needinfo?(egubler)
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
Status: ASSIGNED → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → 119 Branch
Flags: needinfo?(egubler)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: