[email protected] | cb50762 | 2012-03-23 16:17:06 | [diff] [blame] | 1 | // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
[email protected] | 76d7f72 | 2011-10-10 17:22:41 | [diff] [blame] | 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 "net/dns/serial_worker.h" |
| 6 | |
Eric Orth | 8ec1b8e | 2021-10-11 19:33:47 | [diff] [blame] | 7 | #include <memory> |
Eric Orth | 88a6aaa | 2021-10-19 17:20:56 | [diff] [blame] | 8 | #include <utility> |
Eric Orth | 8ec1b8e | 2021-10-11 19:33:47 | [diff] [blame] | 9 | |
[email protected] | 76d7f72 | 2011-10-10 17:22:41 | [diff] [blame] | 10 | #include "base/bind.h" |
Eric Orth | 88a6aaa | 2021-10-19 17:20:56 | [diff] [blame] | 11 | #include "base/callback.h" |
Hans Wennborg | 0924470b | 2020-04-27 21:08:05 | [diff] [blame] | 12 | #include "base/check_op.h" |
[email protected] | 76d7f72 | 2011-10-10 17:22:41 | [diff] [blame] | 13 | #include "base/location.h" |
Hans Wennborg | 0924470b | 2020-04-27 21:08:05 | [diff] [blame] | 14 | #include "base/notreached.h" |
Gabriel Charette | 44db142 | 2018-08-06 11:19:33 | [diff] [blame] | 15 | #include "base/task/post_task.h" |
Gabriel Charette | d5c656c | 2020-02-26 16:35:22 | [diff] [blame] | 16 | #include "base/task/thread_pool.h" |
Robbie McElrath | 31391ba | 2018-11-15 02:04:50 | [diff] [blame] | 17 | #include "base/threading/sequenced_task_runner_handle.h" |
gab | f767595f | 2016-05-11 18:50:35 | [diff] [blame] | 18 | #include "base/threading/thread_task_runner_handle.h" |
[email protected] | 76d7f72 | 2011-10-10 17:22:41 | [diff] [blame] | 19 | |
| 20 | namespace net { |
| 21 | |
Eric Orth | 8ec1b8e | 2021-10-11 19:33:47 | [diff] [blame] | 22 | namespace { |
| 23 | std::unique_ptr<SerialWorker::WorkItem> DoWork( |
| 24 | std::unique_ptr<SerialWorker::WorkItem> work_item) { |
| 25 | DCHECK(work_item); |
| 26 | work_item->DoWork(); |
| 27 | return work_item; |
| 28 | } |
| 29 | } // namespace |
| 30 | |
Eric Orth | 88a6aaa | 2021-10-19 17:20:56 | [diff] [blame] | 31 | void SerialWorker::WorkItem::FollowupWork(base::OnceClosure closure) { |
| 32 | std::move(closure).Run(); |
| 33 | } |
| 34 | |
Eric Orth | 8ec1b8e | 2021-10-11 19:33:47 | [diff] [blame] | 35 | SerialWorker::SerialWorker() : state_(State::kIdle) {} |
[email protected] | 76d7f72 | 2011-10-10 17:22:41 | [diff] [blame] | 36 | |
Chris Watkins | 68b1503 | 2017-12-01 03:07:13 | [diff] [blame] | 37 | SerialWorker::~SerialWorker() = default; |
[email protected] | 76d7f72 | 2011-10-10 17:22:41 | [diff] [blame] | 38 | |
| 39 | void SerialWorker::WorkNow() { |
Francois Doray | becd1ef7 | 2017-09-25 20:58:45 | [diff] [blame] | 40 | DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_); |
[email protected] | 76d7f72 | 2011-10-10 17:22:41 | [diff] [blame] | 41 | switch (state_) { |
Peter Kasting | d039b77 | 2021-08-18 00:06:20 | [diff] [blame] | 42 | case State::kIdle: |
Erik Anderson | c243d71 | 2021-07-30 00:13:41 | [diff] [blame] | 43 | // We are posting weak pointer to OnWorkJobFinished to avoid leak when |
Sami Kyostila | 0b4314e | 2019-07-31 20:47:17 | [diff] [blame] | 44 | // PostTaskAndReply fails to post task back to the original |
Leonid Kaplan | 24c51bc | 2018-09-04 13:23:16 | [diff] [blame] | 45 | // task runner. In this case the callback is not destroyed, and the |
| 46 | // weak reference allows SerialWorker instance to be deleted. |
Eric Orth | 8ec1b8e | 2021-10-11 19:33:47 | [diff] [blame] | 47 | base::ThreadPool::PostTaskAndReplyWithResult( |
Gabriel Charette | d5c656c | 2020-02-26 16:35:22 | [diff] [blame] | 48 | FROM_HERE, |
| 49 | {base::MayBlock(), base::TaskShutdownBehavior::CONTINUE_ON_SHUTDOWN}, |
Eric Orth | 8ec1b8e | 2021-10-11 19:33:47 | [diff] [blame] | 50 | base::BindOnce(&DoWork, CreateWorkItem()), |
Eric Orth | 88a6aaa | 2021-10-19 17:20:56 | [diff] [blame] | 51 | base::BindOnce(&SerialWorker::OnDoWorkFinished, AsWeakPtr())); |
Peter Kasting | d039b77 | 2021-08-18 00:06:20 | [diff] [blame] | 52 | state_ = State::kWorking; |
[email protected] | 76d7f72 | 2011-10-10 17:22:41 | [diff] [blame] | 53 | return; |
Peter Kasting | d039b77 | 2021-08-18 00:06:20 | [diff] [blame] | 54 | case State::kWorking: |
Eric Orth | 8ec1b8e | 2021-10-11 19:33:47 | [diff] [blame] | 55 | // Remember to re-read after `DoWork()` finishes. |
Peter Kasting | d039b77 | 2021-08-18 00:06:20 | [diff] [blame] | 56 | state_ = State::kPending; |
[email protected] | 76d7f72 | 2011-10-10 17:22:41 | [diff] [blame] | 57 | return; |
Peter Kasting | d039b77 | 2021-08-18 00:06:20 | [diff] [blame] | 58 | case State::kCancelled: |
| 59 | case State::kPending: |
[email protected] | 76d7f72 | 2011-10-10 17:22:41 | [diff] [blame] | 60 | return; |
[email protected] | 76d7f72 | 2011-10-10 17:22:41 | [diff] [blame] | 61 | } |
| 62 | } |
| 63 | |
| 64 | void SerialWorker::Cancel() { |
Francois Doray | becd1ef7 | 2017-09-25 20:58:45 | [diff] [blame] | 65 | DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_); |
Peter Kasting | d039b77 | 2021-08-18 00:06:20 | [diff] [blame] | 66 | state_ = State::kCancelled; |
[email protected] | 76d7f72 | 2011-10-10 17:22:41 | [diff] [blame] | 67 | } |
Erik Anderson | c243d71 | 2021-07-30 00:13:41 | [diff] [blame] | 68 | |
Eric Orth | 88a6aaa | 2021-10-19 17:20:56 | [diff] [blame] | 69 | void SerialWorker::OnDoWorkFinished(std::unique_ptr<WorkItem> work_item) { |
| 70 | DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_); |
| 71 | |
| 72 | switch (state_) { |
| 73 | case State::kCancelled: |
| 74 | return; |
| 75 | case State::kWorking: { |
| 76 | WorkItem* work_item_ptr = work_item.get(); |
| 77 | work_item_ptr->FollowupWork( |
| 78 | base::BindOnce(&SerialWorker::OnFollowupWorkFinished, |
| 79 | weak_factory_.GetWeakPtr(), std::move(work_item))); |
| 80 | return; |
| 81 | } |
| 82 | case State::kPending: { |
| 83 | RerunWork(std::move(work_item)); |
| 84 | return; |
| 85 | } |
| 86 | default: |
| 87 | NOTREACHED() << "Unexpected state " << static_cast<int>(state_); |
| 88 | } |
| 89 | } |
| 90 | |
| 91 | void SerialWorker::OnFollowupWorkFinished(std::unique_ptr<WorkItem> work_item) { |
Francois Doray | becd1ef7 | 2017-09-25 20:58:45 | [diff] [blame] | 92 | DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_); |
[email protected] | 76d7f72 | 2011-10-10 17:22:41 | [diff] [blame] | 93 | switch (state_) { |
Peter Kasting | d039b77 | 2021-08-18 00:06:20 | [diff] [blame] | 94 | case State::kCancelled: |
[email protected] | 76d7f72 | 2011-10-10 17:22:41 | [diff] [blame] | 95 | return; |
Peter Kasting | d039b77 | 2021-08-18 00:06:20 | [diff] [blame] | 96 | case State::kWorking: |
| 97 | state_ = State::kIdle; |
Eric Orth | 8ec1b8e | 2021-10-11 19:33:47 | [diff] [blame] | 98 | OnWorkFinished(std::move(work_item)); |
[email protected] | 76d7f72 | 2011-10-10 17:22:41 | [diff] [blame] | 99 | return; |
Peter Kasting | d039b77 | 2021-08-18 00:06:20 | [diff] [blame] | 100 | case State::kPending: |
Eric Orth | 88a6aaa | 2021-10-19 17:20:56 | [diff] [blame] | 101 | RerunWork(std::move(work_item)); |
[email protected] | 76d7f72 | 2011-10-10 17:22:41 | [diff] [blame] | 102 | return; |
| 103 | default: |
Peter Kasting | d039b77 | 2021-08-18 00:06:20 | [diff] [blame] | 104 | NOTREACHED() << "Unexpected state " << static_cast<int>(state_); |
[email protected] | 76d7f72 | 2011-10-10 17:22:41 | [diff] [blame] | 105 | } |
| 106 | } |
| 107 | |
Eric Orth | 88a6aaa | 2021-10-19 17:20:56 | [diff] [blame] | 108 | void SerialWorker::RerunWork(std::unique_ptr<WorkItem> work_item) { |
| 109 | // `WorkNow()` was retriggered while working, so need to redo work |
| 110 | // immediately to ensure up-to-date results. Reuse `work_item` rather than |
| 111 | // returning it to the derived class (and letting it potentially act on a |
| 112 | // potential obsolete result). |
| 113 | DCHECK_EQ(state_, State::kPending); |
| 114 | state_ = State::kWorking; |
| 115 | base::ThreadPool::PostTaskAndReplyWithResult( |
| 116 | FROM_HERE, |
| 117 | {base::MayBlock(), base::TaskShutdownBehavior::CONTINUE_ON_SHUTDOWN}, |
| 118 | base::BindOnce(&DoWork, std::move(work_item)), |
| 119 | base::BindOnce(&SerialWorker::OnDoWorkFinished, AsWeakPtr())); |
| 120 | } |
| 121 | |
Eric Orth | 8ec1b8e | 2021-10-11 19:33:47 | [diff] [blame] | 122 | base::WeakPtr<SerialWorker> SerialWorker::AsWeakPtr() { |
| 123 | return weak_factory_.GetWeakPtr(); |
| 124 | } |
| 125 | |
[email protected] | 76d7f72 | 2011-10-10 17:22:41 | [diff] [blame] | 126 | } // namespace net |