Commit eefdba8c authored by fdoray's avatar fdoray Committed by Commit bot

Remove TaskTraits::with_file_io().

BUG=675660
TBR=jam@chromium.org,eugenebut@chromium.org

Review-Url: https://codereview.chromium.org/2629583003
Cr-Commit-Position: refs/heads/master@{#443549}
parent c0cef7c1
...@@ -141,10 +141,6 @@ class BASE_EXPORT TaskTraits { ...@@ -141,10 +141,6 @@ class BASE_EXPORT TaskTraits {
// Returns true if tasks with these traits may use base/ sync primitives. // Returns true if tasks with these traits may use base/ sync primitives.
bool with_base_sync_primitives() const { return with_base_sync_primitives_; } bool with_base_sync_primitives() const { return with_base_sync_primitives_; }
// DEPRECATED
// TODO(fdoray): Remove this as part of crbug.com/675660
bool with_file_io() const { return may_block(); }
// Returns the priority of tasks with these traits. // Returns the priority of tasks with these traits.
TaskPriority priority() const { return priority_; } TaskPriority priority() const { return priority_; }
......
...@@ -60,7 +60,7 @@ GetBrowserWorkerPoolParamsFromVariations() { ...@@ -60,7 +60,7 @@ GetBrowserWorkerPoolParamsFromVariations() {
size_t BrowserWorkerPoolIndexForTraits(const base::TaskTraits& traits) { size_t BrowserWorkerPoolIndexForTraits(const base::TaskTraits& traits) {
const bool is_background = const bool is_background =
traits.priority() == base::TaskPriority::BACKGROUND; traits.priority() == base::TaskPriority::BACKGROUND;
if (traits.with_file_io()) if (traits.may_block() || traits.with_base_sync_primitives())
return is_background ? BACKGROUND_FILE_IO : FOREGROUND_FILE_IO; return is_background ? BACKGROUND_FILE_IO : FOREGROUND_FILE_IO;
return is_background ? BACKGROUND : FOREGROUND; return is_background ? BACKGROUND : FOREGROUND;
} }
......
...@@ -463,7 +463,7 @@ GetDefaultSchedulerWorkerPoolParams() { ...@@ -463,7 +463,7 @@ GetDefaultSchedulerWorkerPoolParams() {
size_t DefaultBrowserWorkerPoolIndexForTraits(const base::TaskTraits& traits) { size_t DefaultBrowserWorkerPoolIndexForTraits(const base::TaskTraits& traits) {
const bool is_background = const bool is_background =
traits.priority() == base::TaskPriority::BACKGROUND; traits.priority() == base::TaskPriority::BACKGROUND;
if (traits.with_file_io()) if (traits.may_block() || traits.with_base_sync_primitives())
return is_background ? BACKGROUND_FILE_IO : FOREGROUND_FILE_IO; return is_background ? BACKGROUND_FILE_IO : FOREGROUND_FILE_IO;
return is_background ? BACKGROUND : FOREGROUND; return is_background ? BACKGROUND : FOREGROUND;
......
...@@ -120,7 +120,7 @@ GetDefaultSchedulerWorkerPoolParams() { ...@@ -120,7 +120,7 @@ GetDefaultSchedulerWorkerPoolParams() {
size_t DefaultRendererWorkerPoolIndexForTraits(const base::TaskTraits& traits) { size_t DefaultRendererWorkerPoolIndexForTraits(const base::TaskTraits& traits) {
const bool is_background = const bool is_background =
traits.priority() == base::TaskPriority::BACKGROUND; traits.priority() == base::TaskPriority::BACKGROUND;
if (traits.with_file_io()) if (traits.may_block() || traits.with_base_sync_primitives())
return is_background ? BACKGROUND_FILE_IO : FOREGROUND_FILE_IO; return is_background ? BACKGROUND_FILE_IO : FOREGROUND_FILE_IO;
return is_background ? BACKGROUND : FOREGROUND; return is_background ? BACKGROUND : FOREGROUND;
......
...@@ -78,7 +78,7 @@ GetDefaultSchedulerWorkerPoolParams() { ...@@ -78,7 +78,7 @@ GetDefaultSchedulerWorkerPoolParams() {
size_t DefaultBrowserWorkerPoolIndexForTraits(const base::TaskTraits& traits) { size_t DefaultBrowserWorkerPoolIndexForTraits(const base::TaskTraits& traits) {
const bool is_background = const bool is_background =
traits.priority() == base::TaskPriority::BACKGROUND; traits.priority() == base::TaskPriority::BACKGROUND;
if (traits.with_file_io()) if (traits.may_block() || traits.with_base_sync_primitives())
return is_background ? BACKGROUND_FILE_IO : FOREGROUND_FILE_IO; return is_background ? BACKGROUND_FILE_IO : FOREGROUND_FILE_IO;
return is_background ? BACKGROUND : FOREGROUND; return is_background ? BACKGROUND : FOREGROUND;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment