Title: [228926] trunk
Revision
228926
Author
utatane....@gmail.com
Date
2018-02-22 10:54:07 -0800 (Thu, 22 Feb 2018)

Log Message

Remove sleep(double) and sleepMS(double) interfaces
https://bugs.webkit.org/show_bug.cgi?id=183038

Reviewed by Mark Lam.

Source/_javascript_Core:

* bytecode/SuperSampler.cpp:
(JSC::initializeSuperSampler):

Source/WebKit:

* PluginProcess/WebProcessConnection.cpp:
(WebKit::WebProcessConnection::createPluginAsynchronously):
* UIProcess/linux/MemoryPressureMonitor.cpp:
(WebKit::pollIntervalForUsedMemoryPercentage):
(WebKit::MemoryPressureMonitor::MemoryPressureMonitor):
* WebProcess/wpe/WebProcessMainWPE.cpp:

Source/WTF:

This patch removes sleep(double) and sleepMS(double) interfaces.
We can just use sleep(Seconds) instead.

* benchmarks/LockFairnessTest.cpp:
* benchmarks/LockSpeedTest.cpp:
* wtf/CurrentTime.cpp:
(WTF::sleep):
* wtf/CurrentTime.h:
(WTF::sleepMS): Deleted.
* wtf/DebugUtilities.h:
* wtf/Seconds.cpp:
(WTF::sleep): Deleted.
* wtf/Seconds.h:
* wtf/StackShotProfiler.h:

Tools:

* DumpRenderTree/TestNetscapePlugIn/PluginTest.cpp:
(PluginTest::indicateTestFailure):
* DumpRenderTree/TestNetscapePlugIn/Tests/EvaluateJSWithinNPP_New.cpp:
(EvaluteJSWithinNPP_New::NPP_New):
* DumpRenderTree/TestNetscapePlugIn/Tests/InvokeDestroysPluginWithinNPP_New.cpp:
(InvokeDestroysPluginWithinNPP_New::NPP_New):
* DumpRenderTree/TestNetscapePlugIn/Tests/SlowNPPNew.cpp:
* TestWebKitAPI/Tests/WTF/Signals.cpp:
(TEST):
* TestWebKitAPI/Tests/WTF/ThreadGroup.cpp:
(TestWebKitAPI::testThreadGroup):
* TestWebKitAPI/Tests/WTF/WorkQueue.cpp:
(TestWebKitAPI::TEST):
* TestWebKitAPI/Tests/WebKitCocoa/PictureInPictureDelegate.mm:
(TestWebKitAPI::TEST):
* TestWebKitAPI/Tests/ios/DataInteractionTests.mm:
(TestWebKitAPI::TEST):
* TestWebKitAPI/jsconly/PlatformUtilitiesJSCOnly.cpp:
(TestWebKitAPI::Util::sleep):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (228925 => 228926)


--- trunk/Source/_javascript_Core/ChangeLog	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Source/_javascript_Core/ChangeLog	2018-02-22 18:54:07 UTC (rev 228926)
@@ -1,3 +1,13 @@
+2018-02-22  Yusuke Suzuki  <utatane....@gmail.com>
+
+        Remove sleep(double) and sleepMS(double) interfaces
+        https://bugs.webkit.org/show_bug.cgi?id=183038
+
+        Reviewed by Mark Lam.
+
+        * bytecode/SuperSampler.cpp:
+        (JSC::initializeSuperSampler):
+
 2018-02-21  Don Olmstead  <don.olmst...@sony.com>
 
         [CMake] Split declaration of JSC headers into public and private

Modified: trunk/Source/_javascript_Core/bytecode/SuperSampler.cpp (228925 => 228926)


--- trunk/Source/_javascript_Core/bytecode/SuperSampler.cpp	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Source/_javascript_Core/bytecode/SuperSampler.cpp	2018-02-22 18:54:07 UTC (rev 228926)
@@ -60,7 +60,7 @@
                         else
                             out++;
                     }
