Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(750)

Unified Diff: build/android/gdb_content_shell

Issue 10736027: Enhancement for gdb_content_shell, allow set package_name and shared_lib_dir (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Refine the patch Created 8 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « build/android/gdb_apk ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/gdb_content_shell
diff --git a/build/android/gdb_content_shell b/build/android/gdb_content_shell
index 2f0b9830c6fbe4ac3d2bd9653257d4ddb2e17687..27cc3b4df1465d667e61cc1c8a5d7474d2b8fe1b 100755
--- a/build/android/gdb_content_shell
+++ b/build/android/gdb_content_shell
@@ -4,85 +4,12 @@
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
#
-# Attach gdb to a running content shell. Similar to ndk-gdb.
-# Run with --annotate=3 if running under emacs (M-x gdb).
-#
-# TODO(jrg): allow package_name and shared_lib_dir to be set on the
-# command line. Share the guts of this script with other Chromium
-# pieces (like base_unittests_apk) and friends (like WebKit bundles).
-
-adb=$(which adb)
-if [[ "$adb" = "" ]] ; then
- echo "Need adb in your path"
- exit 1
-fi
+# Attach gdb to a running content shell. Redirect to the shell gdb_apk
-rooted_phone=0
+ROOT=$(cd "$(dirname $0)"; pwd)
-root=$(dirname $0)/../..
-package_name=org.chromium.content_shell
-data_dir=/data/data/$package_name
-gdb_server_on_device=$data_dir/lib/gdbserver
-shared_lib_dir=$root/out/Release/lib.target
-
-# Kill any running gdbserver
-pid=$(adb shell ps | awk '/gdbserver/ {print $2}')
-if [[ "$pid" != "" ]] ; then
- if [[ $rooted_phone -eq 1 ]] ; then
- adb shell kill $pid
- else
- adb shell run-as $package_name kill $pid
- fi
-fi
-
-pid=$(adb shell ps | awk "/$package_name/ {print \$2}")
-if [[ "$pid" = "" ]] ; then
- echo "No $package_name running?"
- echo "Try this: adb shell am start -a android.intent.action.VIEW " \
- "-n $package_name/.SomethingActivity (Something might be ContentShell)"
- exit 2
-fi
-
-no_gdb_server=$(adb shell ls $gdb_server_on_device | grep 'No such file')
-if [[ "$no_gdb_server" != "" ]] ; then
- echo "No gdb server on device at $gdb_server_on_device"
- echo "Please install a debug build."
- exit 3
-fi
-
-if [[ $rooted_phone -eq 1 ]] ; then
- adb shell $gdb_server_on_device :4321 --attach $pid &
- adb forward tcp:4321 tcp:4321
-else
- adb shell run-as $package_name lib/gdbserver +debug-socket --attach $pid &
- adb forward tcp:4321 localfilesystem:$data_dir/debug-socket
-fi
-sleep 2
-
-# Pull app_process and C libraries from device if needed
-app_process=${shared_lib_dir}/app_process
-if [[ ! -f ${app_process} ]] ; then
- adb pull /system/bin/app_process ${app_process}
- adb pull /system/lib/libc.so ${shared_lib_dir}
-fi
-
-# gdb commands
-cmdfile=$(mktemp /tmp/gdb_android_XXXXXXXX)
-cat >$cmdfile<<EOF
-# set solib-absolute-prefix null
-set solib-search-path ${shared_lib_dir}
-file ${app_process}
-target remote :4321
-EOF
-
-gdb=$(echo $ANDROID_TOOLCHAIN/*gdb)
-if [[ ! -f ${gdb} ]] ; then
- echo "Wow no gdb in env var ANDROID_TOOLCHAIN which is $ANDROID_TOOLCHAIN"
- exit 4
+if [ $# -gt 0 ]; then
+ exec $ROOT/gdb_apk -r -g "$*"
else
- echo Using $gdb
+ exec $ROOT/gdb_apk -r
fi
-
-# ${gdb} -x $cmdfile $* $app_process
-${gdb} -x $cmdfile $*
-rm $cmdfile
« no previous file with comments | « build/android/gdb_apk ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698