Use an even better random number generation scheme in the prune test

client/prune_crash_reports_test.cc

 // Copyright 2015 The Crashpad Authors. All rights reserved.
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at
 //
 //     http://www.apache.org/licenses/LICENSE-2.0
 //
 // Unless required by applicable law or agreed to in writing, software
 // distributed under the License is distributed on an "AS IS" BASIS,
@@ skipping 190 matching lines @@
 
   std::vector<CrashReportDatabase::Report> reports;
   for (int i = 0; i < 10; ++i) {
     CrashReportDatabase::Report temp;
     temp.uuid.data_1 = i;
     temp.creation_time = NDaysAgo(i * 10);
     reports.push_back(temp);
   }
   // The randomness from std::rand() is not, so use a better rand() instead.
   const auto random_generator = [](int rand_max) {
-    return base::RandUint64() % rand_max;
+    return base::RandInt(0, rand_max - 1);
   };
   std::random_shuffle(reports.begin(), reports.end(), random_generator);
   std::vector<CrashReportDatabase::Report> pending_reports(
       reports.begin(), reports.begin() + 5);
   std::vector<CrashReportDatabase::Report> completed_reports(
       reports.begin() + 5, reports.end());
 
   MockDatabase db;
   EXPECT_CALL(db, GetPendingReports(_)).WillOnce(DoAll(
       SetArgPointee<0>(pending_reports),
       Return(CrashReportDatabase::kNoError)));
   EXPECT_CALL(db, GetCompletedReports(_)).WillOnce(DoAll(
       SetArgPointee<0>(completed_reports),
       Return(CrashReportDatabase::kNoError)));
   for (size_t i = 0; i < reports.size(); ++i) {
     EXPECT_CALL(db, DeleteReport(TestUUID(i)))
         .WillOnce(Return(CrashReportDatabase::kNoError));
   }
 
   StaticCondition delete_all(true);
   PruneCrashReportDatabase(&db, &delete_all);
 }
 
 }  // namespace
 }  // namespace test
 }  // namespace crashpad