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

Unified Diff: media/gpu/avda_surface_bundle.cc

Issue 2692863011: Add ContentVideoViewOverlay to AVDA. (Closed)
Patch Set: fixed avda unittest Created 3 years, 9 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
« no previous file with comments | « media/gpu/avda_surface_bundle.h ('k') | media/gpu/content_video_view_overlay.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/gpu/avda_surface_bundle.cc
diff --git a/media/gpu/avda_surface_bundle.cc b/media/gpu/avda_surface_bundle.cc
index 12186a1b96cf2d5304b7091c85095ae9bfb76be4..608519bb936e3f4163421d2cc3f57fbea95a7554 100644
--- a/media/gpu/avda_surface_bundle.cc
+++ b/media/gpu/avda_surface_bundle.cc
@@ -3,6 +3,7 @@
// found in the LICENSE file.
#include "media/gpu/avda_surface_bundle.h"
+#include "media/base/android/android_overlay.h"
namespace media {
@@ -11,7 +12,8 @@ AVDASurfaceBundle::AVDASurfaceBundle(int surface_id) : surface_id(surface_id) {}
AVDASurfaceBundle::~AVDASurfaceBundle() {
// Explicitly free the surface first, just to be sure that it's deleted before
// the SurfaceTexture is.
- surface = gl::ScopedJavaSurface();
+ surface_texture_surface = gl::ScopedJavaSurface();
+
// Also release the back buffers. We might want to let the consumer do this
// after it has swapped the back buffer it wants, but we don't.
if (surface_texture)
« no previous file with comments | « media/gpu/avda_surface_bundle.h ('k') | media/gpu/content_video_view_overlay.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698