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

Unified Diff: media/base/android/media_player_bridge.cc

Issue 651243003: Remove MediaPlayerAndroid::IsSurfaceInUse() call (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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/base/android/media_player_bridge.h ('k') | media/base/android/media_source_player.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/android/media_player_bridge.cc
diff --git a/media/base/android/media_player_bridge.cc b/media/base/android/media_player_bridge.cc
index c95d8575edcdf454129d03f490a0d0aa64122b80..2a8037753968da804db2874d7becc053bda28270 100644
--- a/media/base/android/media_player_bridge.cc
+++ b/media/base/android/media_player_bridge.cc
@@ -47,7 +47,6 @@ MediaPlayerBridge::MediaPlayerBridge(
can_pause_(true),
can_seek_forward_(true),
can_seek_backward_(true),
- is_surface_in_use_(false),
volume_(-1.0),
allow_credentials_(allow_credentials),
weak_factory_(this) {
@@ -142,7 +141,6 @@ void MediaPlayerBridge::SetVideoSurface(gfx::ScopedJavaSurface surface) {
JNIEnv* env = base::android::AttachCurrentThread();
CHECK(env);
- is_surface_in_use_ = true;
Java_MediaPlayerBridge_setSurface(
env, j_media_player_bridge_.obj(), surface.j_surface().obj());
}
@@ -384,7 +382,6 @@ void MediaPlayerBridge::Release() {
pending_seek_ = GetCurrentTime();
prepared_ = false;
pending_play_ = false;
- is_surface_in_use_ = false;
SetVideoSurface(gfx::ScopedJavaSurface());
JNIEnv* env = base::android::AttachCurrentThread();
Java_MediaPlayerBridge_release(env, j_media_player_bridge_.obj());
@@ -548,8 +545,4 @@ GURL MediaPlayerBridge::GetFirstPartyForCookies() {
return first_party_for_cookies_;
}
-bool MediaPlayerBridge::IsSurfaceInUse() const {
- return is_surface_in_use_;
-}
-
} // namespace media
« no previous file with comments | « media/base/android/media_player_bridge.h ('k') | media/base/android/media_source_player.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698