Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright 2015 The Chromium Authors. All rights reserved. | 1 // Copyright 2015 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "chrome/browser/task_management/providers/arc/arc_process_task_provider .h" | 5 #include "chrome/browser/task_management/providers/arc/arc_process_task_provider .h" |
| 6 | 6 |
| 7 #include <stddef.h> | 7 #include <stddef.h> |
| 8 | 8 |
| 9 #include <queue> | 9 #include <queue> |
| 10 #include <set> | 10 #include <set> |
| (...skipping 74 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 85 // Stale arc process. | 85 // Stale arc process. |
| 86 NotifyObserverTaskRemoved(nspid_to_task_[entry].get()); | 86 NotifyObserverTaskRemoved(nspid_to_task_[entry].get()); |
| 87 nspid_to_task_.erase(entry); | 87 nspid_to_task_.erase(entry); |
| 88 } | 88 } |
| 89 ScheduleNextRequest(); | 89 ScheduleNextRequest(); |
| 90 } | 90 } |
| 91 | 91 |
| 92 void ArcProcessTaskProvider::RequestProcessList() { | 92 void ArcProcessTaskProvider::RequestProcessList() { |
| 93 arc::ArcProcessService* arc_process_service = | 93 arc::ArcProcessService* arc_process_service = |
| 94 arc::ArcProcessService::Get(); | 94 arc::ArcProcessService::Get(); |
| 95 auto callback = base::Bind(&ArcProcessTaskProvider::OnUpdateProcessList, | |
| 96 weak_ptr_factory_.GetWeakPtr()); | |
| 95 if (!arc_process_service || | 97 if (!arc_process_service || |
| 96 !arc_process_service->RequestProcessList( | 98 !arc_process_service->RequestProcessList(callback)) { |
| 97 base::Bind(&ArcProcessTaskProvider::OnUpdateProcessList, | |
| 98 weak_ptr_factory_.GetWeakPtr()))) { | |
| 99 VLOG(2) << "ARC process instance is not ready."; | 99 VLOG(2) << "ARC process instance is not ready."; |
| 100 ScheduleNextRequest(); | 100 // Update the process list with an empty list of ARC processes. |
| 101 callback.Run(std::vector<ArcProcess>()); | |
|
cylee
2016/06/08 09:58:01
Could you add a comment to make it clear the case
Shuhei Takahashi
2016/06/08 10:50:21
Done.
| |
| 101 } | 102 } |
| 102 } | 103 } |
| 103 | 104 |
| 104 void ArcProcessTaskProvider::StartUpdating() { | 105 void ArcProcessTaskProvider::StartUpdating() { |
| 105 is_updating_ = true; | 106 is_updating_ = true; |
| 106 RequestProcessList(); | 107 RequestProcessList(); |
| 107 } | 108 } |
| 108 | 109 |
| 109 void ArcProcessTaskProvider::StopUpdating() { | 110 void ArcProcessTaskProvider::StopUpdating() { |
| 110 is_updating_ = false; | 111 is_updating_ = false; |
| 111 nspid_to_task_.clear(); | 112 nspid_to_task_.clear(); |
| 112 } | 113 } |
| 113 | 114 |
| 114 void ArcProcessTaskProvider::ScheduleNextRequest() { | 115 void ArcProcessTaskProvider::ScheduleNextRequest() { |
| 115 if (!is_updating_) | 116 if (!is_updating_) |
| 116 return; | 117 return; |
| 117 // TODO(nya): Remove this timer once ARC starts to send us UpdateProcessList | 118 // TODO(nya): Remove this timer once ARC starts to send us UpdateProcessList |
| 118 // message when the process list changed. As of today, ARC does not send | 119 // message when the process list changed. As of today, ARC does not send |
| 119 // the process list unless we request it by RequestProcessList message. | 120 // the process list unless we request it by RequestProcessList message. |
| 120 base::ThreadTaskRunnerHandle::Get()->PostDelayedTask( | 121 base::ThreadTaskRunnerHandle::Get()->PostDelayedTask( |
| 121 FROM_HERE, | 122 FROM_HERE, |
| 122 base::Bind(&ArcProcessTaskProvider::RequestProcessList, | 123 base::Bind(&ArcProcessTaskProvider::RequestProcessList, |
| 123 weak_ptr_factory_.GetWeakPtr()), | 124 weak_ptr_factory_.GetWeakPtr()), |
| 124 base::TimeDelta::FromSeconds(kUpdateProcessListDelaySeconds)); | 125 base::TimeDelta::FromSeconds(kUpdateProcessListDelaySeconds)); |
| 125 } | 126 } |
| 126 | 127 |
| 127 } // namespace task_management | 128 } // namespace task_management |
| OLD | NEW |