Use 64 bit shift when convert uint64_t to ByteVector

components/rappor/byte_vector_utils.h

Index: components/rappor/byte_vector_utils.h
diff --git a/components/rappor/byte_vector_utils.h b/components/rappor/byte_vector_utils.h
index 3090d9bc76f5d33bac0d0100600b387477fd4775..19109f4aa454ec67877b01ed0c5fd4de20bd4579 100644
--- a/components/rappor/byte_vector_utils.h
+++ b/components/rappor/byte_vector_utils.h
@@ -17,6 +17,9 @@ namespace rappor {
 // A vector of 8-bit integers used to store a set of binary bits.
 typedef std::vector<uint8_t> ByteVector;
 
+// Converts the lowest |size| bytes of |value| into a ByteVector.
+void Uint64ToByteVector(uint64_t value, uint64_t size, ByteVector* output);

[Alexei Svitkine (slow), 2015/04/29 15:01:07]

Why is size a uint64_t? Should just be an int.

[Steven Holte, 2015/04/29 18:18:56]

Changed to size_t

+
 // Computes a bitwise AND of byte vectors and stores the result in rhs.
 // Returns rhs for chaining.
 ByteVector* ByteVectorAnd(const ByteVector& lhs, ByteVector* rhs);