-                    sleepMS(sleepQuantum);
+                    sleep(Seconds::fromMilliseconds(sleepQuantum));
                 }
                 printSuperSamplerState();
                 if (static_cast<int32_t>(g_superSamplerCount) < 0)

Modified: trunk/Source/WTF/ChangeLog (228925 => 228926)


--- trunk/Source/WTF/ChangeLog	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Source/WTF/ChangeLog	2018-02-22 18:54:07 UTC (rev 228926)
@@ -1,3 +1,25 @@
+2018-02-22  Yusuke Suzuki  <utatane....@gmail.com>
+
+        Remove sleep(double) and sleepMS(double) interfaces
+        https://bugs.webkit.org/show_bug.cgi?id=183038
+
+        Reviewed by Mark Lam.
+
+        This patch removes sleep(double) and sleepMS(double) interfaces.
+        We can just use sleep(Seconds) instead.
+
+        * benchmarks/LockFairnessTest.cpp:
+        * benchmarks/LockSpeedTest.cpp:
+        * wtf/CurrentTime.cpp:
+        (WTF::sleep):
+        * wtf/CurrentTime.h:
+        (WTF::sleepMS): Deleted.
+        * wtf/DebugUtilities.h:
+        * wtf/Seconds.cpp:
+        (WTF::sleep): Deleted.
+        * wtf/Seconds.h:
+        * wtf/StackShotProfiler.h:
+
 2018-02-21  Don Olmstead  <don.olmst...@sony.com>
 
         [CMake][Win] Use cmakeconfig.h rather than config.h and Platform.h

Modified: trunk/Source/WTF/benchmarks/LockFairnessTest.cpp (228925 => 228926)


--- trunk/Source/WTF/benchmarks/LockFairnessTest.cpp	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Source/WTF/benchmarks/LockFairnessTest.cpp	2018-02-22 18:54:07 UTC (rev 228926)
@@ -92,10 +92,10 @@
                 });
         }
     
-        sleepMS(100);
+        sleep(100_ms);
         lock.unlock();
     
-        sleep(secondsPerTest);
+        sleep(Seconds { secondsPerTest });
     
         keepGoing = false;
         lock.lock();

Modified: trunk/Source/WTF/benchmarks/LockSpeedTest.cpp (228925 => 228926)


--- trunk/Source/WTF/benchmarks/LockSpeedTest.cpp	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Source/WTF/benchmarks/LockSpeedTest.cpp	2018-02-22 18:54:07 UTC (rev 228926)
@@ -115,7 +115,7 @@
             }
         }
 
-        sleep(secondsPerTest);
+        sleep(Seconds { secondsPerTest });
         keepGoing = false;
     
         for (unsigned threadIndex = numThreadGroups * numThreadsPerGroup; threadIndex--;)

Modified: trunk/Source/WTF/wtf/CurrentTime.cpp (228925 => 228926)


--- trunk/Source/WTF/wtf/CurrentTime.cpp	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Source/WTF/wtf/CurrentTime.cpp	2018-02-22 18:54:07 UTC (rev 228926)
@@ -327,7 +327,7 @@
 #endif
 }
 
-void sleep(double value)
+void sleep(Seconds value)
 {
     // It's very challenging to find portable ways of sleeping for less than a second. On UNIX, you want to
     // use usleep() but it's hard to #include it in a portable way (you'd think it's in unistd.h, but then
@@ -337,7 +337,7 @@
     Lock fakeLock;
     Condition fakeCondition;
     LockHolder fakeLocker(fakeLock);
-    fakeCondition.waitFor(fakeLock, Seconds(value));
+    fakeCondition.waitFor(fakeLock, value);
 }
 
 } // namespace WTF

Modified: trunk/Source/WTF/wtf/CurrentTime.h (228925 => 228926)


--- trunk/Source/WTF/wtf/CurrentTime.h	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Source/WTF/wtf/CurrentTime.h	2018-02-22 18:54:07 UTC (rev 228926)
@@ -66,13 +66,8 @@
 // than a millisecond.
 WTF_EXPORT_PRIVATE Seconds currentCPUTime();
 
