Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3365)

Unified Diff: runtime/vm/thread_test.cc

Issue 1439483003: - Add an OSThread structure which is the generic TLS structure for all C++ (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: code-review Created 5 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « runtime/vm/thread_registry.cc ('k') | runtime/vm/timeline.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/thread_test.cc
diff --git a/runtime/vm/thread_test.cc b/runtime/vm/thread_test.cc
index 7eb5849d422f1c449aa65c8b91fa9a9767a5faba..2a8e8d7a6a039aaa4928959e007ea25bb04552e5 100644
--- a/runtime/vm/thread_test.cc
+++ b/runtime/vm/thread_test.cc
@@ -38,7 +38,7 @@ UNIT_TEST_CASE(Monitor) {
// This unit test case needs a running isolate.
Dart_CreateIsolate(
NULL, NULL, bin::isolate_snapshot_buffer, NULL, NULL, NULL);
- Thread* thread = Thread::Current();
+ OSThread* thread = OSThread::Current();
// Thread interrupter interferes with this test, disable interrupts.
thread->DisableThreadInterrupts();
Monitor* monitor = new Monitor();
@@ -410,18 +410,18 @@ TEST_CASE(ThreadIterator_Count) {
intptr_t thread_count_1 = 0;
{
- ThreadIterator ti;
+ OSThreadIterator ti;
while (ti.HasNext()) {
- Thread* thread = ti.Next();
+ OSThread* thread = ti.Next();
EXPECT(thread != NULL);
thread_count_0++;
}
}
{
- ThreadIterator ti;
+ OSThreadIterator ti;
while (ti.HasNext()) {
- Thread* thread = ti.Next();
+ OSThread* thread = ti.Next();
EXPECT(thread != NULL);
thread_count_1++;
}
@@ -434,8 +434,8 @@ TEST_CASE(ThreadIterator_Count) {
TEST_CASE(ThreadIterator_FindSelf) {
- Thread* current = Thread::Current();
- EXPECT(Thread::IsThreadInList(current->join_id()));
+ OSThread* current = OSThread::Current();
+ EXPECT(OSThread::IsThreadInList(current->join_id()));
}
@@ -446,16 +446,15 @@ struct ThreadIteratorTestParams {
void ThreadIteratorTestMain(uword parameter) {
- Thread::EnsureInit();
ThreadIteratorTestParams* params =
reinterpret_cast<ThreadIteratorTestParams*>(parameter);
- Thread* thread = Thread::Current();
+ OSThread* thread = OSThread::Current();
EXPECT(thread != NULL);
MonitorLocker ml(params->monitor);
params->spawned_thread_join_id = thread->join_id();
EXPECT(params->spawned_thread_join_id != OSThread::kInvalidThreadJoinId);
- EXPECT(Thread::IsThreadInList(thread->join_id()));
+ EXPECT(OSThread::IsThreadInList(thread->join_id()));
ml.Notify();
}
@@ -471,7 +470,9 @@ TEST_CASE(ThreadIterator_AddFindRemove) {
MonitorLocker ml(params.monitor);
EXPECT(params.spawned_thread_join_id == OSThread::kInvalidThreadJoinId);
// Spawn thread and wait to receive the thread join id.
- OSThread::Start(ThreadIteratorTestMain, reinterpret_cast<uword>(&params));
+ OSThread::Start("ThreadIteratorTest",
+ ThreadIteratorTestMain,
+ reinterpret_cast<uword>(&params));
while (params.spawned_thread_join_id == OSThread::kInvalidThreadJoinId) {
ml.Wait();
}
@@ -480,7 +481,7 @@ TEST_CASE(ThreadIterator_AddFindRemove) {
OSThread::Join(params.spawned_thread_join_id);
}
- EXPECT(!Thread::IsThreadInList(params.spawned_thread_join_id))
+ EXPECT(!OSThread::IsThreadInList(params.spawned_thread_join_id))
delete params.monitor;
}
« no previous file with comments | « runtime/vm/thread_registry.cc ('k') | runtime/vm/timeline.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698