commit f8397a80113a421e8c78f13ed572ef0a4347ed80
author Allen Li <ayatane@chromium.org> Thu Jul 13 13:19:44 2017 -0700
committer chrome-bot <chrome-bot@chromium.org> Fri Jul 14 21:40:47 2017 -0700
tree c7cb9d095c67bb2d4cd4c6c67056dfd9c34b3874
parent ca61e06f8c9595fc4973e4f0b124e76d7a121bd3

sysmon: Improve how other metric is calculated

BUG=chromium:736911
TEST=None

Change-Id: I34713b97604b4078d49aef6404656875df5ced3b
Reviewed-on: https://chromium-review.googlesource.com/570944
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Aviv Keshet <akeshet@chromium.org>

scripts/sysmon/proc_metrics.py

diff --git a/scripts/sysmon/proc_metrics.py b/scripts/sysmon/proc_metrics.py
index 08bf7c6..b592a95 100644
--- a/scripts/sysmon/proc_metrics.py
+++ b/scripts/sysmon/proc_metrics.py
@@ -47,10 +47,10 @@
     self._flush()
 
   def _collect_proc(self, proc):
-    collected = []
     for metric in self._metrics:
-      collected.append(metric.add(proc))
-    if not any(collected):
+      if metric.add(proc):
+        break
+    else:
       self._other_metric.add(proc)
 
   def _flush(self):