android: Wait for Chrome to finish writing trace file before downloading

build/android/adb_profile_chrome

 #!/bin/bash
 #
 # Copyright 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 #
 # Start / stop profiling in chrome.
 
 # The profiling data is saved to directory /sdcard/Download. The files
 # are named beginning chrome-profile-results-
@@ skipping 25 matching lines @@
 
 send_intent() {
   local PACKAGE=$1
   local INTENT=$2
   shift
   shift
   adb shell am broadcast -a $PACKAGE.$INTENT $*
 }
 
 download_latest_trace() {
-  TRACE_FILE=$(adb logcat -d | \
-      grep "Logging performance trace to file: " | \
-      tail -1 | \
-      perl -pi -e "s/.*\/storage\/emulated\/.+\/([^\r]+).*/\/sdcard\/Download\/\\1/g")
-  if [ -z "$TRACE_FILE" ]; then
-    echo "Unable to determine trace file name"
-    exit 1
-  fi
+  (adb logcat -d | grep -q "Logging performance trace to file") || {
+    echo "WARNING: Trace start marker not found. Is the correct version of Chrome running?"
+  }
+
+  local ITERATION=0
+  while true; do
+    # Chrome logs two different messages related to tracing:
+    #
+    # 1. "Logging performance trace to file [...]"
+    # 2. "Profiler finished. Results are in [...]"
+    #
+    # The first one is printed when tracing starts and the second one indicates
+    # that the trace file is ready to be downloaded.
+    #
+    # We have to look for both of these messages to make sure we get the results
+    # from the latest trace and that the trace file is complete. This is done by
+    # first looking for the last instance of the first message and then checking
+    # for the second message in the remaining part of the log.
+    TRACE_FILE=$(adb logcat -d | \
+        tac | \
+        grep --max-count=1 --before-context=100000 "Logging performance trace to file" | \
+        tac | \
+        grep "Profiler finished[.] Results are in " | \
+        perl -pi -e "s{.*/storage/emulated/.+/([^\r]+)[.].*}{/sdcard/Download/\\1}g")
+    if [ -n "$TRACE_FILE" ]; then
+      break
+    fi
+    if [ $ITERATION -eq 0 ]; then
+      echo -n "Waiting for Chrome to finish tracing..."
+    else
+      echo -n "."
+    fi
+    let ITERATION=ITERATION+1
+    if [ $ITERATION -eq 60 ]; then
+      echo "Timed out"
+      exit 1
+    fi
+    sleep 1
+  done
 
   adb pull $TRACE_FILE 2> /dev/null
   LOCAL_TRACE_FILE=$(basename $TRACE_FILE)
   if [ ! -f "$LOCAL_TRACE_FILE" ]; then
     echo "Unable to download trace file"
     exit 1
   fi
 }
 
 do_timed_capture() {
   local PACKAGE=$1
   local INTERVAL=$2
   shift
   shift
   echo -n "Capturing trace..."
   send_intent ${PACKAGE} "GPU_PROFILER_START" $* > /dev/null
   sleep ${INTERVAL}
   send_intent ${PACKAGE} "GPU_PROFILER_STOP" > /dev/null
   echo "done"
 
   echo -n "Downloading trace..."
-  sleep $[${INTERVAL} / 4 + 1]
   download_latest_trace
   echo "done"
 
   echo "Trace written to ${PWD}/${LOCAL_TRACE_FILE}"
 }
 
 PACKAGE=${DEFAULT_PACKAGE:-com.android.chrome}
 
 while test -n "$1"; do
   case "$1" in
@@ skipping 47 matching lines @@
 if [ -z "${INTERVAL}" ] ; then
   if [ -z "${FUNCTION}" ] ; then
     usage
   else
     send_intent ${PACKAGE} ${FUNCTION} ${OUTPUT} ${CATEGORIES} ${CONTINUOUS}
   fi
 else
   do_timed_capture ${PACKAGE} ${INTERVAL} ${CATEGORIES} ${CONTINUOUS}
 fi
 exit 0