Propagate audible state from player to the containing tab

media/base/android/media_player_android.cc

 // Copyright (c) 2013 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_android.h"
 
 #include "base/android/jni_android.h"
 #include "base/logging.h"
 #include "base/message_loop/message_loop_proxy.h"
 #include "media/base/android/media_drm_bridge.h"
@@ skipping 61 matching lines @@
   jobject j_context = base::android::GetApplicationContext();
   DCHECK(j_context);
 
   listener_->CreateMediaPlayerListener(j_context, j_media_player);
 }
 
 void MediaPlayerAndroid::DetachListener() {
   listener_->ReleaseMediaPlayerListenerResources();
 }
 
+void MediaPlayerAndroid::SetAudible(bool is_audible) {
+  manager_->OnAudibleStateChanged(player_id(), is_audible);

[qinmin, 2015/02/03 20:33:24]

For each SetAudible() call, the AudioMonitorAndroid needs to loop through all
the players and check if the tab is audible. 
it is very inefficient if we handing off all the calls to it.
We can simply do a check here to see if audible state changes before we pass the
call to the manager.

[Tima Vaisburd, 2015/02/03 23:53:05]

Added another |bool is_audible_;| to MediaPlayerAndroid.
Maybe it would be also OK to modify 
AudioMonitorAndroid::OnAudibleStateChanged() like

{
  bool old_state = audio_status_map_[Key(rfh, player_id)];
  if (is_audible != old_state) { ... }
}
to keep the logic in one place?

+}
+
 
 }  // namespace media