TabManager: Add more logs to understand how it works when memory is low.

chrome/browser/chromeos/arc/process/arc_process.cc

Index: chrome/browser/chromeos/arc/process/arc_process.cc
diff --git a/chrome/browser/chromeos/arc/process/arc_process.cc b/chrome/browser/chromeos/arc/process/arc_process.cc
index fbb8d7c37930dedffa96818be60a0726e8723edb..322e731d9f5d29111e2ece38b33e27a6951dc9aa 100644
--- a/chrome/browser/chromeos/arc/process/arc_process.cc
+++ b/chrome/browser/chromeos/arc/process/arc_process.cc
@@ -45,4 +45,16 @@ bool ArcProcess::IsKernelKillable() const {
   return process_state() > arc::mojom::ProcessState::PERSISTENT_UI;
 }
 
+std::ostream& operator<<(std::ostream& out, const ArcProcess& arc_process) {
+  out << "Arc process " << arc_process.process_name() << " ("

[Yusuke Sato, 2017/05/18 21:42:49]

nit: it might be unclear that the number inside the paren is a PID. What about
"just" dump everything?

 out << "process_name: " << arc_process.process_name() << ", pid: " <<
arc_process.pid()

[cylee1, 2017/05/18 22:11:06]

Done.

+      << arc_process.pid() << ")"
+      << ", process_state: " << arc_process.process_state()
+      << ", is_focused: " << arc_process.is_focused()
+      << ", last_activity_time: " << arc_process.last_activity_time()
+      << ", packages: ";
+  for (const auto pkg : arc_process.packages())
+    out << pkg << ",";

[Yusuke Sato, 2017/05/18 21:42:49]

The last "," seems redundant.

 #include "base/strings/string_util.h"

then

 << ", packages: " << base::JoinString(arc_process.packages(), ",");

(or using " " as a separator) would be better.

[cylee1, 2017/05/18 22:11:05]

thanks ! haven't work on the code base for a while so forget the util library ..

+  return out;
+}
+
 }  // namespace arc