[webkit-reviews] review granted: [Bug 183895] [TexMap] Make TextureMapperContextAttributes thread-specific : [Attachment 336274] Patch

[bugzilla-daemon at webkit.org]

Carlos Garcia Campos <cgarcia at igalia.com> has granted Zan Dobersek
<zan at falconsigh.net>'s request for review:
Bug 183895: [TexMap] Make TextureMapperContextAttributes thread-specific
https://bugs.webkit.org/show_bug.cgi?id=183895

Attachment 336274: Patch

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




--- Comment #2 from Carlos Garcia Campos <cgarcia at igalia.com> ---
Comment on attachment 336274
  --> https://bugs.webkit.org/attachment.cgi?id=336274
Patch

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

> Source/WebCore/platform/graphics/texmap/TextureMapperContextAttributes.cpp:42
> +    static WTF::ThreadSpecific<TextureMapperContextAttributes>*
s_textureMapperContextAttributes;
> +    if (!s_textureMapperContextAttributes)
> +	   s_textureMapperContextAttributes = new
WTF::ThreadSpecific<TextureMapperContextAttributes>;
> +    return *s_textureMapperContextAttributes;

Should we use std::once here?