Plumb audio focus support for spitzer clients.

content/browser/media/media_web_contents_observer.cc

 // Copyright 2014 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/media/media_web_contents_observer.h"
 
+#include "base/callback.h"
 #include "base/memory/scoped_ptr.h"
 #include "build/build_config.h"
 #include "content/browser/power_save_blocker_impl.h"
 #include "content/browser/web_contents/web_contents_impl.h"
 #include "content/common/frame_messages.h"
 #include "content/public/browser/render_frame_host.h"
 #include "content/public/browser/web_contents.h"
 #include "ipc/ipc_message_macros.h"
+#include "media/base/bind_to_current_loop.h"
 
 namespace content {
 
 MediaWebContentsObserver::MediaWebContentsObserver(WebContents* web_contents)
     : WebContentsObserver(web_contents) {}
 
 MediaWebContentsObserver::~MediaWebContentsObserver() {}
 
 void MediaWebContentsObserver::RenderFrameDeleted(
     RenderFrameHost* render_frame_host) {
@@ skipping 20 matching lines @@
                         OnMediaDestroyedNotification)
     IPC_MESSAGE_HANDLER(FrameHostMsg_MediaPlayingNotification,
                         OnMediaPlayingNotification)
     IPC_MESSAGE_HANDLER(FrameHostMsg_MediaPausedNotification,
                         OnMediaPausedNotification)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
   return handled;
 }
 
+void MediaWebContentsObserver::WasShown() {
+  // Restore power save blocker if there are active video players running.
+  if (!active_video_players_.empty() && !video_power_save_blocker_)
+    CreateVideoPowerSaveBlocker();
+}
+
+void MediaWebContentsObserver::WasHidden() {
+  // If there are entities capturing screenshots or video (e.g., mirroring),
+  // don't release the power save blocker.
+  if (!web_contents()->GetCapturerCount())
+    video_power_save_blocker_.reset();
+}
+
+// static
+void MediaWebContentsObserver::HasAudioFocus(int render_process_id,
+                                             int render_frame_id,
+                                             const base::Closure& on_focus_cb) {
+  if (!BrowserThread::CurrentlyOn(BrowserThread::UI)) {
+    BrowserThread::PostTask(
+        BrowserThread::UI, FROM_HERE,
+        base::Bind(&MediaWebContentsObserver::HasAudioFocus, render_process_id,
+                   render_frame_id, media::BindToCurrentLoop(on_focus_cb)));
+    return;
+  }
+
+  RenderFrameHost* rfh =
+      RenderFrameHost::FromID(render_process_id, render_frame_id);
+  if (rfh) {

[mlamouri (slow - plz ping), 2016/01/14 15:03:41]

nit:
```
if (!rfh)
  return;
```

Also, probably no need to null check `rfh`. `::FromRenderFrameHost()` will not
assume `rfh` is not null.

[DaleCurtis, 2016/01/14 23:16:10]

There was some reason I wrote it this way, but I don't see why now, so done.

+    WebContents* wc = WebContents::FromRenderFrameHost(rfh);

[mlamouri (slow - plz ping), 2016/01/14 15:03:41]

nit: we usually use `web_contents`, I can't remember seeing `wc` in the code
base.

[DaleCurtis, 2016/01/14 23:16:10]

Done.

+    if (wc) {

[mlamouri (slow - plz ping), 2016/01/14 15:03:41]

nit:
```
if (!wc)
  return;
```

[DaleCurtis, 2016/01/14 23:16:10]

Done.

+      MediaWebContentsObserver* p_this =
+          static_cast<WebContentsImpl*>(wc)->media_web_contents_observer();
+      p_this->CheckFocus(render_frame_id, on_focus_cb);
+      return;
+    }
+  }
+}
+
+void MediaWebContentsObserver::CheckFocus(int render_frame_id,
+                                          const base::Closure& on_focus_cb) {
+  NOTIMPLEMENTED();
+}
+
 void MediaWebContentsObserver::OnMediaDestroyedNotification(
     RenderFrameHost* render_frame_host,
     int64_t player_cookie) {
   OnMediaPausedNotification(render_frame_host, player_cookie, true);
 }
 
 void MediaWebContentsObserver::OnMediaPlayingNotification(
     RenderFrameHost* render_frame_host,
     int64_t player_cookie,
     bool has_video,
@@ skipping 76 matching lines @@
   video_power_save_blocker_ = PowerSaveBlocker::Create(
       PowerSaveBlocker::kPowerSaveBlockPreventDisplaySleep,
       PowerSaveBlocker::kReasonVideoPlayback, "Playing video");
 // TODO(mfomitchev): Support PowerSaveBlocker on Aura - crbug.com/546718.
 #if defined(OS_ANDROID) && !defined(USE_AURA)
   static_cast<PowerSaveBlockerImpl*>(video_power_save_blocker_.get())
       ->InitDisplaySleepBlocker(web_contents());
 #endif
 }
 
-void MediaWebContentsObserver::WasShown() {
-  // Restore power save blocker if there are active video players running.
-  if (!active_video_players_.empty() && !video_power_save_blocker_)
-    CreateVideoPowerSaveBlocker();
-}
-
-void MediaWebContentsObserver::WasHidden() {
-  // If there are entities capturing screenshots or video (e.g., mirroring),
-  // don't release the power save blocker.
-  if (!web_contents()->GetCapturerCount())
-    video_power_save_blocker_.reset();
-}
-
 void MediaWebContentsObserver::MaybeReleasePowerSaveBlockers() {
   // If there are no more audio players and we don't have audio stream
   // monitoring, release the audio power save blocker here instead of during
   // NotifyNavigationStateChanged().
   if (active_audio_players_.empty() &&
       !AudioStreamMonitor::monitoring_available()) {
     audio_power_save_blocker_.reset();
   }
 
   // If there are no more video players, clear the video power save blocker.
@@ skipping 39 matching lines @@
   if (it == player_map->end())
     return;
 
   for (int64_t player_cookie : it->second)
     removed_players->insert(MediaPlayerId(render_frame_host, player_cookie));
 
   player_map->erase(it);
 }
 
 }  // namespace content