remove call to get android MediaPlayer class

chrome/browser/history/android/sqlite_cursor.cc

Index: chrome/browser/history/android/sqlite_cursor.cc
diff --git a/chrome/browser/history/android/sqlite_cursor.cc b/chrome/browser/history/android/sqlite_cursor.cc
index 691ed955d7cc3738e7d443ba7c8d3b888f3a4fa0..0d6402a25f34a12cb10e2bcde1c9233563382000 100644
--- a/chrome/browser/history/android/sqlite_cursor.cc
+++ b/chrome/browser/history/android/sqlite_cursor.cc
@@ -60,14 +60,14 @@ ScopedJavaLocalRef<jobject> SQLiteCursor::NewJavaSqliteCursor(
     return ScopedJavaLocalRef<jobject>();
   }
 
-  ScopedJavaLocalRef<jclass> sclass = GetClass(env, kSQLiteCursorClassPath);
   jmethodID method_id = MethodID::Get<MethodID::TYPE_INSTANCE>(
-      env, sclass.obj(), "<init>", "(I)V");
+      env, g_SQLiteCursor_clazz, "<init>", "(I)V");

[bulach, 2013/03/13 15:06:10]

while at it... :)
this is not a system class, but rather our own:
chrome/android/java/src/org/chromium/chrome/browser/database/SQLiteCursor.java

in which case, we should have a static factory method rather than inspect the
constructor..

and 58-75 could be simplified into:

SQLiteCursor* cursor = new SQLiteCursor(...);
ScopedJavaLocalRef<jobjec> ret(env,
    Java_SQLiteCursor_Create(reinterpret_cast<jint>(cursor));
return ret;

[qinmin, 2013/03/13 18:26:58]

Good point, Done.

 
   SQLiteCursor* cursor = new SQLiteCursor(column_names, statement, service,
                                           favicon_service);
   ScopedJavaLocalRef<jobject> obj(env,
-      env->NewObject(sclass.obj(), method_id, reinterpret_cast<jint>(cursor)));
+      env->NewObject(g_SQLiteCursor_clazz, method_id,
+                     reinterpret_cast<jint>(cursor)));
   if (obj.is_null()) {
     delete cursor;
     return ScopedJavaLocalRef<jobject>();