-WTF_EXPORT_PRIVATE void sleep(double);
+WTF_EXPORT_PRIVATE void sleep(Seconds);
 
-inline void sleepMS(double value)
-{
-    sleep(value / 1000.0);
-}
-
 } // namespace WTF
 
 using WTF::currentCPUTime;
@@ -81,6 +76,5 @@
 using WTF::monotonicallyIncreasingTime;
 using WTF::monotonicallyIncreasingTimeMS;
 using WTF::sleep;
-using WTF::sleepMS;
 
 #endif // CurrentTime_h

Modified: trunk/Source/WTF/wtf/DebugUtilities.h (228925 => 228926)


--- trunk/Source/WTF/wtf/DebugUtilities.h	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Source/WTF/wtf/DebugUtilities.h	2018-02-22 18:54:07 UTC (rev 228926)
@@ -33,7 +33,7 @@
 #define SLEEP_THREAD_FOR_DEBUGGER() \
 do { \
     do { \
-        sleep(1.0); \
+        sleep(1_s); \
         if (WTFIsDebuggerAttached()) \
             break; \
     } while (1); \

Modified: trunk/Source/WTF/wtf/Seconds.cpp (228925 => 228926)


--- trunk/Source/WTF/wtf/Seconds.cpp	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Source/WTF/wtf/Seconds.cpp	2018-02-22 18:54:07 UTC (rev 228926)
@@ -69,10 +69,5 @@
     out.print(m_value, " sec");
 }
 
-void sleep(Seconds value)
-{
-    sleep(value.value());
-}
-
 } // namespace WTF
 

Modified: trunk/Source/WTF/wtf/Seconds.h (228925 => 228926)


--- trunk/Source/WTF/wtf/Seconds.h	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Source/WTF/wtf/Seconds.h	2018-02-22 18:54:07 UTC (rev 228926)
@@ -282,8 +282,6 @@
 
 } // inline seconds_literals
 
-WTF_EXPORT_PRIVATE void sleep(Seconds);
-
 } // namespace WTF
 
 namespace std {

Modified: trunk/Source/WTF/wtf/StackShotProfiler.h (228925 => 228926)


--- trunk/Source/WTF/wtf/StackShotProfiler.h	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Source/WTF/wtf/StackShotProfiler.h	2018-02-22 18:54:07 UTC (rev 228926)
@@ -25,6 +25,7 @@
 
 #pragma once
 
+#include <wtf/CurrentTime.h>
 #include <wtf/DataLog.h>
 #include <wtf/Locker.h>
 #include <wtf/ProcessID.h>

Modified: trunk/Source/WebKit/ChangeLog (228925 => 228926)


--- trunk/Source/WebKit/ChangeLog	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Source/WebKit/ChangeLog	2018-02-22 18:54:07 UTC (rev 228926)
@@ -1,3 +1,17 @@
+2018-02-22  Yusuke Suzuki  <utatane....@gmail.com>
+
+        Remove sleep(double) and sleepMS(double) interfaces
+        https://bugs.webkit.org/show_bug.cgi?id=183038
+
+        Reviewed by Mark Lam.
+
+        * PluginProcess/WebProcessConnection.cpp:
+        (WebKit::WebProcessConnection::createPluginAsynchronously):
+        * UIProcess/linux/MemoryPressureMonitor.cpp:
+        (WebKit::pollIntervalForUsedMemoryPercentage):
+        (WebKit::MemoryPressureMonitor::MemoryPressureMonitor):
+        * WebProcess/wpe/WebProcessMainWPE.cpp:
+
 2018-02-22  Youenn Fablet  <you...@apple.com>
 
         Add release logging for CacheStorage::Engine disk related functions

Modified: trunk/Source/WebKit/PluginProcess/WebProcessConnection.cpp (228925 => 228926)


--- trunk/Source/WebKit/PluginProcess/WebProcessConnection.cpp	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Source/WebKit/PluginProcess/WebProcessConnection.cpp	2018-02-22 18:54:07 UTC (rev 228926)
@@ -284,7 +284,7 @@
     uint32_t remoteLayerClientID = 0;
     
     if (creationParameters.artificialPluginInitializationDelayEnabled) {
-        unsigned artificialPluginInitializationDelay = 5;
+        Seconds artificialPluginInitializationDelay { 5_s };
         sleep(artificialPluginInitializationDelay);
     }
 

Modified: trunk/Source/WebKit/UIProcess/linux/MemoryPressureMonitor.cpp (228925 => 228926)


--- trunk/Source/WebKit/UIProcess/linux/MemoryPressureMonitor.cpp	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Source/WebKit/UIProcess/linux/MemoryPressureMonitor.cpp	2018-02-22 18:54:07 UTC (rev 228926)
@@ -46,8 +46,8 @@
 
 static const size_t notSet = static_cast<size_t>(-1);
 
-static const double s_minPollingIntervalInSeconds = 1;
-static const double s_maxPollingIntervalInSeconds = 5;
+static const Seconds s_minPollingInterval { 1_s };
+static const Seconds s_maxPollingInterval { 5_s };
 static const double s_minUsedMemoryPercentageForPolling = 50;
 static const double s_maxUsedMemoryPercentageForPolling = 90;
 static const int s_memoryPresurePercentageThreshold = 95;
@@ -198,17 +198,17 @@
     return ((memoryTotal - memoryAvailable) * 100) / memoryTotal;
 }
 
