0
Commit Graph

12 Commits

Author SHA1 Message Date
Wei-Yin Chen (陳威尹)
0f8750b3d1 docs: Debugging Robolectric tests in Android Studio
NOTRY=true  # docs update

Bug: None
Change-Id: I1c3c71e1d1813b71de780e2e50accdedd1acadfe
Reviewed-on: https://chromium-review.googlesource.com/804137
Reviewed-by: agrieve <agrieve@chromium.org>
Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522882}
2017-12-08 21:42:15 +00:00
Andrew Grieve
4fe99743ac Android: Tweaks for --wait-for-java-debugger
* Add the flag to junit tests
* Better log statements
* Updated docs

Change-Id: If967015a697ef6a840f26f2311768230b874ab46
Reviewed-on: https://chromium-review.googlesource.com/786970
Commit-Queue: agrieve <agrieve@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#519007}
2017-11-23 19:43:16 +00:00
Andrew Grieve
6a686c5c5d android_debugging_instructions.md: Link to android_studio.md
NOTRY=true  # docs update

Change-Id: I4f5840875d6d5256ef14b12de5364643a751f1e4
Reviewed-on: https://chromium-review.googlesource.com/700420
Reviewed-by: Peter Wen <wnwen@chromium.org>
Commit-Queue: Peter Wen <wnwen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506406}
2017-10-04 16:25:22 +00:00
Andrew Grieve
f8b7d9b8f8 Android: Remove --wait-for-java-debugger Chrome flag
It's superseded by the --wait-for-java-debugger wrapper script flag.

The wrapper scripts use "am set-debug-app -w", which cause the
debugger to attach before any application code has been loaded
and is much more useful since it allows breakpoints during
start-up.

TBR=sanfin

Change-Id: Ib4a8f64a33e089c571d0c441415afabe6a8031dc
Reviewed-on: https://chromium-review.googlesource.com/697905
Commit-Queue: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: Luke Halliwell <halliwell@chromium.org>
Reviewed-by: Richard Coles <torne@chromium.org>
Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506281}
2017-10-04 02:43:47 +00:00
Andrew Grieve
0904ce1533 Reland "Android: Add --wait-for-java-debugger to test_runner.py"
This reverts commit f7b7b5ab8c.

Reason for revert: Will update when ready.

Change-Id: I5478e9a738b64a6a34a9937e080b59583f7c3f16
Reviewed-on: https://chromium-review.googlesource.com/698544
Reviewed-by: Yoland Yan <yolandyan@chromium.org>
Commit-Queue: Andrew Grieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506174}
2017-10-03 21:09:12 +00:00
Peter Mayo
f7b7b5ab8c Revert "Android: Add --wait-for-java-debugger to test_runner.py"
This reverts commit d77bb0d559.

Reason for revert: Fails on Android Cronet Perf bot(s).

I    0.000s Main  command: ... Android_Cronet_x86_Builder ... test_runner.py perf ...
E    0.001s Main  Unrecognized error occurred.
Traceback (most recent call last):
  File ".../src/build/android/test_runner.py", line 1001, in main
    return RunTestsCommand(args)
  ...
  File ".../src/build/android/pylib/local/device/local_device_environment.py", line 148, in __init__
    self._wait_for_java_debugger = args.wait_for_java_debugger
AttributeError: 'Namespace' object has no attribute 'wait_for_java_debugger'
  ...
Original change's description:
> Android: Add --wait-for-java-debugger to test_runner.py
  ...
> 
> Change-Id: I7cd58c747534b3b539afcc84fcbae48475c4d9c3
> Reviewed-on: https://chromium-review.googlesource.com/693218
> Commit-Queue: Andrew Grieve <agrieve@chromium.org>
> Reviewed-by: Yoland Yan <yolandyan@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#506049}

TBR=agrieve@chromium.org,yolandyan@chromium.org

Change-Id: I7cf9a5e3c52f29b24df77e0e27eb6272416c5f30
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/698127
Reviewed-by: Peter Mayo <petermayo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506097}
2017-10-03 17:58:29 +00:00
Andrew Grieve
d77bb0d559 Android: Add --wait-for-java-debugger to test_runner.py
I used this to debug test listing in the instrumentation test runner.
This is nicer that using set-debug-app separately, because it disables
all timeouts (or else the process gets killed as you're debugging).

Change-Id: I7cd58c747534b3b539afcc84fcbae48475c4d9c3
Reviewed-on: https://chromium-review.googlesource.com/693218
Commit-Queue: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: Yoland Yan <yolandyan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506049}
2017-10-03 15:44:40 +00:00
Andrew Grieve
c81af4a673 Android: Update docs to use generated apk wrapper scripts
BUG=674250

Change-Id: I336a67f3d528c52ec1d92dd18efd33f4efca84a7
Reviewed-on: https://chromium-review.googlesource.com/586471
Reviewed-by: Tommy Nyquist <nyquist@chromium.org>
Commit-Queue: Tommy Nyquist <nyquist@chromium.org>
Cr-Commit-Position: refs/heads/master@{#489692}
2017-07-26 18:02:13 +00:00
agrieve
a350dbdbfe android: Convert java_deobfuscate.py to Java
This allows it to not buffer output lines, which is essential when
piping logcat through it.

BUG=713710

Review-Url: https://codereview.chromium.org/2956153003
Cr-Commit-Position: refs/heads/master@{#484281}
2017-07-05 15:27:17 +00:00
jbudorick
6a94be3b51 [Android] Remove --test-arguments arg from test_runner.py.
Deprecated in favor of passing such arguments to test_runner.py
directly.

BUG=700366

Review-Url: https://codereview.chromium.org/2855483003
Cr-Commit-Position: refs/heads/master@{#471096}
2017-05-11 22:38:43 +00:00
estevenson
8c9318ff00 Add basic debugging instructions for Android Studio.
The docs still need to be updated with more detailed instructions and
tips, but currently the docs don't even mention debugging in Android
Studio.

BUG=620034

Review-Url: https://codereview.chromium.org/2740353003
Cr-Commit-Position: refs/heads/master@{#456185}
2017-03-10 22:16:35 +00:00
nyquist
c75738d047 Port Android debugging instructions to Markdown
This CL ports the debugging instructions from the Chromium wiki
to Markdown format. The previous location for the documentation
was:
https://www.chromium.org/developers/how-tos/debugging-on-android

BUG=None

Review-Url: https://codereview.chromium.org/2203123006
Cr-Commit-Position: refs/heads/master@{#418332}
2016-09-13 19:28:12 +00:00