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

Side by Side Diff: components/task_scheduler_util/browser/initialization.cc

Issue 2565013002: Split browser-specific and generic code in components/task_scheduler_util/. (Closed)
Patch Set: self-review Created 4 years 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "components/task_scheduler_util/browser/initialization.h"
6
7 #include "base/command_line.h"
8 #include "base/logging.h"
9 #include "base/task_scheduler/switches.h"
10 #include "base/task_scheduler/task_traits.h"
11 #include "base/threading/platform_thread.h"
12 #include "base/threading/sequenced_worker_pool.h"
13 #include "components/task_scheduler_util/common/variations_util.h"
14
15 namespace task_scheduler_util {
16
17 namespace {
18
19 enum WorkerPoolType : size_t {
20 BACKGROUND = 0,
21 BACKGROUND_FILE_IO,
22 FOREGROUND,
23 FOREGROUND_FILE_IO,
24 WORKER_POOL_COUNT // Always last.
25 };
26
27 } // namespace
28
29 std::vector<base::SchedulerWorkerPoolParams> GetBrowserWorkerPoolParams(
30 const std::map<std::string, std::string>& variation_params) {
31 using ThreadPriority = base::ThreadPriority;
32 std::vector<SchedulerConstantWorkerPoolParams> constant_worker_pool_params;
33 DCHECK_EQ(BACKGROUND, constant_worker_pool_params.size());
34 constant_worker_pool_params.emplace_back("Background",
35 ThreadPriority::BACKGROUND);
36 DCHECK_EQ(BACKGROUND_FILE_IO, constant_worker_pool_params.size());
37 constant_worker_pool_params.emplace_back("BackgroundFileIO",
38 ThreadPriority::BACKGROUND);
39 DCHECK_EQ(FOREGROUND, constant_worker_pool_params.size());
40 constant_worker_pool_params.emplace_back("Foreground",
41 ThreadPriority::NORMAL);
42 DCHECK_EQ(FOREGROUND_FILE_IO, constant_worker_pool_params.size());
43 constant_worker_pool_params.emplace_back("ForegroundFileIO",
44 ThreadPriority::NORMAL);
45 return GetWorkerPoolParams(constant_worker_pool_params, variation_params);
46 }
47
48 size_t BrowserWorkerPoolIndexForTraits(const base::TaskTraits& traits) {
49 const bool is_background =
50 traits.priority() == base::TaskPriority::BACKGROUND;
51 if (traits.with_file_io())
52 return is_background ? BACKGROUND_FILE_IO : FOREGROUND_FILE_IO;
53 return is_background ? BACKGROUND : FOREGROUND;
54 }
55
56 void MaybePerformBrowserTaskSchedulerRedirection(
57 const std::map<std::string, std::string>& variation_params) {
58 // TODO(gab): Remove this when http://crbug.com/622400 concludes.
59 const auto sequenced_worker_pool_param =
60 variation_params.find("RedirectSequencedWorkerPools");
61 if (!base::CommandLine::ForCurrentProcess()->HasSwitch(
62 switches::kDisableBrowserTaskScheduler) &&
63 sequenced_worker_pool_param != variation_params.end() &&
64 sequenced_worker_pool_param->second == "true") {
65 base::SequencedWorkerPool::EnableWithRedirectionToTaskSchedulerForProcess();
66 }
67 }
68
69 } // namespace task_scheduler_util
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698