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

Unified Diff: chrome/browser/task_management/providers/arc/arc_process_task_provider.cc

Issue 2023643002: Add ARC app icons to the Task Manager UI (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: revert back to the original test code Created 4 years, 7 months 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/task_management/providers/arc/arc_process_task_provider.cc
diff --git a/chrome/browser/task_management/providers/arc/arc_process_task_provider.cc b/chrome/browser/task_management/providers/arc/arc_process_task_provider.cc
index dff7a7b38730e30f4304c6b68ef2c41b523005e3..8756626034d94e5c22b816dae07c8aad040acf15 100644
--- a/chrome/browser/task_management/providers/arc/arc_process_task_provider.cc
+++ b/chrome/browser/task_management/providers/arc/arc_process_task_provider.cc
@@ -60,23 +60,24 @@ void ArcProcessTaskProvider::OnUpdateProcessList(
nspid_to_remove.insert(entry.first);
for (const auto& entry : processes) {
- if (nspid_to_remove.erase(entry.nspid) == 0) {
+ if (nspid_to_remove.erase(entry.nspid()) == 0) {
// New arc process.
- std::unique_ptr<ArcProcessTask>& task = nspid_to_task_[entry.nspid];
+ std::unique_ptr<ArcProcessTask>& task = nspid_to_task_[entry.nspid()];
// After calling NotifyObserverTaskAdded(), the raw pointer of |task| is
// remebered somewhere else. One should not (implicitly) delete the
// referenced object before calling NotifyObserverTaskRemoved() first
// (crbug.com/587707).
DCHECK(!task.get()) <<
"Task with the same pid should not be added twice.";
- task.reset(new ArcProcessTask(
- entry.pid, entry.nspid, entry.process_name, entry.process_state));
+ task.reset(new ArcProcessTask(entry.pid(), entry.nspid(),
+ entry.process_name(), entry.process_state(),
+ entry.packages()));
NotifyObserverTaskAdded(task.get());
} else {
// Update process state of existing process.
- std::unique_ptr<ArcProcessTask>& task = nspid_to_task_[entry.nspid];
+ std::unique_ptr<ArcProcessTask>& task = nspid_to_task_[entry.nspid()];
DCHECK(task.get());
- task->SetProcessState(entry.process_state);
+ task->SetProcessState(entry.process_state());
}
}
« no previous file with comments | « chrome/browser/task_management/providers/arc/arc_process_task.cc ('k') | chrome/chrome_browser_chromeos.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698