General cleanup. No behavior change.
Created attachment 371724 [details]
*** Bug 198705 has been marked as a duplicate of this bug. ***
Changing to P1 since it blocks 198704.
Created attachment 372022 [details]
Created attachment 372066 [details]
Comment on attachment 372066 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=372066&action=review
r=me with just a nitpick.
> + auto textureTypeIndex = std::find(m_textureTypeNames, m_textureTypeNames + WTF_ARRAY_LENGTH(m_textureTypeNames), nativeTypeDeclaration.name()) - m_textureTypeNames;
I understand that is more concise, but I am not entirely comfortable with replacing some reasonable code by pointer arithmetic (the - m_textureTypeNames at the end).
It gets me especially wary because of the auto. Is this std::ptrdiff_t ?uintptr_t ?
Committed r246428: <https://trac.webkit.org/changeset/246428>