Summary: | [TextureMapper] Remove TextureMapperGLHeaders.h | ||
---|---|---|---|
Product: | WebKit | Reporter: | Carlos Garcia Campos <cgarcia> |
Component: | Platform | Assignee: | Carlos Garcia Campos <cgarcia> |
Status: | NEW --- | ||
Severity: | Normal | CC: | alex, Hironori.Fujii, webkit-bug-importer |
Priority: | P2 | Keywords: | Gtk, InRadar |
Version: | WebKit Nightly Build | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=175313 |
Description
Carlos Garcia Campos
2023-11-28 01:44:50 PST
Pull request: https://github.com/WebKit/WebKit/pull/20983 Grouping the includes in a single header is not the problem, it just makes easier to make a mistake and include the common header from another header. For example, in angle we have ANGLEHeaders.h and it's not a problem as long as it's only included from cpp files and not unified with others, see the comment in the header: // Note: this file can't be compiled in the same unified source file // as others which include the system's OpenGL headers. I can keep the common header if you prefer, but we need to make sure it's never included in headers and cpp files including it are not unified. |