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

Unified Diff: base/test/task_runner_test_template.h

Issue 146833008: Threading: Remove TaskRunnerHandlesNonZeroDelays from TaskRunnerTestDelegate (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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 | « base/test/sequenced_task_runner_test_template.h ('k') | base/threading/sequenced_worker_pool_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/test/task_runner_test_template.h
diff --git a/base/test/task_runner_test_template.h b/base/test/task_runner_test_template.h
index 2dd88144d6d5bf38a396ac1f32b62833a07d1282..73aa1f4a479f7dc7a6b99c83f96631f38695013d 100644
--- a/base/test/task_runner_test_template.h
+++ b/base/test/task_runner_test_template.h
@@ -30,11 +30,6 @@
// void StopTaskRunner() {
// ...
// }
-//
-// // Returns whether or not the task runner obeys non-zero delays.
-// bool TaskRunnerHandlesNonZeroDelays() const {
-// return true;
-// }
// };
//
// The TaskRunnerTest test harness will have a member variable of
@@ -141,11 +136,6 @@ TYPED_TEST_P(TaskRunnerTest, Basic) {
// Post a bunch of delayed tasks to the task runner. They should all
// complete.
TYPED_TEST_P(TaskRunnerTest, Delayed) {
- if (!this->delegate_.TaskRunnerHandlesNonZeroDelays()) {
- DLOG(INFO) << "This TaskRunner doesn't handle non-zero delays; skipping";
- return;
- }
-
std::map<int, int> expected_task_run_counts;
int expected_total_tasks = 0;
« no previous file with comments | « base/test/sequenced_task_runner_test_template.h ('k') | base/threading/sequenced_worker_pool_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698