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

chrome/browser/ui/cocoa/browser_window_cocoa.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.
 
 #include "chrome/browser/ui/cocoa/browser_window_cocoa.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/logging.h"
 #import "base/mac/sdk_forward_declarations.h"
@@ skipping 70 matching lines @@
 // Used for defining the layout of the NSAlert and NSTextField within the
 // accessory view.
 const int kAppTextFieldVerticalSpacing = 2;
 const int kAppTextFieldWidth = 200;
 const int kAppTextFieldHeight = 22;
 const int kBookmarkAppBubbleViewWidth = 200;
 const int kBookmarkAppBubbleViewHeight = 46;
 
 const int kIconPreviewTargetSize = 128;
 
+const char kWhiteSpaceCharacter = ' ';
+
 base::string16 TrimText(NSString* controlText) {
   base::string16 text = base::SysNSStringToUTF16(controlText);
   base::TrimWhitespace(text, base::TRIM_ALL, &text);
   return text;
 }
 
 }  // namespace
 
 @interface TextRequiringDelegate : NSObject<NSTextFieldDelegate> {
  @private
@@ skipping 188 matching lines @@
 
 gfx::NativeWindow BrowserWindowCocoa::GetNativeWindow() const {
   return window();
 }
 
 StatusBubble* BrowserWindowCocoa::GetStatusBubble() {
   return [controller_ statusBubble];
 }
 
 void BrowserWindowCocoa::UpdateTitleBar() {
-  NSString* newTitle =
-      base::SysUTF16ToNSString(browser_->GetWindowTitleForCurrentTab());
+  NSString* newTitle = GetWindowTitle();
 
-  pending_window_title_.reset(
-      [BrowserWindowUtils scheduleReplaceOldTitle:pending_window_title_.get()
-                                     withNewTitle:newTitle
-                                        forWindow:window()]);
+  pending_window_title_.reset([BrowserWindowUtils
+      scheduleReplaceOldTitle:pending_window_title_.get()
+                 withNewTitle:newTitle
+                    forWindow:window()]);
+}
+
+void BrowserWindowCocoa::UpdateMediaState(TabMediaState media_state) {
+  media_state_ = media_state;
+  UpdateTitleBar();
+}
+
+NSString* BrowserWindowCocoa::GetWindowTitle() {
+  NSMutableString* newTitle = [NSMutableString
+      stringWithString:base::SysUTF16ToNSString(
+                           browser_->GetWindowTitleForCurrentTab())];
+
+  if (media_state_ == TAB_MEDIA_STATE_AUDIO_PLAYING)

[Robert Sesek, 2015/10/28 18:45:57]

nit: needs braces {} around both if and else if.

+    [newTitle
+        appendFormat:@"%c%@", kWhiteSpaceCharacter,

[Robert Sesek, 2015/10/28 18:45:57]

… continuing comment from the GRD file. Then this would be:

l10n_util::GetNSStringF(IDS_WINDOW_AUDIO_PLAYING_MAC,
browser_->GetWindowTitleForCurrentTab())

+                     l10n_util::GetNSString(IDS_WINDOW_AUDIO_PLAYING_MAC)];
+  else if (media_state_ == TAB_MEDIA_STATE_AUDIO_MUTING)
+    [newTitle appendFormat:@"%c%@", kWhiteSpaceCharacter,
+                           l10n_util::GetNSString(IDS_WINDOW_AUDIO_MUTE_MAC)];
+  return newTitle;
+}
+
+TabMediaState BrowserWindowCocoa::GetMediaState() {
+  return media_state_;
 }
 
 void BrowserWindowCocoa::BookmarkBarStateChanged(
     BookmarkBar::AnimateChangeType change_type) {
   [[controller_ bookmarkBarController]
       updateState:browser_->bookmark_bar_state()
        changeType:change_type];
 }
 
 void BrowserWindowCocoa::UpdateDevTools() {
@@ skipping 538 matching lines @@
 void BrowserWindowCocoa::UnhideDownloadShelf() {
   GetDownloadShelf()->Unhide();
 }
 
 void BrowserWindowCocoa::HideDownloadShelf() {
   GetDownloadShelf()->Hide();
   StatusBubble* statusBubble = GetStatusBubble();
   if (statusBubble)
     statusBubble->Hide();
 }