device/bluetooth: split out transport enum

device/bluetooth/bluetooth_common.h

Index: device/bluetooth/bluetooth_common.h
diff --git a/device/bluetooth/bluetooth_common.h b/device/bluetooth/bluetooth_common.h
new file mode 100644
index 0000000000000000000000000000000000000000..ae4e13bf06ee54e8160ee2ae4b361404d8f63180
--- /dev/null
+++ b/device/bluetooth/bluetooth_common.h
@@ -0,0 +1,29 @@
+// Copyright 2016 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 DEVICE_BLUETOOTH_BLUETOOTH_TYPES_H_
+#define DEVICE_BLUETOOTH_BLUETOOTH_TYPES_H_
+
+#include "device/bluetooth/bluetooth_export.h"
+
+// This file is for enums and small types common to several
+// parts of bluetooth.
+
+namespace device {
+
+// Devices and adapters can support a number of transports,
+// and bluetooth hosts can scan for devices based on the
+// transports they support.
+enum BluetoothTransport : uint8_t {

[Eric Caruso, 2016/06/15 22:32:10]

As per offline discussion with ortuno@, using enum class here would prevent us
from neatly manipulating the transport values like a bitset. I proposed going
back to a regular unscoped enum or using a std::bitset and we chose to go with
the former.

Because of this, I changed the names of the values to be a little more in line
with the style of unscoped enums.

+  BLUETOOTH_TRANSPORT_INVALID = 0x00,
+  // Valid transports are given as a bitset.
+  BLUETOOTH_TRANSPORT_CLASSIC = 0x01,
+  BLUETOOTH_TRANSPORT_LE = 0x02,
+  BLUETOOTH_TRANSPORT_DUAL =
+      (BLUETOOTH_TRANSPORT_CLASSIC | BLUETOOTH_TRANSPORT_LE)
+};
+
+}  // namespace device
+
+#endif  // DEVICE_BLUETOOTH_BLUETOOTH_TYPES_H_