From: borenet <borenet@google.com>
Date: Tue, 15 Dec 2015 12:25:39 +0000 (-0800)
Subject: adb_wait_for_device: More fixes
X-Git-Tag: accepted/tizen/5.0/unified/20181102.025319~184^2~316
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f64be4466793280a07ad4aa2319efe152d0e455d;p=platform%2Fupstream%2FlibSkiaSharp.git

adb_wait_for_device: More fixes

BUG=skia:4690

Review URL: https://codereview.chromium.org/1527713002
---

diff --git a/platform_tools/android/bin/adb_wait_for_device b/platform_tools/android/bin/adb_wait_for_device
index fc8a0f1..acf73b1 100755
--- a/platform_tools/android/bin/adb_wait_for_device
+++ b/platform_tools/android/bin/adb_wait_for_device
@@ -6,7 +6,9 @@ SCRIPT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )"
 source $SCRIPT_DIR/android_setup.sh
 source $SCRIPT_DIR/utils/setup_adb.sh
 
-function _get_battery_level {
+# Helper function used by get_battery_level. Parses the battery level from
+# dumpsys output.
+function _parse_battery_level {
   SPLIT=( $@ )
 
   HAS_BATTERY=1
@@ -18,7 +20,7 @@ function _get_battery_level {
     fi
     if [ "${SPLIT[$i]}" = "present:" ]; then
       PRESENT="$(echo "${SPLIT[$i+1]}" | tr -d '\r')"
-      if [ "$PRESENT" -eq "0" ]; then
+      if [ "$PRESENT" = "0" ]; then
         HAS_BATTERY=0
       fi
       if [ "$PRESENT" = "false" ]; then
@@ -36,10 +38,11 @@ function _get_battery_level {
   echo "100"
 }
 
+# Echo the battery level percentage of the attached Android device.
 function get_battery_level {
   STATS="$($ADB $DEVICE_SERIAL shell dumpsys batteryproperties)"
   SPLIT=( $STATS )
-  RV="$(_get_battery_level ${SPLIT[@]})"
+  RV="$(_parse_battery_level ${SPLIT[@]})"
   if [ -n "$RV" ]; then
     echo "$RV"
     return
@@ -49,7 +52,7 @@ function get_battery_level {
 
   STATS="$($ADB $DEVICE_SERIAL shell dumpsys battery)"
   SPLIT=( $STATS )
-  RV="$(_get_battery_level ${SPLIT[@]})"
+  RV="$(_parse_battery_level ${SPLIT[@]})"
   if [ "$RV" != "-1" ]; then
     echo "$RV"
     return