Separated BluetoothServiceRecord interface from BluetoothServiceRecordChromeOs.

device/bluetooth/bluetooth_device_chromeos.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 "device/bluetooth/bluetooth_device_chromeos.h"
 
 #include <map>
 #include <string>
 #include <vector>
 
@@ skipping 10 matching lines @@
 #include "chromeos/dbus/bluetooth_device_client.h"
 #include "chromeos/dbus/bluetooth_input_client.h"
 #include "chromeos/dbus/bluetooth_out_of_band_client.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
 #include "chromeos/dbus/introspectable_client.h"
 #include "dbus/bus.h"
 #include "dbus/object_path.h"
 #include "device/bluetooth/bluetooth_adapter_chromeos.h"
 #include "device/bluetooth/bluetooth_out_of_band_pairing_data.h"
 #include "device/bluetooth/bluetooth_service_record.h"
+#include "device/bluetooth/bluetooth_service_record_chromeos.h"
 #include "device/bluetooth/bluetooth_socket_chromeos.h"
 #include "device/bluetooth/bluetooth_utils.h"
 #include "third_party/cros_system_api/dbus/service_constants.h"
 
 using device::BluetoothDevice;
 using device::BluetoothOutOfBandPairingData;
 using device::BluetoothServiceRecord;
 using device::BluetoothSocket;
 
 namespace chromeos {
@@ skipping 356 matching lines @@
     bool success) {
   if (!success) {
     error_callback.Run();
     return;
   }
 
   ScopedVector<BluetoothServiceRecord> records;
   for (BluetoothDeviceClient::ServiceMap::const_iterator i =
       service_map.begin(); i != service_map.end(); ++i) {
     records.push_back(
-        new BluetoothServiceRecord(address(), i->second));
+        new BluetoothServiceRecordChromeOs(address(), i->second));

[miket_OOO, 2013/01/15 18:40:19]

FYI:

$ grep -oR ChromeOS * | wc -l
1124
$ grep -oR ChromeOs * | wc -l
221

I think you're technically in compliance with the style guide, but the masses do
seem to favor the other style. Either's OK with me.

[keybuk, 2013/01/15 19:42:00]

I agree, let's name this ChromeOS and fix the others as we go

[youngki, 2013/01/16 00:39:38]

Done.

   }
   callback.Run(records);
 }
 
 void BluetoothDeviceChromeOs::OnSetTrusted(const base::Closure& callback,
                                            const ErrorCallback& error_callback,
                                            bool success) {
   if (success) {
     callback.Run();
   } else {
@@ skipping 301 matching lines @@
 }
 
 
 // static
 BluetoothDeviceChromeOs* BluetoothDeviceChromeOs::Create(
     BluetoothAdapterChromeOs* adapter) {
   return new BluetoothDeviceChromeOs(adapter);
 }
 
 }  // namespace chromeos