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

Unified Diff: base/test/test_simple_task_runner.h

Issue 2331423002: Replace TestSimpleTaskRunner::GetPendingTasks with TakePendingTasks (Closed)
Patch Set: make a Location result base::Optional Created 4 years, 3 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 | « no previous file | base/test/test_simple_task_runner.cc » ('j') | extensions/common/one_shot_event_unittest.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/test/test_simple_task_runner.h
diff --git a/base/test/test_simple_task_runner.h b/base/test/test_simple_task_runner.h
index eb600f40c3a68c9396614537993f1468b138b0bb..f412e3a7d44f715af115d304df381bb2135d496d 100644
--- a/base/test/test_simple_task_runner.h
+++ b/base/test/test_simple_task_runner.h
@@ -9,6 +9,7 @@
#include "base/compiler_specific.h"
#include "base/macros.h"
+#include "base/optional.h"
#include "base/single_thread_task_runner.h"
#include "base/synchronization/lock.h"
#include "base/test/test_pending_task.h"
@@ -51,10 +52,13 @@ class TestSimpleTaskRunner : public SingleThreadTaskRunner {
bool RunsTasksOnCurrentThread() const override;
- std::deque<TestPendingTask> GetPendingTasks() const;
+ std::deque<TestPendingTask> TakePendingTasks();
size_t NumPendingTasks() const;
bool HasPendingTask() const;
base::TimeDelta NextPendingTaskDelay() const;
+ base::TimeDelta FinalPendingTaskDelay() const;
+ base::Optional<tracked_objects::Location> GetPendingTaskLocationAt(
+ size_t index) const;
// Clears the queue of pending tasks without running them.
void ClearPendingTasks();
« no previous file with comments | « no previous file | base/test/test_simple_task_runner.cc » ('j') | extensions/common/one_shot_event_unittest.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698