WebGL2 - Implement WebGLSync

RESOLVED FIXED in Firefox 39

Status

()

defect
RESOLVED FIXED
5 years ago
3 years ago

People

(Reporter: kamidphish, Assigned: kamidphish)

Tracking

({dev-doc-complete})

unspecified
mozilla39
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox39 fixed)

Details

()

Attachments

(2 attachments)

Implement functions and types to support WebGLSync object.
Comment on attachment 8579685 [details] [diff] [review]
Implement WebGL2Sync.

Review of attachment 8579685 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/canvas/WebGLSync.h
@@ +20,5 @@
>  {
>      friend class WebGL2Context;
>  
>  public:
> +    explicit WebGLSync(WebGLContext* webgl, GLsync name);

I'd rather make this RAII, instead of passing some externally-created GLsync in. (Also, `explicit` is only for single-arg ctors, IIRC)
Attachment #8579685 - Flags: review?(jgilbert) → review+
(In reply to Jeff Gilbert [:jgilbert] from comment #4)
> > +    explicit WebGLSync(WebGLContext* webgl, GLsync name);
> 
> I'd rather make this RAII, instead of passing some externally-created GLsync
> in. (Also, `explicit` is only for single-arg ctors, IIRC)

We can do that. I think I changed to this why because of how we pass in the glname for other objects, such as textures, render buffers, frame buffers, etc.

Good spot on explicit. This function used to be single-arg ctor.
https://hg.mozilla.org/mozilla-central/rev/f5957afe1f8d
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
You need to log in before you can comment on or make changes to this bug.