Add native debugging support to android content shell.

Add native debugging support to android content shell.

How to native debug:
- apply the patch listed in the code review description
- adb_run_content_shell
- gdb_content_shell
- attach Eclipse; continue
- now boom!  SEGV in gdb.

Sadly the native library crashes while loading so it's not registered
with gdb yet (e.g no symbols or stack).

This CL also adds in a file that got lost
(testing/android/README.chromium).

BUG=
TEST=


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=135652

[John Grabowski, 2012-04-28 02:47:58]

+ jam for OWNERS
+ cc nilesh; we probably want to use the same pattern for all apks

(Ted: apk is now 97% smaller so installs just a hair faster :-)

[John Grabowski, 2012-04-28 02:48:41]

Oh -- here's the patch mentioned in the CL description.

diff --git
a/content/shell/android/java/org/chromium/content_shell/ContentShellActivity.java
b/content/shell/android/java/org/chromium/content_shell/ContentShellActivity.java
index a3a1028..918124e 100644
---
a/content/shell/android/java/org/chromium/content_shell/ContentShellActivity.java
+++
b/content/shell/android/java/org/chromium/content_shell/ContentShellActivity.java
@@ -7,6 +7,7 @@ package org.chromium.content_shell;
 import android.app.Activity;
 import android.content.Intent;
 import android.os.Bundle;
+import android.os.Debug;
 import android.text.TextUtils;
 import android.util.Log;
 import android.view.KeyEvent;
@@ -98,6 +99,8 @@ public class ContentShellActivity extends Activity {
     private static final String NATIVE_LIBRARY = "content_shell_content_view";
 
     private void loadNativeLibrary() throws UnsatisfiedLinkError {
+        Log.i(TAG, "waiting for debugger");
+        Debug.waitForDebugger();
         Log.i(TAG, "loading: " + NATIVE_LIBRARY);
         try {
             System.loadLibrary(NATIVE_LIBRARY);

[jam, 2012-04-30 16:03:22]

rubberstamp lgtm

[Satish, 2012-05-03 10:28:03]

https://chromiumcodereview.appspot.com/10254028/diff/2001/build/android/gdb_c...
File build/android/gdb_content_shell (right):

https://chromiumcodereview.appspot.com/10254028/diff/2001/build/android/gdb_c...
build/android/gdb_content_shell:32: adb shell kill $pid
indentation

https://chromiumcodereview.appspot.com/10254028/diff/2001/build/android/gdb_c...
build/android/gdb_content_shell:34: adb shell run-as $package_name kill $pid
ditto

https://chromiumcodereview.appspot.com/10254028/diff/2001/build/android/gdb_c...
build/android/gdb_content_shell:59: adb shell $gdb_server_on_device :4321
--attach $pid &
ditto

https://chromiumcodereview.appspot.com/10254028/diff/2001/build/android/gdb_c...
build/android/gdb_content_shell:61: adb shell run-as $package_name
$gdb_server_on_device :4321 --attach $pid &
ditto

https://chromiumcodereview.appspot.com/10254028/diff/2001/build/android/gdb_c...
build/android/gdb_content_shell:63: sleep 2
is this sleep required? are there other ways to wait such as some logcat output
that gdbserver prints?

https://chromiumcodereview.appspot.com/10254028/diff/2001/build/android/gdb_c...
build/android/gdb_content_shell:82: echo "Wow no gdb in env var
ANDROID_TOOLCHAIN which is $ANDROID_TOOLCHAIN"
indentation

https://chromiumcodereview.appspot.com/10254028/diff/2001/build/android/gdb_c...
build/android/gdb_content_shell:85: echo Using $gdb
ditto

https://chromiumcodereview.appspot.com/10254028/diff/2001/content/shell/andro...
File content/shell/android/content_shell_apk.xml (right):

https://chromiumcodereview.appspot.com/10254028/diff/2001/content/shell/andro...
content/shell/android/content_shell_apk.xml:19: <property
name="resource.absolute.dir" value="res"/>
should this be "${out.dir}/res" as well since the name says 'absolute.dir' ?

[John Grabowski, 2012-05-07 12:40:15]

https://chromiumcodereview.appspot.com/10254028/diff/2001/build/android/gdb_c...
File build/android/gdb_content_shell (right):

https://chromiumcodereview.appspot.com/10254028/diff/2001/build/android/gdb_c...
build/android/gdb_content_shell:32: adb shell kill $pid
On 2012/05/03 10:28:03, Satish wrote:
> indentation

emacs sees [[ as double indent.  Sigh.

https://chromiumcodereview.appspot.com/10254028/diff/2001/build/android/gdb_c...
build/android/gdb_content_shell:63: sleep 2
On 2012/05/03 10:28:03, Satish wrote:
> is this sleep required? are there other ways to wait such as some logcat
output
> that gdbserver prints?

There is no logcat output.  There is stdio output but since I run this in the
background (with &) it's a little tricky to parse.  Do you feel strongly?

[commit-bot: I haz the power, 2012-05-07 13:35:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jrg@chromium.org/10254028/9002

[commit-bot: I haz the power, 2012-05-07 16:25:59]

Change committed as 135652