Minor performance improvement. Use "ps" instead of "sed" to find parent

processes. And properly mark variables as local, so that the exit code
from "identify" is correct. Otherwise, we needlessly scan for more
processes when returning from "identify".

TEST=none
BUG=none
Review URL: http://codereview.chromium.org/3173046

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@57368 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 2 insertions, 1 deletions

diff --git a/chrome/tools/chrome-process-identifier b/chrome/tools/chrome-process-identifier
index ece1669..770529b 100755
--- a/chrome/tools/chrome-process-identifier
+++ b/chrome/tools/chrome-process-identifier
@@ -28,7 +28,7 @@ fi
 ls -l "/proc/$pid/exe" 2>/dev/null|egrep -q '/chrome$' || {
   echo "Cannot find any running instance of Chrome" >&2; exit 1; }
 while :; do
-  ppid=$(sed 's/PPid:[^0-9]*//;t1;d;:1;q' /proc/$pid/status 2>/dev/null)
+  ppid="$(ps h --format ppid --pid "$pid" 2>/dev/null)"
   [ -n "$ppid" ] || {
     echo "Cannot find any running instance of Chrome" >&2; exit 1; }
   ls "/proc/$ppid/exe" 2>/dev/null|egrep -q '/chrome$' && pid="$ppid" || break
@@ -36,6 +36,7 @@ done
 
 # Iterate over child processes and try to identify them
 identify() {
+  local child cmd foundzygote plugin seccomp type
   foundzygote=0
   for child in $(ps h --format pid --ppid $1); do
     cmd="$(xargs -0 </proc/$child/cmdline|sed 's/ -/\n-/g')" 2>/dev/null