Move weave packet to common location b/t generator and receiver

components/proximity_auth/ble/bluetooth_low_energy_weave_packet_generator.h

Index: components/proximity_auth/ble/bluetooth_low_energy_weave_packet_generator.h
diff --git a/components/proximity_auth/ble/bluetooth_low_energy_weave_packet_generator.h b/components/proximity_auth/ble/bluetooth_low_energy_weave_packet_generator.h
index eb3204ca91bab4fc900e096dfb2f576225f34363..e48f5d1ebd89b8a95fe4f24008b4e204cabf0d8e 100644
--- a/components/proximity_auth/ble/bluetooth_low_energy_weave_packet_generator.h
+++ b/components/proximity_auth/ble/bluetooth_low_energy_weave_packet_generator.h
@@ -12,9 +12,12 @@
 #include <string>
 #include <vector>
 
+#include "components/proximity_auth/ble/bluetooth_low_energy_weave_packet.h"
 #include "build/build_config.h"
 
 namespace proximity_auth {
+namespace weave {
+
 // Generates the messages sent using the uWeave protocol.
 class BluetoothLowEnergyWeavePacketGenerator {
  public:
@@ -35,28 +38,6 @@ class BluetoothLowEnergyWeavePacketGenerator {
     static Factory* factory_instance_;
   };
 
-  // Determine whether a packet is a control packet or a data packet.
-  enum PacketType { DATA = 0x00, CONTROL = 0x01 };
-
-  // Identify the action intended by the control packet.
-  enum ControlCommand {
-    CONNECTION_REQUEST = 0x00,
-    CONNECTION_RESPONSE = 0x01,
-    CONNECTION_CLOSE = 0x02
-  };
-
-  // Sent with the ConnectionClose control packet.
-  // Identify why the client/server wished to close the connection.
-  enum ReasonForClose {
-    CLOSE_WITHOUT_ERROR = 0x00,
-    UNKNOWN_ERROR = 0x01,
-    NO_COMMON_VERSION_SUPPORTED = 0x02,
-    RECEIVED_PACKET_OUT_OF_SEQUENCE = 0x03,
-    APPLICATION_ERROR = 0x80
-  };
-
-  typedef std::vector<uint8_t> Packet;
-
   Packet CreateConnectionRequest();
   Packet CreateConnectionResponse();
   Packet CreateConnectionClose(ReasonForClose reason_for_close);
@@ -80,12 +61,13 @@ class BluetoothLowEnergyWeavePacketGenerator {
 
   // The default max packet length is 20 unless SetDataPacketLength() is called
   // and specified otherwise.
-  uint32_t max_packet_size_;
+  uint16_t max_packet_size_;

[Kyle Horimoto, 2016/06/27 22:54:17]

If you're changing this, you also have to change SetMaxPacketSize() to take a
uint16_t. But you should do this in another CL as this one is unrelated.

[jingxuy, 2016/06/27 23:06:28]

good point, I overlooked it. I might have to do it in this CL as the constants I
am using is uint16_t and if this is not matched, it might break on a windows
machine. I think it broke earlier because of a uint16 to uint8 mismatch though
that one is implicitly casting from uint16 to uint8, so truncating. I am not
sure whether implicit elogation is allowed on window.

[Kyle Horimoto, 2016/06/27 23:09:14]

Alright, go ahead and do it this CL then.

[jingxuy, 2016/06/27 23:13:10]

Done.

 
   // Counter for the number of packets sent, starting at 0.
-  uint32_t next_packet_counter_;
+  uint8_t next_packet_counter_;
 };
 
+}  // namespace weave
 }  // namespace proximity_auth
 
 #endif  // COMPONENTS_PROXIMITY_AUTH_BLE_BLUETOOTH_LOW_ENERGY_WEAVE_PACKET_GENERATOR_H_