-static inline double pollIntervalForUsedMemoryPercentage(int usedPercentage)
+static inline Seconds pollIntervalForUsedMemoryPercentage(int usedPercentage)
 {
     // Use a different poll interval depending on the currently memory used,
     // to avoid polling too often when the system is under low memory usage.
     if (usedPercentage < s_minUsedMemoryPercentageForPolling)
-        return s_maxPollingIntervalInSeconds;
+        return s_maxPollingInterval;
 
     if (usedPercentage >= s_maxUsedMemoryPercentageForPolling)
-        return s_minPollingIntervalInSeconds;
+        return s_minPollingInterval;
 
-    return s_minPollingIntervalInSeconds + (s_maxPollingIntervalInSeconds - s_minPollingIntervalInSeconds) *
+    return s_minPollingInterval + (s_maxPollingInterval - s_minPollingInterval) *
         ((usedPercentage - s_minUsedMemoryPercentageForPolling) / (s_maxUsedMemoryPercentageForPolling - s_minUsedMemoryPercentageForPolling));
 }
 
@@ -249,7 +249,7 @@
         return;
 
     Thread::create("MemoryPressureMonitor", [this] {
-        double pollInterval = s_maxPollingIntervalInSeconds;
+        Seconds pollInterval = s_maxPollingInterval;
         while (true) {
             sleep(pollInterval);
 

Modified: trunk/Source/WebKit/WebProcess/wpe/WebProcessMainWPE.cpp (228925 => 228926)


--- trunk/Source/WebKit/WebProcess/wpe/WebProcessMainWPE.cpp	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Source/WebKit/WebProcess/wpe/WebProcessMainWPE.cpp	2018-02-22 18:54:07 UTC (rev 228926)
@@ -44,7 +44,7 @@
     {
 #if ENABLE(DEVELOPER_MODE)
         if (g_getenv("WEBKIT2_PAUSE_WEB_PROCESS_ON_LAUNCH"))
-            WTF::sleep(30);
+            WTF::sleep(30_s);
 #endif
 
         return true;

Modified: trunk/Tools/ChangeLog (228925 => 228926)


--- trunk/Tools/ChangeLog	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Tools/ChangeLog	2018-02-22 18:54:07 UTC (rev 228926)
@@ -1,3 +1,30 @@
+2018-02-22  Yusuke Suzuki  <utatane....@gmail.com>
+
+        Remove sleep(double) and sleepMS(double) interfaces
+        https://bugs.webkit.org/show_bug.cgi?id=183038
+
+        Reviewed by Mark Lam.
+
+        * DumpRenderTree/TestNetscapePlugIn/PluginTest.cpp:
+        (PluginTest::indicateTestFailure):
+        * DumpRenderTree/TestNetscapePlugIn/Tests/EvaluateJSWithinNPP_New.cpp:
+        (EvaluteJSWithinNPP_New::NPP_New):
+        * DumpRenderTree/TestNetscapePlugIn/Tests/InvokeDestroysPluginWithinNPP_New.cpp:
+        (InvokeDestroysPluginWithinNPP_New::NPP_New):
+        * DumpRenderTree/TestNetscapePlugIn/Tests/SlowNPPNew.cpp:
+        * TestWebKitAPI/Tests/WTF/Signals.cpp:
+        (TEST):
+        * TestWebKitAPI/Tests/WTF/ThreadGroup.cpp:
+        (TestWebKitAPI::testThreadGroup):
+        * TestWebKitAPI/Tests/WTF/WorkQueue.cpp:
+        (TestWebKitAPI::TEST):
+        * TestWebKitAPI/Tests/WebKitCocoa/PictureInPictureDelegate.mm:
+        (TestWebKitAPI::TEST):
+        * TestWebKitAPI/Tests/ios/DataInteractionTests.mm:
+        (TestWebKitAPI::TEST):
+        * TestWebKitAPI/jsconly/PlatformUtilitiesJSCOnly.cpp:
+        (TestWebKitAPI::Util::sleep):
+
 2018-02-21  Don Olmstead  <don.olmst...@sony.com>
 
         [CMake][Win] Use cmakeconfig.h rather than config.h and Platform.h

Modified: trunk/Tools/DumpRenderTree/TestNetscapePlugIn/PluginTest.cpp (228925 => 228926)


--- trunk/Tools/DumpRenderTree/TestNetscapePlugIn/PluginTest.cpp	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Tools/DumpRenderTree/TestNetscapePlugIn/PluginTest.cpp	2018-02-22 18:54:07 UTC (rev 228926)
@@ -28,6 +28,7 @@
 #include "PluginObject.h"
 #include <assert.h>
 #include <string.h>
+#include <wtf/CurrentTime.h>
 #include <wtf/Platform.h>
 #include <wtf/ExportMacros.h>
 
@@ -81,11 +82,7 @@
     // This should really be an assert, but there's no way for the test framework
     // to know that the plug-in process crashed, so we'll just sleep for a while
     // to ensure that the test times out.
-#if defined(XP_WIN)
-    ::Sleep(100000);
-#else
-    sleep(1000);
-#endif
+    sleep(1000_s);
 }
 
 NPError PluginTest::NPP_New(NPMIMEType pluginType, uint16_t mode, int16_t argc, char *argn[], char *argv[], NPSavedData *saved)

Modified: trunk/Tools/DumpRenderTree/TestNetscapePlugIn/Tests/EvaluateJSWithinNPP_New.cpp (228925 => 228926)


--- trunk/Tools/DumpRenderTree/TestNetscapePlugIn/Tests/EvaluateJSWithinNPP_New.cpp	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Tools/DumpRenderTree/TestNetscapePlugIn/Tests/EvaluateJSWithinNPP_New.cpp	2018-02-22 18:54:07 UTC (rev 228926)
@@ -26,6 +26,7 @@
 #include "PluginTest.h"
 
 #include "PluginObject.h"
+#include <wtf/CurrentTime.h>
 
 using namespace std;
 
@@ -48,7 +49,7 @@
 NPError EvaluteJSWithinNPP_New::NPP_New(NPMIMEType pluginType, uint16_t mode, int16_t argc, char* argn[], char* argv[], NPSavedData *saved)
 {
     // Give the WebProcess enough time to be deadlocked waiting for the PluginProcess.
-    usleep(15000);
+    sleep(15_ms);
     executeScript("var theLocation = window.location;");
     return NPERR_NO_ERROR;
 }

Modified: trunk/Tools/DumpRenderTree/TestNetscapePlugIn/Tests/InvokeDestroysPluginWithinNPP_New.cpp (228925 => 228926)


--- trunk/Tools/DumpRenderTree/TestNetscapePlugIn/Tests/InvokeDestroysPluginWithinNPP_New.cpp	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Tools/DumpRenderTree/TestNetscapePlugIn/Tests/InvokeDestroysPluginWithinNPP_New.cpp	2018-02-22 18:54:07 UTC (rev 228926)
@@ -26,6 +26,7 @@
 #include "PluginTest.h"
 
 #include "PluginObject.h"
+#include <wtf/CurrentTime.h>
 
 using namespace std;
 
@@ -48,7 +49,7 @@
 NPError InvokeDestroysPluginWithinNPP_New::NPP_New(NPMIMEType pluginType, uint16_t mode, int16_t argc, char* argn[], char* argv[], NPSavedData *saved)
 {
     // Give the WebProcess enough time to be deadlocked waiting for the PluginProcess if things aren't working correctly.
-    usleep(15000);
+    sleep(15_ms);
     
     NPObject* windowObject = 0;
     if (NPN_GetValue(NPNVWindowNPObject, &windowObject) != NPERR_NO_ERROR)

Modified: trunk/Tools/DumpRenderTree/TestNetscapePlugIn/Tests/SlowNPPNew.cpp (228925 => 228926)


--- trunk/Tools/DumpRenderTree/TestNetscapePlugIn/Tests/SlowNPPNew.cpp	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Tools/DumpRenderTree/TestNetscapePlugIn/Tests/SlowNPPNew.cpp	2018-02-22 18:54:07 UTC (rev 228926)
@@ -26,6 +26,7 @@
 #include "PluginTest.h"
 
 #include <string.h>
+#include <wtf/CurrentTime.h>
 
 using namespace std;
 

Modified: trunk/Tools/TestWebKitAPI/Tests/WTF/Signals.cpp (228925 => 228926)


--- trunk/Tools/TestWebKitAPI/Tests/WTF/Signals.cpp	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Tools/TestWebKitAPI/Tests/WTF/Signals.cpp	2018-02-22 18:54:07 UTC (rev 228926)
@@ -26,6 +26,7 @@
 #include "config.h"
 
 #include <type_traits>
+#include <wtf/CurrentTime.h>
 #include <wtf/DataLog.h>
 #include <wtf/Threading.h>
 #include <wtf/threads/Signals.h>
@@ -57,7 +58,7 @@
         std::unique_lock<std::mutex> locker(static_cast<ReflectedThread&>(receiverThread.get()).m_mutex);
         receiverShouldKeepRunning.store(false);
         EXPECT_FALSE(static_cast<ReflectedThread&>(receiverThread.get()).hasExited());
-        sleep(1);
+        sleep(1_s);
         signalFired = !pthread_kill(static_cast<ReflectedThread&>(receiverThread.get()).m_handle, std::get<0>(toSystemSignal(Signal::Usr)));
     }
 

Modified: trunk/Tools/TestWebKitAPI/Tests/WTF/ThreadGroup.cpp (228925 => 228926)


--- trunk/Tools/TestWebKitAPI/Tests/WTF/ThreadGroup.cpp	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Tools/TestWebKitAPI/Tests/WTF/ThreadGroup.cpp	2018-02-22 18:54:07 UTC (rev 228926)
@@ -77,7 +77,7 @@
         }
 
         // While holding ThreadGroup lock, threads do not exit.
-        WTF::sleep(0.1);
+        WTF::sleep(100_ms);
         EXPECT_EQ(threadGroup->threads(threadGroupLocker).size(), numberOfThreads);
     }
     {

Modified: trunk/Tools/TestWebKitAPI/Tests/WTF/WorkQueue.cpp (228925 => 228926)


--- trunk/Tools/TestWebKitAPI/Tests/WTF/WorkQueue.cpp	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Tools/TestWebKitAPI/Tests/WTF/WorkQueue.cpp	2018-02-22 18:54:07 UTC (rev 228926)
@@ -231,7 +231,7 @@
         dispatchAfterTestCompleted.wait(lock, [&] {
             return completed;
         });
-        WTF::sleep(0.1);
+        WTF::sleep(100_ms);
     }
 }
 

