Use 64 bit shift when convert uint64_t to ByteVector

components/rappor/byte_vector_utils.h

 // Copyright 2014 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 COMPONENTS_RAPPOR_BYTE_VECTOR_UTILS_H_
 #define COMPONENTS_RAPPOR_BYTE_VECTOR_UTILS_H_
 
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/macros.h"
 #include "components/rappor/rappor_parameters.h"
 #include "crypto/hmac.h"
 
 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);
 
 // Computes a bitwise OR of byte vectors and stores the result in rhs.
 // Returns rhs for chaining.
 ByteVector* ByteVectorOr(const ByteVector& lhs, ByteVector* rhs);
 
 // Merges the contents of lhs and rhs  vectors according to a mask vector.
 // The i-th bit of the result vector will be the i-th bit of either the lhs
@@ skipping 76 matching lines @@
 
   // Total number of bytes streamed from the HMAC_DRBG Generate Process.
   size_t generated_bytes_;
 
   DISALLOW_ASSIGN(HmacByteVectorGenerator);
 };
 
 }  // namespace rappor
 
 #endif  // COMPONENTS_RAPPOR_BYTE_VECTOR_UTILS_H_