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

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

Issue 12625005: remove call to get android MediaPlayer class (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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 | « no previous file | no next file » | 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 4ebde7dfbe42e12aeb1fe0d1f434d067d5642cc0..b83c9e4a4a542032895f68cd5057660deba68f51 100644
--- a/media/base/android/media_player_bridge.cc
+++ b/media/base/android/media_player_bridge.cc
@@ -328,10 +328,8 @@ void MediaPlayerBridge::GetMetadata() {
JNIEnv* env = AttachCurrentThread();
CHECK(env);
- ScopedJavaLocalRef<jclass> media_player_class(
- GetClass(env, "android/media/MediaPlayer"));
Yaron 2013/03/11 19:58:40 Sorry to do this to you, but since you're already
jmethodID method = MethodID::Get<MethodID::TYPE_INSTANCE>(
- env, media_player_class.obj(), "getMetadata",
+ env, g_MediaPlayer_clazz, "getMetadata",
"(ZZ)Landroid/media/Metadata;");
ScopedJavaLocalRef<jobject> j_metadata(
env, env->CallObjectMethod(
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698