Show camera and microphone status indicators.

chrome/browser/browser_process_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 "chrome/browser/browser_process_impl.h"
 
 #include <map>
 #include <set>
 #include <vector>
 
@@ skipping 16 matching lines @@
 #include "chrome/browser/debugger/remote_debugging_server.h"
 #include "chrome/browser/download/download_request_limiter.h"
 #include "chrome/browser/download/download_status_updater.h"
 #include "chrome/browser/extensions/extension_event_router_forwarder.h"
 #include "chrome/browser/extensions/extension_tab_id_map.h"
 #include "chrome/browser/first_run/upgrade_util.h"
 #include "chrome/browser/google/google_url_tracker.h"
 #include "chrome/browser/icon_manager.h"
 #include "chrome/browser/intranet_redirect_detector.h"
 #include "chrome/browser/io_thread.h"
+#include "chrome/browser/media/media_stream_capture_indicator.h"
 #include "chrome/browser/metrics/metrics_service.h"
 #include "chrome/browser/metrics/thread_watcher.h"
 #include "chrome/browser/net/chrome_net_log.h"
 #include "chrome/browser/net/crl_set_fetcher.h"
 #include "chrome/browser/net/sdch_dictionary_fetcher.h"
 #include "chrome/browser/notifications/notification_ui_manager.h"
 #include "chrome/browser/policy/browser_policy_connector.h"
 #include "chrome/browser/policy/policy_service_impl.h"
 #include "chrome/browser/policy/policy_service_stub.h"
 #include "chrome/browser/prefs/browser_prefs.h"
@@ skipping 503 matching lines @@
   return tab_closeable_state_watcher_.get();
 }
 
 BackgroundModeManager* BrowserProcessImpl::background_mode_manager() {
   DCHECK(CalledOnValidThread());
   if (!background_mode_manager_.get())
     CreateBackgroundModeManager();
   return background_mode_manager_.get();
 }
 
+MediaStreamCaptureIndicator*
+BrowserProcessImpl::media_stream_capture_indicator() {

[MAD, 2012/04/25 15:34:53]

The coding style wants the name of the function on the same line as the return
type, so I would prefer this format:
MediaStreamCaptureIndicator* BrowserProcessImpl::media_stream_capture_indicator(
    ) {

[Nico, 2012/04/25 15:52:51]

line 584 (20 lines below) has

returntype
  BrowserProcessImpl::method_name() {

so I'd be consistent with that. Just indent the method name 4.

[no longer working on chromium, 2012/04/30 09:59:38]

Thanks Nico for addressing this, it looks like
returntype
  BrowserProcessImpl::method_name() {
is pretty common in the file, so I stick with the style.

[no longer working on chromium, 2012/04/30 09:59:38]

Done.

+  DCHECK(CalledOnValidThread());
+  if (!media_stream_capture_indicator_.get())
+    media_stream_capture_indicator_.reset(new MediaStreamCaptureIndicator());
+  return media_stream_capture_indicator_.get();
+}
+
 StatusTray* BrowserProcessImpl::status_tray() {
   DCHECK(CalledOnValidThread());
   if (!status_tray_.get())
     CreateStatusTray();
   return status_tray_.get();
 }
 
 
 SafeBrowsingService* BrowserProcessImpl::safe_browsing_service() {
   DCHECK(CalledOnValidThread());
@@ skipping 345 matching lines @@
 }
 
 void BrowserProcessImpl::OnAutoupdateTimer() {
   if (CanAutorestartForUpdate()) {
     DLOG(WARNING) << "Detected update.  Restarting browser.";
     RestartBackgroundInstance();
   }
 }
 
 #endif  // (defined(OS_WIN) || defined(OS_LINUX)) && !defined(OS_CHROMEOS)