(Copy)TexImage* should not allow internalformat of GL_BGRA8_EXT |
||||||
Issue description
In FormatTypeValidator in texture_manager.cc, an internalformat of GL_BGRA8_EXT is allowed so that it works for TexStorage. However, it also allows BGRA8 for (Copy)TexImage*, which is wrong.
// Exposed by GL_APPLE_texture_format_BGRA8888 for TexStorage*
// TODO(kainino): this actually exposes it for (Copy)TexImage* as well,
// which is incorrect. crbug.com/663086
{GL_BGRA8_EXT, GL_BGRA_EXT, GL_UNSIGNED_BYTE},
,
Nov 8 2017
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Nov 9 2017
No reason AFAIK that this would be fixed.
,
Nov 9
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Nov 9
kbr@: should this be assigned to someone to fix as a priority, or leave as available?
,
Nov 10
We should reinvestigate and fix this but it's not a priority. |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by kainino@chromium.org
, Nov 7 2016