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

Issue 787333 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 26
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocked on:
issue 710673



Sign in to add a comment

Missing BindTexture when resizing target texture in PaintCanvasVideoRenderer::CopyVideoFrameSingleTextureToGLTexture

Project Member Reported by vmi...@chromium.org, Nov 21 2017

Issue description

PaintCanvasVideoRenderer::CopyVideoFrameSingleTextureToGLTexture runs TexImage2D without first binding the target texture.  This could result in the clobbering of other textures.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 21 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9300c95d365861c2f0165787bff9fd03e1f6d1ed

commit 9300c95d365861c2f0165787bff9fd03e1f6d1ed
Author: Victor Miura <vmiura@chromium.org>
Date: Tue Nov 21 21:49:27 2017

Add missing BindTexture in PaintCanvasVideoRenderer.

PaintCanvasVideoRenderer::CopyVideoFrameSingleTextureToGLTexture
runs TexImage2D without first binding the target texture.
This could result in the clobbering of other textures.

BUG= 787333 , 710673 

Change-Id: Ieff4c3f6068ad3682b3bd356491b6979624dfd66
Reviewed-on: https://chromium-review.googlesource.com/781422
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Commit-Queue: Victor Miura <vmiura@chromium.org>
Cr-Commit-Position: refs/heads/master@{#518408}
[modify] https://crrev.com/9300c95d365861c2f0165787bff9fd03e1f6d1ed/media/renderers/paint_canvas_video_renderer.cc

Project Member

Comment 2 by sheriffbot@chromium.org, Nov 22

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: vmi...@chromium.org
Status: Fixed (was: Untriaged)
Assume this was fixed with the above CL.

Sign in to add a comment