New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 601729 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

gfx::ScopedActiveTexture and gfx::ScopedTextureBinder are incompatible

Project Member Reported by ccameron@chromium.org, Apr 8 2016

Issue description

gfx::ScopedTextureBinder has an optimization where it restores the state from the ContextState, instead of querying the GL driver.

This backfires when GL_ACTIVE_TEXTURE has changed, because gfx::ScopedTextureBinder will re-bind whatever texture the ContextState thinks was active, which will not be accurate, because it will not be updated gfx::ScopedActiveTexture.

One fix for this is to have gfx::ScopedActiveTexture update ContextState. That seems a little bit tricky, and potentially a bad idea -- my understanding of StateRestorer is that it isn't supposed to modify ContextState.

For the moment, in issue 599314, I'm proposing to just note this bug and use a scoped block runner to reset the state.
 
Components: -Internals>GPU Internals>GPU>Internals
Status: Available (was: Untriaged)
Cc: c.pa...@samsung.com
ccameron@: I had submitted https://codereview.chromium.org/2762833003 to use ContextState restoration for ActiveTexture. Would this help?
Project Member

Comment 3 by sheriffbot@chromium.org, Jun 11 2018

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
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
Owner: ccameron@chromium.org
Status: Assigned (was: Untriaged)
Status: Fixed (was: Assigned)
Probably fixed by #2. If not, then this can be archived and just kept for future reference.

Sign in to add a comment