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

Unified Diff: media/gpu/android_video_surface_chooser_impl.h

Issue 2883913003: Add multiple destruction callbacks to AndroidOverlay. (Closed)
Patch Set: rebased.... though i got not conflicts? Created 3 years, 7 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/android_video_surface_chooser.h ('k') | media/gpu/android_video_surface_chooser_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/gpu/android_video_surface_chooser_impl.h
diff --git a/media/gpu/android_video_surface_chooser_impl.h b/media/gpu/android_video_surface_chooser_impl.h
index d8c72bda219c529533f9fb199d57fc2b31870959..37ad5c9f570fe2753adffc40125bc2f57446651e 100644
--- a/media/gpu/android_video_surface_chooser_impl.h
+++ b/media/gpu/android_video_surface_chooser_impl.h
@@ -24,7 +24,6 @@ class MEDIA_GPU_EXPORT AndroidVideoSurfaceChooserImpl
// AndroidVideoSurfaceChooser
void Initialize(UseOverlayCB use_overlay_cb,
UseSurfaceTextureCB use_surface_texture_cb,
- StopUsingOverlayImmediatelyCB stop_immediately_cb,
AndroidOverlayFactoryCB initial_factory) override;
void ReplaceOverlayFactory(AndroidOverlayFactoryCB factory) override;
@@ -32,12 +31,10 @@ class MEDIA_GPU_EXPORT AndroidVideoSurfaceChooserImpl
// AndroidOverlay callbacks.
void OnOverlayReady(AndroidOverlay*);
void OnOverlayFailed(AndroidOverlay*);
- void OnSurfaceDestroyed(AndroidOverlay*);
// Client callbacks.
UseOverlayCB use_overlay_cb_;
UseSurfaceTextureCB use_surface_texture_cb_;
- StopUsingOverlayImmediatelyCB stop_immediately_cb_;
// Current overlay that we've constructed but haven't received ready / failed
// callbacks yet. Will be nullptr if we haven't constructed one, or if we
« no previous file with comments | « media/gpu/android_video_surface_chooser.h ('k') | media/gpu/android_video_surface_chooser_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698