Indicate in the Window menu which Chrome window has an active sound playing

chrome/browser/ui/cocoa/tabs/tab_strip_controller.mm

 // 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.
 
 #import "chrome/browser/ui/cocoa/tabs/tab_strip_controller.h"
 
 #import <QuartzCore/QuartzCore.h>
 
 #include <cmath>
 #include <limits>
@@ skipping 233 matching lines @@
 - (NSInteger)numberOfOpenPinnedTabs;
 - (NSInteger)numberOfOpenNonPinnedTabs;
 - (void)mouseMoved:(NSEvent*)event;
 - (void)setTabTrackingAreasEnabled:(BOOL)enabled;
 - (void)droppingURLsAt:(NSPoint)point
             givesIndex:(NSInteger*)index
            disposition:(WindowOpenDisposition*)disposition;
 - (void)setNewTabButtonHoverState:(BOOL)showHover;
 - (void)themeDidChangeNotification:(NSNotification*)notification;
 - (void)setNewTabImages;
+- (void)updateWindowMediaState:(TabMediaState)mediaState
+                            on:(content::WebContents*)changed;
+- (BOOL)isAnyOtherTab:(content::WebContents*)selected
+            withState:(TabMediaState)state;
+
 @end
 
 // A simple view class that contains the traffic light buttons. This class
 // ensures that the buttons display the icons when the mouse hovers over
 // them by overriding the _mouseInGroup method.
 @interface CustomWindowControlsView : NSView {
  @private
   BOOL mouseInside_;
 }
 
@@ skipping 1355 matching lines @@
       } else if (newState == kTabCrashed) {
         [tabController setIconImage:sadFaviconImage withToastAnimation:YES];
       } else {
         [tabController setIconImage:throbberImage];
       }
     } else {
       [tabController setIconImage:nil];
     }
   }
 
-  [tabController setMediaState:chrome::GetTabMediaStateForContents(contents)];
+  TabMediaState mediaState = chrome::GetTabMediaStateForContents(contents);
+
+  [self updateWindowMediaState:mediaState on:contents];
+  [tabController setMediaState:mediaState];
 
   [tabController updateVisibility];
 }
 
 // Called when a notification is received from the model that the given tab
 // has been updated. |loading| will be YES when we only want to update the
 // throbber state, not anything else about the (partially) loading tab.
 - (void)tabChangedWithContents:(content::WebContents*)contents
                        atIndex:(NSInteger)modelIndex
                     changeType:(TabStripModelObserver::TabChangeType)change {
@@ skipping 687 matching lines @@
     [[newTabButton_ cell] setImage:Overlay(background, hover, alpha)
                     forButtonState:image_button_cell::kHoverStateBackground];
   } else {
     [[newTabButton_ cell] setImage:nil
                     forButtonState:image_button_cell::kDefaultStateBackground];
     [[newTabButton_ cell] setImage:nil
                     forButtonState:image_button_cell::kHoverStateBackground];
   }
 }
 
+// Gets the tab and the media state to check whether the window
+// media state should be updated or not. If the tab media state is
+// AUDIO_PLAYING, the window media state should be set to AUDIO_PLAYING.
+// If the tab media state is AUDIO_MUTING, this method would check if the
+// window has no other tab with state AUDIO_PLAYING, then the window
+// media state will be set to AUDIO_MUTING. If the tab media state is NONE,
+// this method checks if the window has no playing or muting tab, then window
+// media state will be set as NONE.
+- (void)updateWindowMediaState:(TabMediaState)mediaState
+                            on:(content::WebContents*)selected {
+  NSWindow* window = [tabStripView_ window];
+  BrowserWindowController* windowController =
+      [BrowserWindowController browserWindowControllerForWindow:window];
+  if (mediaState == TAB_MEDIA_STATE_NONE) {
+    if (![self isAnyOtherTab:selected
+          withState:TAB_MEDIA_STATE_AUDIO_PLAYING] &&

[Robert Sesek, 2015/10/23 14:26:37]

nit: align colons

+        ![self isAnyOtherTab:selected withState:TAB_MEDIA_STATE_AUDIO_MUTING]) {
+      [windowController setMediaState:TAB_MEDIA_STATE_NONE];
+    } else if ([self isAnyOtherTab:selected
+              withState:TAB_MEDIA_STATE_AUDIO_MUTING]) {

[Robert Sesek, 2015/10/23 14:26:37]

nit: align colons

+      [windowController setMediaState:TAB_MEDIA_STATE_AUDIO_MUTING];
+    }
+  }
+  else if (mediaState == TAB_MEDIA_STATE_AUDIO_MUTING) {
+    if (![self isAnyOtherTab:selected withState:TAB_MEDIA_STATE_AUDIO_PLAYING])
+      [windowController setMediaState:TAB_MEDIA_STATE_AUDIO_MUTING];
+  }  else {
+    [windowController setMediaState:mediaState];
+  }
+}
+
+// Checks if tabs (excluding selected) has media state equals to the second
+// parameter. It returns YES when it finds the first tab with the criterion.
+- (BOOL)isAnyOtherTab:(content::WebContents*)selected
+            withState:(TabMediaState)state{
+  const int existingTabCount = tabStripModel_->count();
+  for (int i = 0; i < existingTabCount; ++i) {
+    content::WebContents* currentContents =
+             tabStripModel_->GetWebContentsAt(i);

[Robert Sesek, 2015/10/23 14:26:37]

nit: over-indented. Only indent 4 spaces from the previous line

+    if (selected == currentContents)
+      continue;
+    TabMediaState currentMediaStateForContents =
+                  chrome::GetTabMediaStateForContents(currentContents);

[Robert Sesek, 2015/10/23 14:26:37]

nit: same comment about over-indented

+    if (currentMediaStateForContents == state)
+      return YES;
+  }
+  return NO;
+}
+
 @end