Propagate audible state from player to the containing tab

content/browser/web_contents/web_contents_impl.cc

 // Copyright (c) 2012 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 "content/browser/web_contents/web_contents_impl.h"
 
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/lazy_instance.h"
@@ skipping 2475 matching lines @@
   }
   Send(new ViewMsg_StopFinding(GetRoutingID(), action));
 }
 
 void WebContentsImpl::InsertCSS(const std::string& css) {
   GetMainFrame()->Send(new FrameMsg_CSSInsertRequest(
       GetMainFrame()->GetRoutingID(), css));
 }
 
 bool WebContentsImpl::WasRecentlyAudible() {
+#if defined(ENABLE_BROWSER_CDMS) && defined(OS_ANDROID)

[no sievers, 2015/02/10 01:33:03]

Why is there an #ifdef here?
Can you put a comment explaining that? Including what is different between the
two implementations and why this is handled differently on Android.

[Tima Vaisburd, 2015/02/11 03:34:05]

Added a comment, please take a look.

+  return media_web_contents_observer_->IsAudible();
+#else
   return audio_stream_monitor_.WasRecentlyAudible();
+#endif
 }
 
 void WebContentsImpl::GetManifest(const GetManifestCallback& callback) {
   manifest_manager_host_->GetManifest(GetMainFrame(), callback);
 }
 
 void WebContentsImpl::ExitFullscreen() {
   // Clean up related state and initiate the fullscreen exit.
   GetRenderViewHostImpl()->RejectMouseLockOrUnlockIfNecessary();
   ExitFullscreenMode();
@@ skipping 1969 matching lines @@
   node->render_manager()->ResumeResponseDeferredAtStart();
 }
 
 void WebContentsImpl::SetForceDisableOverscrollContent(bool force_disable) {
   force_disable_overscroll_content_ = force_disable;
   if (view_)
     view_->SetOverscrollControllerEnabled(CanOverscrollContent());
 }
 
 }  // namespace content