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

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: CR robliao #21 + fix build error 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 <map>
8 #include <string>
9
10 #include "base/command_line.h"
11 #include "base/logging.h"
12 #include "base/task_scheduler/scheduler_worker_pool_params.h"
13 #include "base/task_scheduler/switches.h"
14 #include "base/task_scheduler/task_traits.h"
15 #include "base/threading/platform_thread.h"
16 #include "base/threading/sequenced_worker_pool.h"
17 #include "components/task_scheduler_util/common/variations_util.h"
18 #include "components/variations/variations_associated_data.h"
19
20 namespace task_scheduler_util {
21
22 namespace {
23
24 constexpr char kFieldTrialName[] = "BrowserScheduler";
25
26 enum WorkerPoolType : size_t {
27 BACKGROUND = 0,
28 BACKGROUND_FILE_IO,
29 FOREGROUND,
30 FOREGROUND_FILE_IO,
31 WORKER_POOL_COUNT // Always last.
32 };
33
34 } // namespace
35
36 std::vector<base::SchedulerWorkerPoolParams>
37 GetBrowserWorkerPoolParamsFromVariations() {
38 using ThreadPriority = base::ThreadPriority;
39
40 std::map<std::string, std::string> variation_params;
41 if (!::variations::GetVariationParams(kFieldTrialName, &variation_params))
42 return {};
43
44 std::vector<SchedulerImmutableWorkerPoolParams> constant_worker_pool_params;
45 DCHECK_EQ(BACKGROUND, constant_worker_pool_params.size());
46 constant_worker_pool_params.emplace_back("Background",
47 ThreadPriority::BACKGROUND);
48 DCHECK_EQ(BACKGROUND_FILE_IO, constant_worker_pool_params.size());
49 constant_worker_pool_params.emplace_back("BackgroundFileIO",
50 ThreadPriority::BACKGROUND);
51 DCHECK_EQ(FOREGROUND, constant_worker_pool_params.size());
52 constant_worker_pool_params.emplace_back("Foreground",
53 ThreadPriority::NORMAL);
54 DCHECK_EQ(FOREGROUND_FILE_IO, constant_worker_pool_params.size());
55 constant_worker_pool_params.emplace_back("ForegroundFileIO",
56 ThreadPriority::NORMAL);
57
58 return GetWorkerPoolParams(constant_worker_pool_params, variation_params);
59 }
60
61 size_t BrowserWorkerPoolIndexForTraits(const base::TaskTraits& traits) {
62 const bool is_background =
63 traits.priority() == base::TaskPriority::BACKGROUND;
64 if (traits.with_file_io())
65 return is_background ? BACKGROUND_FILE_IO : FOREGROUND_FILE_IO;
66 return is_background ? BACKGROUND : FOREGROUND;
67 }
68
69 void MaybePerformBrowserTaskSchedulerRedirection() {
70 // TODO(gab): Remove this when http://crbug.com/622400 concludes.
71 if (!base::CommandLine::ForCurrentProcess()->HasSwitch(
72 switches::kDisableBrowserTaskScheduler) &&
73 variations::GetVariationParamValue(
74 kFieldTrialName, "RedirectSequencedWorkerPools") == "true") {
75 base::SequencedWorkerPool::EnableWithRedirectionToTaskSchedulerForProcess();
76 }
77 }
78
79 } // namespace task_scheduler_util
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698