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

Unified Diff: chrome/browser/task_management/providers/arc/arc_process_task.h

Issue 2034543003: Retry icon fetching after intent_helper is ready (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address comments 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
« no previous file with comments | « no previous file | chrome/browser/task_management/providers/arc/arc_process_task.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/task_management/providers/arc/arc_process_task.h
diff --git a/chrome/browser/task_management/providers/arc/arc_process_task.h b/chrome/browser/task_management/providers/arc/arc_process_task.h
index 17d8437e055cc3bfe5d3545e478d78f2ed32ffc0..9cae4d868ede965fefba38a1c96535a64bdbd69e 100644
--- a/chrome/browser/task_management/providers/arc/arc_process_task.h
+++ b/chrome/browser/task_management/providers/arc/arc_process_task.h
@@ -12,13 +12,14 @@
#include "base/macros.h"
#include "base/memory/weak_ptr.h"
#include "chrome/browser/task_management/providers/task.h"
+#include "components/arc/arc_bridge_service.h"
#include "components/arc/common/process.mojom.h"
#include "components/arc/intent_helper/activity_icon_loader.h"
namespace task_management {
// Defines a task that represents an ARC process.
-class ArcProcessTask : public Task {
+class ArcProcessTask : public Task, public arc::ArcBridgeService::Observer {
public:
ArcProcessTask(base::ProcessId pid,
base::ProcessId nspid,
@@ -33,18 +34,23 @@ class ArcProcessTask : public Task {
bool IsKillable() override;
void Kill() override;
+ // arc::ArcBridgeService::Observer:
+ void OnIntentHelperInstanceReady() override;
+
void SetProcessState(arc::mojom::ProcessState process_state);
base::ProcessId nspid() const { return nspid_; }
const std::string& process_name() const { return process_name_; }
private:
+ void StartIconLoading();
void OnIconLoaded(
std::unique_ptr<arc::ActivityIconLoader::ActivityToIconsMap> icons);
const base::ProcessId nspid_;
const std::string process_name_;
arc::mojom::ProcessState process_state_;
+ const std::string package_name_;
// Note: This should remain the last member so it'll be destroyed and
// invalidate the weak pointers before any other members are destroyed.
« no previous file with comments | « no previous file | chrome/browser/task_management/providers/arc/arc_process_task.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698