Modified: trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/PictureInPictureDelegate.mm (228925 => 228926)


--- trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/PictureInPictureDelegate.mm	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/PictureInPictureDelegate.mm	2018-02-22 18:54:07 UTC (rev 228926)
@@ -41,6 +41,7 @@
 #import <WebKit/WKViewPrivate.h>
 #import <WebKit/WKWebViewConfigurationPrivate.h>
 #import <WebKit/WKWebViewPrivate.h>
+#import <wtf/CurrentTime.h>
 #import <wtf/RetainPtr.h>
 #import <wtf/mac/AppKitCompatibilityDeclarations.h>
 
@@ -137,7 +138,7 @@
     TestWebKitAPI::Util::run(&hasVideoInPictureInPictureCalled);
     ASSERT_TRUE(hasVideoInPictureInPictureValue);
     
-    sleep(1); // Wait for PIPAgent to launch, or it won't call -pipDidClose: callback.
+    sleep(1_s); // Wait for PIPAgent to launch, or it won't call -pipDidClose: callback.
     
     hasVideoInPictureInPictureCalled = false;
     [webView mouseDown:event];
@@ -185,7 +186,7 @@
     TestWebKitAPI::Util::run(&hasVideoInPictureInPictureCalled);
     ASSERT_TRUE(hasVideoInPictureInPictureValue);
     
