[scheduler] Teach scheduler about audio state

content/browser/web_contents/web_contents_impl.cc

Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 397c14b9dbbf688fadb669dfc336582cdd7a1488..0d392b178664246bc0f3e0cc64d01d2575ecf972 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -882,6 +882,29 @@ int WebContentsImpl::SendToAllFrames(IPC::Message* message) {
   return number_of_messages;
 }
 
+int WebContentsImpl::SendToAllViews(IPC::Message* message) {

[clamy, 2016/10/05 12:11:35]

I think it would make a lot more sense to have the ViewMsg_AudioStateChanged
become a FrameMsg_AudioStateChanged rather than add this function. On top of
that it is a bit misleading since it may potentially send the same IPC multiple
times to the same view (if several frames share the same view).

[altimin, 2016/10/05 13:26:37]

Done.

+  std::unordered_set<RenderViewHost*> render_view_hosts;
+  for (RenderFrameHost* rfh : GetAllFrames()) {
+    if (!rfh->IsRenderFrameLive())
+      continue;
+
+    render_view_hosts.insert(rfh->GetRenderViewHost());
+  }
+
+  int number_of_messages = 0;
+  for (RenderViewHost* rvh : render_view_hosts) {
+    if (!rvh->IsRenderViewLive())
+      continue;
+
+    ++number_of_messages;
+    IPC::Message* message_copy = new IPC::Message(*message);
+    message_copy->set_routing_id(rvh->GetRoutingID());
+    rvh->Send(message_copy);
+  }
+  delete message;
+  return number_of_messages;
+}
+
 void WebContentsImpl::SendPageMessage(IPC::Message* msg) {
   frame_tree_.root()->render_manager()->SendPageMessage(msg, nullptr);
 }
@@ -1278,8 +1301,7 @@ void WebContentsImpl::SetAudioMuted(bool mute) {
   FOR_EACH_OBSERVER(WebContentsObserver, observers_,
                     DidUpdateAudioMutingState(mute));
 
-  // Notification for UI updates in response to the changed muting state.
-  NotifyNavigationStateChanged(INVALIDATE_TYPE_TAB);
+  OnAudioStateChanged(!mute && audio_stream_monitor_.IsCurrentlyAudible());
 }
 
 bool WebContentsImpl::IsConnectedToBluetoothDevice() const {
@@ -1339,6 +1361,14 @@ void WebContentsImpl::NotifyNavigationStateChanged(
     GetOuterWebContents()->NotifyNavigationStateChanged(changed_flags);
 }
 
+void WebContentsImpl::OnAudioStateChanged(bool is_audio_playing) {
+  SendToAllViews(
+      new ViewMsg_AudioStateChanged(GetRoutingID(), is_audio_playing));
+
+  // Notification for UI updates in response to the changed muting state.
+  NotifyNavigationStateChanged(INVALIDATE_TYPE_TAB);
+}
+
 base::TimeTicks WebContentsImpl::GetLastActiveTime() const {
   return last_active_time_;
 }