util_test Clock.SleepNanoseconds: don’t set an upper bound for the amount of time spent sleeping

util/misc/clock_test.cc

 // Copyright 2014 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 43 matching lines @@
 
   SleepNanoseconds(nanoseconds);
 
   uint64_t end = ClockMonotonicNanoseconds();
   uint64_t diff = end - start;
 
   // |nanoseconds| is the lower bound for the actual amount of time spent
   // sleeping.
   EXPECT_GE(diff, nanoseconds);
 
-  // It’s difficult to set an upper bound for the time spent sleeping. Allow
-  // sleeps twice as long as requested, or sleeps a millisecond longer than
-  // requested, whichever is larger. This is quite a lot of slop, but the
-  // alternative would be test flakiness.
-  uint64_t slop = std::max(static_cast<uint64_t>(1E6), nanoseconds);
-  EXPECT_LE(diff, nanoseconds + slop);
+  // It’s difficult to set an upper bound for the time spent sleeping, and
+  // attempting to do so results in a flaky test.
 }
 
 TEST(Clock, SleepNanoseconds) {
   const uint64_t kTestData[] = {
       0,
       1,
       static_cast<uint64_t>(1E3),  // 1 microsecond
       static_cast<uint64_t>(1E4),  // 10 microseconds
       static_cast<uint64_t>(1E5),  // 100 microseconds
       static_cast<uint64_t>(1E6),  // 1 millisecond
       static_cast<uint64_t>(1E7),  // 10 milliseconds
       static_cast<uint64_t>(2E7),  // 20 milliseconds
       static_cast<uint64_t>(5E7),  // 50 milliseconds
   };
 
   for (size_t index = 0; index < arraysize(kTestData); ++index) {
     const uint64_t nanoseconds = kTestData[index];
     SCOPED_TRACE(
         base::StringPrintf("index %zu, nanoseconds %llu", index, nanoseconds));
 
     TestSleepNanoseconds(nanoseconds);
   }
 }
 
 }  // namespace
 }  // namespace test
 }  // namespace crashpad