[webkit-reviews] review granted: [Bug 216310] WebGL2RenderingContext should use Graphics Context types : [Attachment 408327] Patch

[bugzilla-daemon at webkit.org]

Kenneth Russell <kbr at google.com> has granted Don Olmstead
<don.olmstead at sony.com>'s request for review:
Bug 216310: WebGL2RenderingContext should use Graphics Context types
https://bugs.webkit.org/show_bug.cgi?id=216310

Attachment 408327: Patch

https://bugs.webkit.org/attachment.cgi?id=408327&action=review




--- Comment #2 from Kenneth Russell <kbr at google.com> ---
Comment on attachment 408327
  --> https://bugs.webkit.org/attachment.cgi?id=408327
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=408327&action=review

Thanks for fixing this build! r+ with one comment.

> Source/WebCore/html/canvas/WebGL2RenderingContext.cpp:2109
> +	   return GraphicsContextGL::WAIT_FAILED_WIN;

We could consider naming this WAIT_FAILED_WEBGL in all cases.