-    sleep(1); // Wait for PIPAgent to launch, or it won't call -pipDidClose: callback.
+    sleep(1_s); // Wait for PIPAgent to launch, or it won't call -pipDidClose: callback.
     
     hasVideoInPictureInPictureCalled = false;
     webView.simulateButtonClick(kWKEventMouseButtonLeftButton, 5, 5, 0);

Modified: trunk/Tools/TestWebKitAPI/Tests/ios/DataInteractionTests.mm (228925 => 228926)


--- trunk/Tools/TestWebKitAPI/Tests/ios/DataInteractionTests.mm	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Tools/TestWebKitAPI/Tests/ios/DataInteractionTests.mm	2018-02-22 18:54:07 UTC (rev 228926)
@@ -40,6 +40,7 @@
 #import <WebKit/WKWebViewConfigurationPrivate.h>
 #import <WebKit/WebItemProviderPasteboard.h>
 #import <WebKit/_WKProcessPoolConfiguration.h>
+#import <wtf/CurrentTime.h>
 
 typedef void (^FileLoadCompletionBlock)(NSURL *, BOOL, NSError *);
 typedef void (^DataLoadCompletionBlock)(NSData *, NSError *);
@@ -1399,7 +1400,7 @@
     auto slowItem = adoptNS([[UIItemProvider alloc] init]);
     [slowItem registerDataRepresentationForTypeIdentifier:(NSString *)kUTTypeUTF8PlainText options:nil loadHandler:^NSProgress *(UIItemProviderDataLoadCompletionBlock completionBlock)
     {
-        sleep(2);
+        sleep(2_s);
         completionBlock([slowString dataUsingEncoding:NSUTF8StringEncoding], nil);
         return nil;
     }];

Modified: trunk/Tools/TestWebKitAPI/jsconly/PlatformUtilitiesJSCOnly.cpp (228925 => 228926)


--- trunk/Tools/TestWebKitAPI/jsconly/PlatformUtilitiesJSCOnly.cpp	2018-02-22 18:51:54 UTC (rev 228925)
+++ trunk/Tools/TestWebKitAPI/jsconly/PlatformUtilitiesJSCOnly.cpp	2018-02-22 18:54:07 UTC (rev 228926)
@@ -50,7 +50,7 @@
 
 void sleep(double seconds)
 {
-    WTF::sleep(seconds);
+    WTF::sleep(Seconds { seconds });
 }
 
 } // namespace Util
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to