check the permission for wake lock before calling setWakeMode

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 7ab0fe8ba64332361190c95eeebd351c2e65f77a..7381ee1ec022ec059f2610279a07ce6cc985818f 100644
--- a/media/base/android/java/src/org/chromium/media/MediaPlayerListener.java
+++ b/media/base/android/java/src/org/chromium/media/MediaPlayerListener.java
@@ -5,6 +5,8 @@
 package org.chromium.media;
 
 import android.content.Context;
+import android.content.pm.PackageManager;
+import android.Manifest.permission;
 import android.media.MediaPlayer;
 
 import org.chromium.base.CalledByNative;
@@ -93,7 +95,10 @@ class MediaPlayerListener implements MediaPlayer.OnPreparedListener,
         mediaPlayer.setOnPreparedListener(listener);
         mediaPlayer.setOnSeekCompleteListener(listener);
         mediaPlayer.setOnVideoSizeChangedListener(listener);
-        mediaPlayer.setWakeMode(context, android.os.PowerManager.FULL_WAKE_LOCK);
+        if (PackageManager.PERMISSION_GRANTED ==

[Yaron, 2012/10/31 19:26:39]

Shouldn't we just add the WAKE_LOCK permission? Is there a way that the
permission can be rejected? Is there a specific bug associated with this?

+                context.checkCallingPermission(permission.WAKE_LOCK)) {
+            mediaPlayer.setWakeMode(context, android.os.PowerManager.FULL_WAKE_LOCK);
+        }
     }
 
     /**