Clean-up inline members of nested classes (base/)

base/system_monitor/system_monitor.cc

Index: base/system_monitor/system_monitor.cc
diff --git a/base/system_monitor/system_monitor.cc b/base/system_monitor/system_monitor.cc
index d1dd4391cc4a2b68871b4dd77ab0fc4f15e9d529..c9b84852f22ce1c5603855a8ea8acde0dc0e4502 100644
--- a/base/system_monitor/system_monitor.cc
+++ b/base/system_monitor/system_monitor.cc
@@ -50,6 +50,17 @@ SystemMonitor::~SystemMonitor() {
   g_system_monitor = NULL;
 }
 
+SystemMonitor::MediaDeviceInfo::MediaDeviceInfo(

[jar (doing other things), 2012/08/09 16:30:27]

nit: Since the declaration of this method in the nested class came before the
constructor in the surrounding class, the definition should be in that order.
(i.e., move this declaration to in front of line 25)

It seems a little weird... but they insisted on the type (and hence nested
classes) coming first, and so the corresponding definitions need to come first.

[hans, 2012/08/09 18:28:48]

Done.

+    const std::string& id,
+    const string16& device_name,
+    MediaDeviceType device_type,
+    const FilePath::StringType& device_location)
+    : unique_id(id),
+      name(device_name),
+      type(device_type),
+      location(device_location) {
+}
+
 // static
 SystemMonitor* SystemMonitor::Get() {
   return g_system_monitor;