[Android] Add a java method counting script.

[Android] Add a java method counting script.

BUG=497911
Committed: https://crrev.com/465efd23412d344b5bf72e7dc9401016fa78f1f7
Cr-Commit-Position: refs/heads/master@{#335296}

[jbudorick, 2015-06-19 02:08:54]

jbudorick@chromium.org changed reviewers:
+ mikecase@chromium.org

[mikecase (-- gone --), 2015-06-19 15:45:26]

https://codereview.chromium.org/1188253013/diff/1/build/android/method_count.py
File build/android/method_count.py (right):

https://codereview.chromium.org/1188253013/diff/1/build/android/method_count....
build/android/method_count.py:27: print MethodCount(args.dexfile)
Can you use python logging here? Is something going to parse this scripts
stdout?

https://codereview.chromium.org/1188253013/diff/1/build/android/pylib/sdk/dex...
File build/android/pylib/sdk/dexdump.py (right):

https://codereview.chromium.org/1188253013/diff/1/build/android/pylib/sdk/dex...
build/android/pylib/sdk/dexdump.py:25: args =
[os.path.join(constants.ANDROID_SDK_TOOLS, 'dexdump')] + dexfiles
s/os.path.join(constants.ANDROID_SDK_TOOLS, 'dexdump')/_DEXDUMP_PATH

[jbudorick, 2015-06-19 16:25:56]

https://codereview.chromium.org/1188253013/diff/1/build/android/method_count.py
File build/android/method_count.py (right):

https://codereview.chromium.org/1188253013/diff/1/build/android/method_count....
build/android/method_count.py:27: print MethodCount(args.dexfile)
On 2015/06/19 at 15:45:26, mikecase wrote:
> Can you use python logging here? Is something going to parse this scripts
stdout?

this actually needs to use build/util/lib/common/perf_tests_results_helper.py

https://codereview.chromium.org/1188253013/diff/1/build/android/pylib/sdk/dex...
File build/android/pylib/sdk/dexdump.py (right):

https://codereview.chromium.org/1188253013/diff/1/build/android/pylib/sdk/dex...
build/android/pylib/sdk/dexdump.py:25: args =
[os.path.join(constants.ANDROID_SDK_TOOLS, 'dexdump')] + dexfiles
On 2015/06/19 at 15:45:26, mikecase wrote:
> s/os.path.join(constants.ANDROID_SDK_TOOLS, 'dexdump')/_DEXDUMP_PATH

hah, I got halfway there.

Done.

[mikecase (-- gone --), 2015-06-19 17:00:36]

lgtm

[jbudorick, 2015-06-19 17:01:11]

The CQ bit was checked by jbudorick@chromium.org

[commit-bot: I haz the power, 2015-06-19 17:02:32]

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

[commit-bot: I haz the power, 2015-06-19 18:18:55]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-06-19 18:20:00]

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