Android: MediaPlayerBridge JNI cleanup.

media/base/android/java/src/org/chromium/media/MediaPlayerListener.java

Index: media/base/android/java/src/org/chromium/media/MediaPlayerListener.java
diff --git a/media/base/android/java/src/org/chromium/media/MediaPlayerListener.java b/media/base/android/java/src/org/chromium/media/MediaPlayerListener.java
index dc64bc7bf2340dcd29f5e1432cba3cf376069126..c1b1baba73aed2f6af5e9d9e28f9ee5e14668255 100644
--- a/media/base/android/java/src/org/chromium/media/MediaPlayerListener.java
+++ b/media/base/android/java/src/org/chromium/media/MediaPlayerListener.java
@@ -4,6 +4,7 @@
 
 package org.chromium.media;
 
+import android.content.Context;
 import android.media.MediaPlayer;
 
 import org.chromium.base.CalledByNative;
@@ -83,8 +84,17 @@ class MediaPlayerListener implements MediaPlayer.OnPreparedListener,
     }
 
     @CalledByNative
-    private static MediaPlayerListener create(int nativeMediaPlayerListener) {
-        return new MediaPlayerListener(nativeMediaPlayerListener);
+    private static MediaPlayerListener create(int nativeMediaPlayerListener,
+            Context context, MediaPlayer mediaPlayer) {
+        MediaPlayerListener listener = new MediaPlayerListener(nativeMediaPlayerListener);
+        mediaPlayer.setOnBufferingUpdateListener(listener);
+        mediaPlayer.setOnCompletionListener(listener);
+        mediaPlayer.setOnErrorListener(listener);
+        mediaPlayer.setOnPreparedListener(listener);
+        mediaPlayer.setOnSeekCompleteListener(listener);
+        mediaPlayer.setOnVideoSizeChangedListener(listener);
+        mediaPlayer.setWakeMode(context, android.os.PowerManager.FULL_WAKE_LOCK);
+        return listener;

[qinmin, 2012/09/20 14:45:21]

do we really need to return the listener here? Since we passed in the
MediaPlayer object here,  this object is no longer needed after we set the
listener to MediaPlayer.

[bulach, 2012/09/20 17:53:01]

you're right! removed..

     }
 
     /**