fixup! [MM] Enabling CAPI for ME

Resolves muted start.

Previously, SetVolume function refused to set volume when player state is
PLAYER_STATE_IDLE but actual precondition accept idle state to set volume.
Also player_set_mute is removed from SetVolume because mute state is
actually handled at Webkit side.
Reviewed by: SeungSeop Park, Venugopal S M

Change-Id: I3717bb653e5003670c1d05980b6a2c90ca3293a9
Signed-off-by: ws29.jung <ws29.jung@samsung.com>

diff --git a/tizen_src/chromium_impl/media/base/tizen/media_player_bridge_capi.cc b/tizen_src/chromium_impl/media/base/tizen/media_player_bridge_capi.cc
index 59c222c..cd47cc9 100644 (file)
--- a/tizen_src/chromium_impl/media/base/tizen/media_player_bridge_capi.cc
+++ b/tizen_src/chromium_impl/media/base/tizen/media_player_bridge_capi.cc
@@ -437,27 +437,13 @@ void MediaPlayerBridgeCapi::Release() {
 
 void MediaPlayerBridgeCapi::SetVolume(double volume) {
   VLOG(1) << __FUNCTION__ << " : Player Id = " << GetPlayerId()
-    << " : Volume : " << volume;
-  if (GetPlayerState() > PLAYER_STATE_IDLE){
-    if(volume == 0.0){
-      if(player_set_mute(player_,true)
-        != PLAYER_ERROR_NONE)
-      LOG(ERROR) << "|player_set_mute(true)| failed in MediaPlayerBridgeCapi::"
-          << __FUNCTION__;
+          << " : Volume : " << volume;
 
-      return;
-    }
-
-    if(player_set_mute(player_,false)
-        != PLAYER_ERROR_NONE)
-      LOG(ERROR) << "|player_set_mute(false)| failed in MediaPlayerBridgeCapi::"
-          << __FUNCTION__;
+  if (GetPlayerState() == PLAYER_STATE_NONE)
+    return;
 
-    if (player_set_volume(player_, (float)volume, (float)volume)
-      != PLAYER_ERROR_NONE)
-      LOG(ERROR) << "|player_set_volume| failed in MediaPlayerBridgeCapi::"
-          << __FUNCTION__;
-  }
+  if (player_set_volume(player_, volume, volume) != PLAYER_ERROR_NONE)
+    LOG(ERROR) << "|player_set_volume| failed";
 }
 
 void MediaPlayerBridgeCapi::UpdateMediaType() {