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

Unified Diff: base/task_scheduler/scheduler_worker_pool_params.h

Issue 2611753004: Change const TimeDelta& to TimeDelta in base/task_scheduler/. (Closed)
Patch Set: 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
Index: base/task_scheduler/scheduler_worker_pool_params.h
diff --git a/base/task_scheduler/scheduler_worker_pool_params.h b/base/task_scheduler/scheduler_worker_pool_params.h
index f3cd20fd7a3be7315b6ad972dfee2426ba5d73de..1e5a6a316393a95cf00ca8f0b81c11cf8a1725aa 100644
--- a/base/task_scheduler/scheduler_worker_pool_params.h
+++ b/base/task_scheduler/scheduler_worker_pool_params.h
@@ -9,11 +9,10 @@
#include "base/macros.h"
#include "base/threading/platform_thread.h"
+#include "base/time/time.h"
namespace base {
-class TimeDelta;
-
class BASE_EXPORT SchedulerWorkerPoolParams final {
public:
enum class StandbyThreadPolicy {
@@ -36,7 +35,7 @@ class BASE_EXPORT SchedulerWorkerPoolParams final {
ThreadPriority priority_hint,
StandbyThreadPolicy standby_thread_policy,
int max_threads,
- const TimeDelta& suggested_reclaim_time);
+ TimeDelta suggested_reclaim_time);
SchedulerWorkerPoolParams(SchedulerWorkerPoolParams&& other);
SchedulerWorkerPoolParams& operator=(SchedulerWorkerPoolParams&& other);
@@ -46,9 +45,7 @@ class BASE_EXPORT SchedulerWorkerPoolParams final {
return standby_thread_policy_;
}
size_t max_threads() const { return max_threads_; }
- const TimeDelta& suggested_reclaim_time() const {
- return suggested_reclaim_time_;
- }
+ TimeDelta suggested_reclaim_time() const { return suggested_reclaim_time_; }
private:
std::string name_;
« no previous file with comments | « base/task_scheduler/scheduler_worker_pool_impl_unittest.cc ('k') | base/task_scheduler/scheduler_worker_pool_params.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698