[webkit-reviews] review denied: [Bug 67832] [chromium] Add GraphicsContext3DPrivate:createGraphicsContextForAnotherThread : [Attachment 106877] now with comments.

[bugzilla-daemon at webkit.org]

Kenneth Russell <kbr at google.com> has denied Nat Duca <nduca at chromium.org>'s
request for review:
Bug 67832: [chromium] Add
GraphicsContext3DPrivate:createGraphicsContextForAnotherThread
https://bugs.webkit.org/show_bug.cgi?id=67832

Attachment 106877: now with comments.
https://bugs.webkit.org/attachment.cgi?id=106877&action=review

------- Additional Comments from Kenneth Russell <kbr at google.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=106877&action=review


Overall this looks good but one small thing needs cleanup before landing.

> Source/WebKit/chromium/src/GraphicsContext3DChromium.cpp:122
> +PassRefPtr<GraphicsContext3D>
GraphicsContext3DPrivate::createGraphicsContextFromWebContext(PassOwnPtr<WebKit
::WebGraphicsContext3D> webContext, GraphicsContext3D::Attributes attrs,
HostWindow* hostWindow, GraphicsContext3D::RenderStyle renderStyle, bool
forUseOnAnotherThread)

WebKit style strongly discourages the use of bool arguments because they aren't
self-describing at the call site. Please define a little enum in
GraphicsContext3DPrivate.h like ThreadUsage { ForUseOnThisThread,
ForUseOnAnotherThread } and pass that instead.