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

Unified Diff: content/browser/media/android/browser_surface_view_manager.cc

Issue 2791723003: Update GpuSurfaceTracker to include Android surfaces. (Closed)
Patch Set: rebased Created 3 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/media/android/browser_surface_view_manager.cc
diff --git a/content/browser/media/android/browser_surface_view_manager.cc b/content/browser/media/android/browser_surface_view_manager.cc
index 483ec0a56e9bb7e01bb967099f1fca8abe4b12be..dc854eeb91454b8a44f8da705617878b034920e1 100644
--- a/content/browser/media/android/browser_surface_view_manager.cc
+++ b/content/browser/media/android/browser_surface_view_manager.cc
@@ -41,7 +41,6 @@ void BrowserSurfaceViewManager::SetVideoSurface(gl::ScopedJavaSurface surface) {
if (surface.IsEmpty()) {
DCHECK_NE(surface_id_, media::SurfaceManager::kNoSurfaceID);
gpu::GpuSurfaceTracker::Get()->RemoveSurface(surface_id_);
- gpu::GpuSurfaceTracker::Get()->UnregisterViewSurface(surface_id_);
SendDestroyingVideoSurface(surface_id_);
surface_id_ = media::SurfaceManager::kNoSurfaceID;
} else {
@@ -49,9 +48,8 @@ void BrowserSurfaceViewManager::SetVideoSurface(gl::ScopedJavaSurface surface) {
// lookup will go through the Android specific path and get the java
// surface directly, so there's no need to add a valid native widget here.
surface_id_ = gpu::GpuSurfaceTracker::Get()->AddSurfaceForNativeWidget(
- gfx::kNullAcceleratedWidget);
- gpu::GpuSurfaceTracker::GetInstance()->RegisterViewSurface(
- surface_id_, surface.j_surface().obj());
+ gpu::GpuSurfaceTracker::SurfaceRecord(gfx::kNullAcceleratedWidget,
+ surface.j_surface().obj()));
SendSurfaceID(surface_id_);
}
}

Powered by Google App Engine
This is Rietveld 408576698