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

base/system_monitor/system_monitor.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 "base/system_monitor/system_monitor.h"
 
 #include <utility>
 
 #include "base/logging.h"
 #include "base/message_loop.h"
@@ skipping 32 matching lines @@
 }
 
 SystemMonitor::~SystemMonitor() {
 #if defined(OS_MACOSX)
   PlatformDestroy();
 #endif
   DCHECK_EQ(this, g_system_monitor);
   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;
 }
 
 void SystemMonitor::ProcessPowerMessage(PowerEvent event_id) {
   // Suppress duplicate notifications.  Some platforms may
   // send multiple notifications of the same event.
   switch (event_id) {
     case POWER_STATE_EVENT:
@@ skipping 112 matching lines @@
 void SystemMonitor::NotifyResume() {
   DVLOG(1) << "Power Resuming";
   power_observer_list_->Notify(&PowerObserver::OnResume);
 }
 
 void SystemMonitor::BatteryCheck() {
   ProcessPowerMessage(SystemMonitor::POWER_STATE_EVENT);
 }
 
 }  // namespace base