Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: test/UpdateCheck.cpp

Issue 29367003: Issue #4711 - Rewrite legacy thread facilities
Patch Set: Created Dec. 7, 2016, 4:44 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« src/Thread.h ('K') | « test/Thread.cpp ('k') | test/WebRequest.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/UpdateCheck.cpp
===================================================================
--- a/test/UpdateCheck.cpp
+++ b/test/UpdateCheck.cpp
@@ -113,7 +113,7 @@
Reset();
ForceUpdateCheck();
- AdblockPlus::Sleep(100);
+ std::this_thread::sleep_for(std::chrono::milliseconds(100));
ASSERT_FALSE(eventCallbackCalled);
ASSERT_TRUE(updateCallbackCalled);
@@ -150,7 +150,7 @@
Reset();
ForceUpdateCheck();
- AdblockPlus::Sleep(100);
+ std::this_thread::sleep_for(std::chrono::milliseconds(100));
ASSERT_TRUE(eventCallbackCalled);
ASSERT_EQ(1u, eventCallbackParams.size());
@@ -189,7 +189,7 @@
Reset();
ForceUpdateCheck();
- AdblockPlus::Sleep(100);
+ std::this_thread::sleep_for(std::chrono::milliseconds(100));
ASSERT_TRUE(eventCallbackCalled);
ASSERT_EQ(1u, eventCallbackParams.size());
@@ -213,7 +213,7 @@
Reset();
ForceUpdateCheck();
- AdblockPlus::Sleep(100);
+ std::this_thread::sleep_for(std::chrono::milliseconds(100));
ASSERT_FALSE(eventCallbackCalled);
ASSERT_TRUE(updateCallbackCalled);
@@ -235,7 +235,7 @@
Reset();
ForceUpdateCheck();
- AdblockPlus::Sleep(100);
+ std::this_thread::sleep_for(std::chrono::milliseconds(100));
ASSERT_FALSE(eventCallbackCalled);
ASSERT_TRUE(updateCallbackCalled);
@@ -257,7 +257,7 @@
Reset();
ForceUpdateCheck();
- AdblockPlus::Sleep(100);
+ std::this_thread::sleep_for(std::chrono::milliseconds(100));
ASSERT_FALSE(eventCallbackCalled);
ASSERT_TRUE(updateCallbackCalled);
« src/Thread.h ('K') | « test/Thread.cpp ('k') | test/WebRequest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld