Closed Bug 1048719 Opened 10 years ago Closed 9 years ago

WebGL2 - Implement WebGLQuery

Categories

(Core :: Graphics: CanvasWebGL, defect)

Product:
Core
Component:
Graphics: CanvasWebGL
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla36

People

(Reporter: u480271, Assigned: u480271)

References

(
URL
)

Details

(Keywords: dev-doc-complete)

Attachments

(2 files)

[WebGL2] Query objects
33.75 KB, patch
jgilbert
: review+
Details | Diff | Splinter Review
[WebGL2] remove WebGLQueryRefPtr typedef
2.91 KB, patch
jgilbert
: review+
Details | Diff | Splinter Review 
Implement functions and types to support WebGLQuery object.
Moved the previous implementation to keep the theme.
Attachment #8527390 - Flags: review?(jgilbert)
Comment on attachment 8527390 [details] [diff] [review]
[WebGL2] Query objects

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

::: dom/canvas/WebGL2ContextQueries.cpp
@@ +8,1 @@
>  #include "GLContext.h"

Alphabetical, with parent header on top.

@@ +24,5 @@
> +GetQueryTargetEnumString(GLenum target)
> +{
> +    switch (target)
> +    {
> +        case LOCAL_GL_ANY_SAMPLES_PASSED:

I'm assuming this file is basically just moved code.

::: dom/canvas/WebGLQuery.cpp
@@ +28,5 @@
>      mContext->gl->fGenQueries(1, &mGLName);
>  }
>  
> +void
> +WebGLQuery::Delete() {

{ on next line.
Attachment #8527390 - Flags: review?(jgilbert) → review+
(In reply to Jeff Gilbert [:jgilbert] from comment #2)
> I'm assuming this file is basically just moved code.

Exactly what I mean by comment #1
https://hg.mozilla.org/mozilla-central/rev/e5bccb2e00cb
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36

        Attachment #8528785 -
        Flags: review?(jgilbert)

    
    

    
  
This slipped through.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---

    
    

    
  
Comment on attachment 8528785 [details] [diff] [review]
[WebGL2] remove WebGLQueryRefPtr typedef

You seem to have color codes in your diff here. The changes are r+ though.

        Attachment #8528785 -
        Flags: review?(jgilbert) → review+

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/af5fc587f98b
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → FIXED

          You need to log in
          before you can comment on or make changes to this bug.