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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: content/common/gpu/media/android_video_decode_accelerator.cc

Issue 1682343002: AVDACodecImages keep a reference to the SurfaceTexture backing them (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 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 side-by-side diff with in-line comments
Download patch
« content/common/gpu/media/android_deferred_rendering_backing_strategy.cc ('K') | « content/common/gpu/media/android_deferred_rendering_backing_strategy.cc ('k') | content/common/gpu/media/avda_codec_image.h » ('j') | content/common/gpu/media/avda_shared_state.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/common/gpu/media/android_video_decode_accelerator.cc
diff --git a/content/common/gpu/media/android_video_decode_accelerator.cc b/content/common/gpu/media/android_video_decode_accelerator.cc
index 126daba961d2389210d109d03a51ffeb62f915a6..eebb137cd49cf121d8d30d8a6cb42ba912800027 100644
--- a/content/common/gpu/media/android_video_decode_accelerator.cc
+++ b/content/common/gpu/media/android_video_decode_accelerator.cc
@@ -873,7 +873,6 @@ AndroidVideoDecodeAccelerator::GetGlDecoder() const {
void AndroidVideoDecodeAccelerator::OnFrameAvailable() {
// Remember: this may be on any thread.
- DCHECK(strategy_);
liberato (no reviews please) 2016/02/10 06:39:12 why remove this?
watk 2016/02/11 00:26:51 strategy_ can't be null any more because it's alwa
strategy_->OnFrameAvailable();
}
« content/common/gpu/media/android_deferred_rendering_backing_strategy.cc ('K') | « content/common/gpu/media/android_deferred_rendering_backing_strategy.cc ('k') | content/common/gpu/media/avda_codec_image.h » ('j') | content/common/gpu/media/avda_shared_state.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698