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

Unified Diff: content/public/test/test_browser_thread_bundle.h

Issue 2628903002: Change DONT_START_THREADS to DONT_CREATE_THREADS in TestBrowserThreadBundle. (Closed)
Patch Set: CR gab #27 Created 3 years, 11 months 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 | « chrome/browser/io_thread_unittest.cc ('k') | content/public/test/test_browser_thread_bundle.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/public/test/test_browser_thread_bundle.h
diff --git a/content/public/test/test_browser_thread_bundle.h b/content/public/test/test_browser_thread_bundle.h
index 0138a72374111d0e6908965eb74e9bed50cda0e6..4a6b030c21f67b83e59c3cc0a4aacfaf06baf790 100644
--- a/content/public/test/test_browser_thread_bundle.h
+++ b/content/public/test/test_browser_thread_bundle.h
@@ -26,13 +26,11 @@
// REAL_IO_THREAD.
//
// For some tests it is important to emulate real browser startup. During real
-// browser startup some initialization is done (e.g. creation of thread objects)
-// between creating the main thread message loop, which is bound to the existing
-// main thread, and starting the other threads. Passing DONT_START_THREADS to
-// constructor will delay staring these other threads until the test explicitly
-// calls Start().
+// browser startup, the main MessageLoop is created before other threads.
+// Passing DONT_CREATE_THREADS to constructor will delay creating other threads
+// until the test explicitly calls CreateThreads().
//
-// DONT_START_THREADS should only be used when the options specify at least
+// DONT_CREATE_THREADS should only be used when the options specify at least
// one real thread other than the main thread.
#ifndef CONTENT_PUBLIC_TEST_TEST_BROWSER_THREAD_BUNDLE_H_
@@ -64,15 +62,15 @@ class TestBrowserThreadBundle {
REAL_DB_THREAD = 0x02,
REAL_FILE_THREAD = 0x08,
REAL_IO_THREAD = 0x10,
- DONT_START_THREADS = 0x20,
+ DONT_CREATE_THREADS = 0x20,
};
TestBrowserThreadBundle();
explicit TestBrowserThreadBundle(int options);
- // Start the real threads; should only be called from other classes if the
- // DONT_START_THREADS option was used when the bundle was created.
- void Start();
+ // Creates threads; should only be called from other classes if the
+ // DONT_CREATE_THREADS option was used when the bundle was created.
+ void CreateThreads();
~TestBrowserThreadBundle();
@@ -90,7 +88,7 @@ class TestBrowserThreadBundle {
std::unique_ptr<TestBrowserThread> io_thread_;
int options_;
- bool threads_started_;
+ bool threads_created_;
DISALLOW_COPY_AND_ASSIGN(TestBrowserThreadBundle);
};
« no previous file with comments | « chrome/browser/io_thread_unittest.cc ('k') | content/public/test/test_browser_thread_bundle.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698