TabManagerDelegate: Better prioritize ARC processes.

chrome/browser/chromeos/arc/arc_process.h

Index: chrome/browser/chromeos/arc/arc_process.h
diff --git a/chrome/browser/chromeos/arc/arc_process.h b/chrome/browser/chromeos/arc/arc_process.h
index 01562a25006edb5e8bda1b62bb6f0ad02d80f6e1..18d2c9542d323d69e7b357a182fd4a42468eb724 100644
--- a/chrome/browser/chromeos/arc/arc_process.h
+++ b/chrome/browser/chromeos/arc/arc_process.h
@@ -15,10 +15,13 @@ namespace arc {
 
 class ArcProcess {
  public:
+  // TODO(cylee): Deprecate process_state if possible.
   ArcProcess(base::ProcessId nspid,
              base::ProcessId pid,
              const std::string& process_name,
-             mojom::ProcessState process_state);
+             mojom::ProcessState process_state,
+             int adj,

[Yusuke Sato, 2016/06/27 16:40:50]

Can you use a more descriptive name for this?

https://google.github.io/styleguide/cppguide.html#General_Naming_Rules says
"eschew abbreviation."

[cylee1, 2016/07/14 19:15:24]

The parameter is replaced by is_focused.

+             int64_t last_activity_time);
   ~ArcProcess();
 
   ArcProcess(ArcProcess&& other);
@@ -28,6 +31,8 @@ class ArcProcess {
   base::ProcessId pid() const { return pid_; }
   const std::string& process_name() const { return process_name_; }
   mojom::ProcessState process_state() const { return process_state_; }
+  int adj() const { return adj_; }
+  int64_t last_activity_time() const { return last_activity_time_; }
   std::vector<std::string>& packages() { return packages_; }
   const std::vector<std::string>& packages() const { return packages_; }
 
@@ -36,6 +41,8 @@ class ArcProcess {
   base::ProcessId pid_;
   std::string process_name_;
   mojom::ProcessState process_state_;
+  int adj_;

[Yusuke Sato, 2016/06/27 16:40:50]

Please document what this is.

[cylee1, 2016/07/14 19:15:24]

Replaced.

+  int64_t last_activity_time_;

[Yusuke Sato, 2016/06/27 16:40:51]

same. (what is the epoch of the time btw?)

[cylee1, 2016/07/14 19:15:24]

Done.

   std::vector<std::string> packages_;
 
   DISALLOW_COPY_AND_ASSIGN(ArcProcess);