[webkit-reviews] review denied: [Bug 219661] Add runtime flag for ANGLE on Metal : [Attachment 415694] Patch

[bugzilla-daemon at webkit.org]

Sam Weinig <sam at webkit.org> has denied Dean Jackson <dino at apple.com>'s request
for review:
Bug 219661: Add runtime flag for ANGLE on Metal
https://bugs.webkit.org/show_bug.cgi?id=219661

Attachment 415694: Patch

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




--- Comment #3 from Sam Weinig <sam at webkit.org> ---
Comment on attachment 415694
  --> https://bugs.webkit.org/attachment.cgi?id=415694
Patch

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

> Source/WebCore/platform/graphics/cocoa/GraphicsContextGLOpenGLCocoa.mm:99
> +    if (RuntimeEnabledFeatures::sharedFeatures().webGLUsingMetal()) {

This is a layering violation. Code in platform/ is not allowed to use
RuntimeEnabledFeatures as it is not a platform concept.

Instead, the check should be done by the WebCore code that creates this object,
and it should pass down this state.