Add PipelineStatus UMA logging to media_internals

content/browser/media/media_internals.h

 // 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.
 
 #ifndef CONTENT_BROWSER_MEDIA_MEDIA_INTERNALS_H_
 #define CONTENT_BROWSER_MEDIA_MEDIA_INTERNALS_H_
 
 #include <string>
 #include <vector>
 
 #include "base/callback_forward.h"
 #include "base/compiler_specific.h"
 #include "base/lazy_instance.h"
 #include "base/strings/string16.h"
 #include "base/synchronization/lock.h"
 #include "base/values.h"
 #include "content/common/content_export.h"
 #include "media/audio/audio_logging.h"
+#include "media/base/media_log.h"
 #include "media/video/capture/video_capture_device_info.h"
 
 namespace media {
 class AudioParameters;
 struct MediaLogEvent;
 }
 
 namespace content {
 
 // This class stores information about currently active media.
@@ skipping 23 matching lines @@
   void SendVideoCaptureDeviceCapabilities();
 
   // Called to inform of the capabilities enumerated for video devices.
   void UpdateVideoCaptureDeviceCapabilities(
       const media::VideoCaptureDeviceInfos& video_capture_device_infos);
 
   // AudioLogFactory implementation.  Safe to call from any thread.
   scoped_ptr<media::AudioLog> CreateAudioLog(AudioComponent component) override;
 
  private:
+  // Inner class to handle reporting pipelinestatus to UMA
+  class MediaInternalsUMAHandler;
+
   friend class AudioLogImpl;
   friend class MediaInternalsTest;
   friend struct base::DefaultLazyInstanceTraits<MediaInternals>;
 
+  scoped_ptr<MediaInternalsUMAHandler> uma_handler;

[DaleCurtis, 2014/11/11 19:24:01]

Needs underscore_ should be at the end of the class with the other variables.

[prabhur1, 2014/11/12 21:32:37]

Done.

+
   MediaInternals();
 
   // Sends |update| to each registered UpdateCallback.  Safe to call from any
   // thread, but will forward to the IO thread.
   void SendUpdate(const base::string16& update);
 
   // Caches |value| under |cache_key| so that future SendAudioStreamData() calls
   // will include the current data.  Calls JavaScript |function|(|value|) for
   // each registered UpdateCallback.  SendUpdateAndPurgeCache() is similar but
   // purges the cache entry after completion instead.
   void SendUpdateAndCacheAudioStreamKey(const std::string& cache_key,
                                         const std::string& function,
                                         const base::DictionaryValue* value);
   void SendUpdateAndPurgeAudioStreamCache(const std::string& cache_key,
                                           const std::string& function,
                                           const base::DictionaryValue* value);
 
   // Must only be accessed on the IO thread.
   std::vector<UpdateCallback> update_callbacks_;
   base::ListValue video_capture_capabilities_cached_data_;
 
   // All variables below must be accessed under |lock_|.
   base::Lock lock_;
   base::DictionaryValue audio_streams_cached_data_;
   int owner_ids_[AUDIO_COMPONENT_MAX];
 
   DISALLOW_COPY_AND_ASSIGN(MediaInternals);
 };
 
-} // namespace content
+}  // namespace content
 
 #endif  // CONTENT_BROWSER_MEDIA_MEDIA_INTERNALS_H_