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

Issue 2607283002: Use TaskScheduler instead of WorkerPool in address_sorter_win.cc. (Closed)

Created:
3 years, 11 months ago by fdoray
Modified:
3 years, 11 months ago
Reviewers:
eroman
CC:
chromium-reviews, cbentzel+watch_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Use TaskScheduler instead of WorkerPool in address_sorter_win.cc. The following traits are used: Priority: Inherited (default) The priority is inherited from the calling context (i.e. TaskTraits are initialized with the priority of the current task). Shutdown behavior: CONTINUE_ON_SHUTDOWN Tasks posted with this mode which have not started executing before shutdown is initiated will never run. Tasks with this mode running at shutdown will be ignored (the worker will not be joined). Note: Tasks that were previously posted to base::WorkerPool should use this shutdown behavior because this is how base::WorkerPool handles all its tasks. May Block: Tasks posted with MayBlock() may block. This includes but is not limited to tasks that wait on synchronous file I/O operations: read or write a file from disk, interact with a pipe or a socket, rename or delete a file, enumerate files in a directory, etc. This trait isn't required for the mere use of locks. BUG=659191 Review-Url: https://codereview.chromium.org/2607283002 Cr-Commit-Position: refs/heads/master@{#442356} Committed: https://chromium.googlesource.com/chromium/src/+/6fc67b4140d07a660d3b2b6d2eae7a4fd8b6c9dd

Patch Set 1 #

Patch Set 2 : Add ScopedTaskScheduler. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -11 lines) Patch
M net/dns/address_sorter_unittest.cc View 1 2 chunks +4 lines, -0 lines 0 comments Download
M net/dns/address_sorter_win.cc View 1 3 chunks +9 lines, -11 lines 0 comments Download

Messages

Total messages: 16 (12 generated)
fdoray
PTAL
3 years, 11 months ago (2017-01-09 16:46:43 UTC) #10
eroman
lgtm
3 years, 11 months ago (2017-01-09 20:09:43 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2607283002/20001
3 years, 11 months ago (2017-01-09 20:11:24 UTC) #13
commit-bot: I haz the power
3 years, 11 months ago (2017-01-09 21:34:45 UTC) #16
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/6fc67b4140d07a660d3b2b6d2eae...

Powered by Google App Engine
This is Rietveld 408576698