Show camera and microphone status indicators.

chrome/browser/browser_process_impl.cc

Index: chrome/browser/browser_process_impl.cc
diff --git a/chrome/browser/browser_process_impl.cc b/chrome/browser/browser_process_impl.cc
index c72d25073d6d50bb6577783b3b1f5aff62a2c994..cec54b2a63415db98261c1540c7dfae726647d41 100644
--- a/chrome/browser/browser_process_impl.cc
+++ b/chrome/browser/browser_process_impl.cc
@@ -34,6 +34,7 @@
 #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"
@@ -557,6 +558,14 @@ BackgroundModeManager* BrowserProcessImpl::background_mode_manager() {
   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())