Refactoring bluetooth API code.

chrome/browser/chromeos/bluetooth/bluetooth_service_record_dbus.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 CHROME_BROWSER_CHROMEOS_BLUETOOTH_BLUETOOTH_SERVICE_RECORD_H_
-#define CHROME_BROWSER_CHROMEOS_BLUETOOTH_BLUETOOTH_SERVICE_RECORD_H_
+#ifndef CHROME_BROWSER_CHROMEOS_BLUETOOTH_BLUETOOTH_SERVICE_RECORD_DBUS_H_
+#define CHROME_BROWSER_CHROMEOS_BLUETOOTH_BLUETOOTH_SERVICE_RECORD_DBUS_H_
 
 #include <string>
 
 #include "base/basictypes.h"
+#include "base/compiler_specific.h"
+#include "chrome/browser/chromeos/bluetooth/bluetooth_service_record_interface.h"
 
 class XmlReader;
 
 namespace chromeos {
 
-// The BluetoothServiceRecord represents an SDP service record.
+// The BluetoothServiceRecordDBus class is an implementation of
+// BluetoothServiceRecordInterface using DBus for Linux/ChromeOS platform.

[bryeung, 2012/09/07 18:35:57]

I think that the XML format is actually dictated by the Bluetooth Spec (i.e.
this might actually be platform independent).

Please verify that this is true, and just leave a comment to the XML spec here
(if you can find one).  (And obviously undo the renaming.)

[youngki, 2012/09/13 18:05:02]

I think the logic in this file is pretty much about processing XML data, so I
think we can reuse this in windows platform as well. I also verified that
third_party/libxml compiles in Windows too. I renamed this file back to
bluetooth_service_record.{h|cc} and also BluetoothServiceRecord as well.

 //
 // This implementation is currently incomplete: it only supports those fields
 // that have been necessary so far.
-class BluetoothServiceRecord {
+class BluetoothServiceRecordDBus : public BluetoothServiceRecordInterface {
  public:
-   BluetoothServiceRecord(
+   BluetoothServiceRecordDBus(
        const std::string& address,
        const std::string& xml_data);
 
    // The human-readable name of this service.
-   const std::string& name() const { return name_; }
+   virtual const std::string& name() const OVERRIDE;
+
+   // The UUID of the service.  This field may be empty if no UUID was
+   // specified in the service record.
+   virtual const std::string& uuid() const OVERRIDE;
 
    // The address of the BluetoothDevice providing this service.
    const std::string& address() const { return address_; }
 
-   // The UUID of the service.  This field may be empty if no UUID was
-   // specified in the service record.
-   const std::string& uuid() const { return uuid_; }
-
    // Indicates if this service supports RFCOMM communication.
    bool SupportsRfcomm() const { return supports_rfcomm_; }
 
    // The RFCOMM channel to use, if this service supports RFCOMM communication.
    // The return value is undefined if SupportsRfcomm() returns false.
    uint8_t rfcomm_channel() const { return rfcomm_channel_; }
 
  private:
   void ExtractChannels(XmlReader* reader);
   void ExtractUuid(XmlReader* reader);
 
   std::string address_;
   std::string name_;
   std::string uuid_;
 
   bool supports_rfcomm_;
   uint8_t rfcomm_channel_;
 
-  DISALLOW_COPY_AND_ASSIGN(BluetoothServiceRecord);
+  DISALLOW_COPY_AND_ASSIGN(BluetoothServiceRecordDBus);
 };
 
 }  // namespace chromeos
 
-#endif  // CHROME_BROWSER_CHROMEOS_BLUETOOTH_BLUETOOTH_SERVICE_RECORD_H_
+#endif  // CHROME_BROWSER_CHROMEOS_BLUETOOTH_BLUETOOTH_SERVICE_RECORD_DBUS_H_