remove call to get android MediaPlayer class

media/base/android/media_player_bridge.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/base/android/media_player_bridge.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "base/basictypes.h"
 #include "base/logging.h"
@@ skipping 310 matching lines @@
 
   GetMetadata();
 
   media_prepared_cb_.Run(player_id_, duration_);
 }
 
 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 updating this, can you remove
the rest of the manual bindings? Marcus and I have started doing this and would
appreciate your help. E.g. https://codereview.chromium.org/12604003/ and
https://codereview.chromium.org/12638003/

   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(
           j_media_player_.obj(), method, JNI_FALSE, JNI_FALSE));
   CheckException(env);
   if (j_metadata.is_null())
     return;
 
   ScopedJavaLocalRef<jclass> metadata_class(
       GetClass(env, "android/media/Metadata"));
@@ skipping 41 matching lines @@
 
 bool MediaPlayerBridge::RegisterMediaPlayerBridge(JNIEnv* env) {
   bool ret = RegisterNativesImpl(env);
   DCHECK(g_MediaPlayerBridge_clazz);
   if (ret)
     ret = JNI_MediaPlayer::RegisterNativesImpl(env);
   return ret;
 }
 
 }  // namespace media