762263 - Add EGLSync

Status: RESOLVED FIXED

(Core :: Graphics: CanvasWebGL, defect)

Description

[Kelsey Gilbert [:jgilbert]]

We need EGLSync for sychronizing between contexts without 'sharing' them, particularly for EGLImage work.

Comment 1

[Kelsey Gilbert [:jgilbert]]

Attachment: patch

Add EGLSync support.

Comment 2

[Benoit Jacob [:bjacob] (mostly away)]

Comment on attachment 630753 [details] [diff] [review]
patch

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

::: gfx/gl/GLDefs.h
@@ +3261,5 @@
> +#define LOCAL_EGL_SIGNALED                    0x30F2
> +#define LOCAL_EGL_UNSIGNALED                  0x30F3
> +#define LOCAL_EGL_SYNC_PRIOR_COMMANDS_COMPLETE  0x30F0
> +#define LOCAL_EGL_SYNC_FLUSH_COMMANDS_BIT     0x0001
> +#define LOCAL_EGL_FOREVER                     0xFFFFFFFFFFFFFFFFull

Is this a swear word?

Comment 3

[Kelsey Gilbert [:jgilbert]]

I hate these long constants. I am always suspicious they have the wrong number of Fs.

Comment 4

[Kelsey Gilbert [:jgilbert]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/26a6e89c1efa

Comment 5

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/26a6e89c1efa