Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(194)

Side by Side Diff: content/common/gpu/media/avda_codec_image.cc

Issue 1818073002: Always apply UniformMatrix4fvStreamTextureMatrixCHROMIUM matrix argument. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: GetTextureMatrix should be the only available method Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/common/gpu/media/avda_codec_image.h" 5 #include "content/common/gpu/media/avda_codec_image.h"
6 6
7 #include <string.h> 7 #include <string.h>
8 8
9 #include "base/metrics/histogram_macros.h" 9 #include "base/metrics/histogram_macros.h"
10 #include "content/common/gpu/media/avda_shared_state.h" 10 #include "content/common/gpu/media/avda_shared_state.h"
(...skipping 213 matching lines...) Expand 10 before | Expand all | Expand 10 after
224 surface_texture_->DetachFromGLContext(); 224 surface_texture_->DetachFromGLContext();
225 shared_state_->DidDetachSurfaceTexture(); 225 shared_state_->DidDetachSurfaceTexture();
226 glBindTexture(GL_TEXTURE_EXTERNAL_OES, bound_service_id); 226 glBindTexture(GL_TEXTURE_EXTERNAL_OES, bound_service_id);
227 } else { 227 } else {
228 // Surface texture is already attached, so just update it. 228 // Surface texture is already attached, so just update it.
229 UpdateSurfaceTexture(kDoRestoreBindings); 229 UpdateSurfaceTexture(kDoRestoreBindings);
230 } 230 }
231 } 231 }
232 232
233 memcpy(matrix, gl_matrix_, sizeof(gl_matrix_)); 233 memcpy(matrix, gl_matrix_, sizeof(gl_matrix_));
234 YInvertMatrix(matrix);
234 } 235 }
235 236
236 bool AVDACodecImage::IsCodecBufferOutstanding() const { 237 bool AVDACodecImage::IsCodecBufferOutstanding() const {
237 return codec_buffer_index_ != kInvalidCodecBufferIndex && media_codec_; 238 return codec_buffer_index_ != kInvalidCodecBufferIndex && media_codec_;
238 } 239 }
239 240
240 } // namespace content 241 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698