Make it easier to debug failed symbolization

Make it easier to debug failed symbolization

This just adds a --verbose argument and some debug logging sprinkled in
useful places.

Adds '.' as a candidate directory for libraries (gn puts shared
libraries at the root build directory).

Committed: https://crrev.com/17ffc51dfc9bd6dfe0d38ce8f6f8313544ea25af
Cr-Commit-Position: refs/heads/master@{#298952}

[cjhopman, 2014-10-09 18:16:23]

cjhopman@chromium.org changed reviewers:
+ rmcilroy@chromium.org

[cjhopman, 2014-10-09 18:16:23]

rmcilroy: *

[rmcilroy, 2014-10-09 18:27:53]

Lgtm, thanks.

https://codereview.chromium.org/646683002/diff/50001/third_party/android_plat...
File third_party/android_platform/README.chromium (right):

https://codereview.chromium.org/646683002/diff/50001/third_party/android_plat...
third_party/android_platform/README.chromium:18: modified to better suit Chrome
development.
I'm fine with doing this, but the cl upload check will fail unless you modify
this file anyway, so unless we can change that I would rather each CL just added
a line here to avoid the next cl having to needlessly modify this file some
other way.

[cjhopman, 2014-10-09 18:41:02]

https://codereview.chromium.org/646683002/diff/50001/third_party/android_plat...
File third_party/android_platform/README.chromium (right):

https://codereview.chromium.org/646683002/diff/50001/third_party/android_plat...
third_party/android_platform/README.chromium:18: modified to better suit Chrome
development.
On 2014/10/09 18:27:52, rmcilroy wrote:
> I'm fine with doing this, but the cl upload check will fail unless you modify
> this file anyway, so unless we can change that I would rather each CL just
added
> a line here to avoid the next cl having to needlessly modify this file some
> other way.

I mostly did this because the list of local modifications wasn't actually
accurate and I expect that even if it is updated to be accurate it wouldn't stay
that way for long.

That being said, I have kept the list and just added a message that makes it
clear that we may have local modifications that aren't listed here (and added
some missing modifications).

[cjhopman, 2014-10-09 18:41:20]

The CQ bit was checked by cjhopman@chromium.org

[commit-bot: I haz the power, 2014-10-09 18:42:32]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/646683002/130001

[commit-bot: I haz the power, 2014-10-09 19:07:18]

Committed patchset #4 (id:130001)

[commit-bot: I haz the power, 2014-10-09 19:07:55]

Patchset 4 (id:??) landed as
https://crrev.com/17ffc51dfc9bd6dfe0d38ce8f6f8313544ea25af
Cr-Commit-Position: refs/heads/master@{#298952}

[maniscalco, 2014-10-09 22:51:36]

A revert of this CL (patchset #4 id:130001) has been created in
https://codereview.chromium.org/642843003/ by maniscalco@chromium.org.

The reason for reverting is: "Android Tests" bot is failing in "run stack tool
with logcat dump" step.

http://build.chromium.org/p/chromium.linux/builders/Android%20Tests

Suspecting this patch.  Reverting.


@@@BUILD_STEP Run stack tool with logcat dump@@@
>
/b/build/slave/Android_Tests/build/src/third_party/android_platform/development/scripts/stack
--more-info /b/build/slave/Android_Tests/build/src/out/Release/full_log.txt
/b/build/slave/Android_Tests/build/src/third_party/android_platform/development/scripts/../../../../third_party/android_tools/ndk/toolchains/arm-linux-androideabi-4.6/prebuilt/linux-x86_64/bin/arm-linux-androideabi-addr2line:
/b/build/slave/Android_Tests/build/src/third_party/android_platform/development/scripts/../../../../out/Release/icudtl.dat:
File format not recognized


See bug https://code.google.com/p/chromium/issues/detail?id=422090.