Add SystemMonitor::GetMediaDevices()

chrome/browser/media_gallery/media_device_manager.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.
+
+// MediaDeviceManager keeps track of media devices and makes them available to
+// interested parties like the Media Gallery API.
+
+#ifndef CHROME_BROWSER_MEDIA_GALLERY_MEDIA_DEVICE_MANAGER_H_
+#define CHROME_BROWSER_MEDIA_GALLERY_MEDIA_DEVICE_MANAGER_H_
+#pragma once
+
+#include <string>
+#include <utility>
+#include <vector>
+
+#include "base/basictypes.h"
+#include "base/memory/scoped_ptr.h"
+
+class FilePath;
+
+namespace chrome {
+
+class MediaDeviceManager {
+ public:
+  typedef std::pair<std::string, FilePath> MediaDeviceInfo;
+
+  MediaDeviceManager();
+  ~MediaDeviceManager();
+
+  // Returns a vector of media devices in |results|.
+  void GetDevices(std::vector<MediaDeviceInfo>* results);

[vandebo (ex-Chrome), 2012/05/17 21:17:08]

I was going to ask why this had an separate implementation, but then... why not
just add a GetDevices method to base/SystemMonitor?  It's already maintaining
that list. 

+
+ private:
+  class MediaDeviceManagerImpl;
+
+  scoped_ptr<MediaDeviceManagerImpl> impl_;
+
+  DISALLOW_COPY_AND_ASSIGN(MediaDeviceManager);
+};
+
+}  // namespace chrome
+
+#endif  // CHROME_BROWSER_MEDIA_GALLERY_MEDIA_DEVICE_MANAGER_H_