summaryrefslogtreecommitdiffstats
path: root/build/android
Commit message (Collapse)AuthorAgeFilesLines
* Create output dir in logcat printer if it doesn't exist.bpastene2016-03-251-4/+13
| | | | | | | | | | Also add a statement about where to find the actual logcat. BUG=597771,448050 Review URL: https://codereview.chromium.org/1838483002 Cr-Commit-Position: refs/heads/master@{#383330}
* Include isolate.py in data for Android unit testsagrieve2016-03-242-22/+170
| | | | | | | | | | | This is required for any test that uses an .isolate to push files to the device (e.g. base_unittests). BUG=589318 Review URL: https://codereview.chromium.org/1784373002 Cr-Commit-Position: refs/heads/master@{#383127}
* Stop lint.py pulling in implicit dependencies via AndroidManifest.xmlagrieve2016-03-241-2/+8
| | | | | | | | | | | This will mean even fewer UnusedResource checks, but that is covered by: https://bugs.chromium.org/p/chromium/issues/detail?id=595810 BUG=597007 Review URL: https://codereview.chromium.org/1826563002 Cr-Commit-Position: refs/heads/master@{#382986}
* Default no-locals to 0 in dex.py.yfriedman2016-03-241-1/+1
| | | | | | | | | | | | gyp always specified a value for --no-locals when compiling targets but GN doesn't. This resulted in GN erroneously stripping them. BUG=596134 Review URL: https://codereview.chromium.org/1830693003 Cr-Commit-Position: refs/heads/master@{#382985}
* Support packaging secondary abi native libraries in APK.michaelbai2016-03-231-13/+51
| | | | | | | | BUG=594807 Review URL: https://codereview.chromium.org/1828523002 Cr-Commit-Position: refs/heads/master@{#382917}
* Replace usages of DEPRECATED_java_in_dir with java_filesagrieve2016-03-231-1/+1
| | | | | | | | BUG=484854 Review URL: https://codereview.chromium.org/1829823002 Cr-Commit-Position: refs/heads/master@{#382905}
* Reland of Android: Run lint using a cache in the output directory (fix-up)agrieve2016-03-234-50/+27
| | | | | | | | | | | | No longer trying to guess name of cached .bin file. Previous commit forgot to set --cache-dir for GN. BUG=583661,597251 Review URL: https://codereview.chromium.org/1828693002 Cr-Commit-Position: refs/heads/master@{#382871}
* lint.py - print original exception for empty results.xmlagrieve2016-03-231-31/+48
| | | | | | | | BUG=583661 Review URL: https://codereview.chromium.org/1823173002 Cr-Commit-Position: refs/heads/master@{#382848}
* Revert of 🌈 Android: Run lint using a cache in the output directory ↵aberent2016-03-233-20/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (fix-up) (patchset #2 id:20001 of https://codereview.chromium.org/1824093002/ ) Reason for revert: Breaks on Android 64bit unpublished builders BUG=597251 Original issue's description: > Android: Run lint using a cache in the output directory (fix-up) > > Previous commit forgot to set --cache-dir for GN. > > BUG=583661 > > Committed: https://crrev.com/99748af41ad073389f3f38274a024a830585609c > Cr-Commit-Position: refs/heads/master@{#382699} TBR=jbudorick@chromium.org,agrieve@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=583661 Review URL: https://codereview.chromium.org/1829683002 Cr-Commit-Position: refs/heads/master@{#382838}
* Android: Run lint using a cache in the output directory (fix-up)agrieve2016-03-223-35/+20
| | | | | | | | | | Previous commit forgot to set --cache-dir for GN. BUG=583661 Review URL: https://codereview.chromium.org/1824093002 Cr-Commit-Position: refs/heads/master@{#382699}
* [Android] Make lint XML parsing failures nonfatal without --can-fail-build.jbudorick2016-03-221-0/+2
| | | | | | | | BUG=583661 Review URL: https://codereview.chromium.org/1817943005 Cr-Commit-Position: refs/heads/master@{#382582}
* Roll third_party/android_tools, updating support lib to 23.2.1ianwen2016-03-222-0/+2
| | | | | | | | BUG=415300 Review URL: https://codereview.chromium.org/1809893002 Cr-Commit-Position: refs/heads/master@{#382468}
* adb_gdb: Detect TARGET_ARCH from symbols_dir rather than deviceagrieve2016-03-211-34/+39
| | | | | | | | | | This fixes arch detection when 32-bit Chrome is run on a 64-bit device BUG=596088 Review URL: https://codereview.chromium.org/1811363002 Cr-Commit-Position: refs/heads/master@{#382319}
* [Android] Run lint using a cache in the output directory.jbudorick2016-03-215-96/+229
| | | | | | | | BUG=583661 Review URL: https://codereview.chromium.org/1815563005 Cr-Commit-Position: refs/heads/master@{#382306}
* adb_gdb: Add --device flagagrieve2016-03-181-2/+6
| | | | | | | | | | and fix it not erroring out when multiple devices are attached BUG=none Review URL: https://codereview.chromium.org/1818433002 Cr-Commit-Position: refs/heads/master@{#382036}
* [Android] Fix instrumentation clearing application state twice and not ↵rnephew2016-03-151-1/+0
| | | | | | | | | | setting permissions. BUG=591224 Review URL: https://codereview.chromium.org/1800283002 Cr-Commit-Position: refs/heads/master@{#381323}
* GN: Fix ChromePublic.apk not rebuilding for native changesagrieve2016-03-151-3/+3
| | | | | | | | | | | It was not being rebuilt when is_component_build=true, and the only change to the .apk's inputs was to an indirect .cr.so file. BUG=594409 Review URL: https://codereview.chromium.org/1802853002 Cr-Commit-Position: refs/heads/master@{#381253}
* Add android_tools/.../libc++.so to Android's swarming data depsagrieve2016-03-151-0/+1
| | | | | | | | | | | Should fix aapt failure on bots. TBR=jbudorick BUG=589318 Review URL: https://codereview.chromium.org/1804843002 Cr-Commit-Position: refs/heads/master@{#381160}
* [Android] Disable lint on gyp targets that use empty or placeholder java ↵jbudorick2016-03-141-0/+1
| | | | | | | | | | directories. BUG=583661 Review URL: https://codereview.chromium.org/1785333004 Cr-Commit-Position: refs/heads/master@{#381035}
* test_runner.py: Auto-detect output dir when running on botsagrieve2016-03-141-0/+10
| | | | | | | | BUG=594105,593973 Review URL: https://codereview.chromium.org/1799073004 Cr-Commit-Position: refs/heads/master@{#381032}
* Flip proprietary codecs to false for Android chromium builds.dalecurtis2016-03-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | Official builds still using branding=Chrome so they will continue to support codecs and containers like h264, mp3, aac, and mp4. Adds support for video/webm videos using vp8,vorbis to the webview tests so they will continue to work w/o proprietary codecs. We have approval to do this now. Summary will be posted to the src= launch bug: http://crbug.com/533190#c17 This will require some changes to the build flags for the official WebView AOSP builders, which I'll land shortly after this. BUG=570762 TEST=webview tests pass. CQ_INCLUDE_TRYBOTS=tryserver.chromium.android:android_chromium_gn_rel Review URL: https://codereview.chromium.org/1577433004 Cr-Commit-Position: refs/heads/master@{#380839}
* Remove devil/ from android.isolate. It's pulled in via catapult.stip2016-03-121-1/+0
| | | | | | | | BUG=525873 Review URL: https://codereview.chromium.org/1788963002 Cr-Commit-Position: refs/heads/master@{#380836}
* Don't write deps that GN already knows about to .d filesagrieve2016-03-111-2/+12
| | | | | | | | | | | | | | | | | | | It causes extra targets to be compiled when GN args change, and a target has logic like: target("foo") { if (arg) { deps = [":A"] } else { deps = [":B"] } } BUG=589311 Review URL: https://codereview.chromium.org/1730993002 Cr-Commit-Position: refs/heads/master@{#380704}
* [Android] Update devil SDK binary references from 23.0.0 to 23.0.1.jbudorick2016-03-112-6/+6
| | | | | | | | BUG=589318 Review URL: https://codereview.chromium.org/1785483005 Cr-Commit-Position: refs/heads/master@{#380648}
* [Android] Add SDK binary tools to devil_py data.jbudorick2016-03-101-3/+8
| | | | | | | | | | | This also narrows some of the specified catapult directories to avoid isolating directories containing dependencies downloaded from cloud storage. BUG=589318 Review URL: https://codereview.chromium.org/1783603003 Cr-Commit-Position: refs/heads/master@{#380464}
* [Android] Remove chromium version of devil.jbudorick2016-03-1093-15616/+13
| | | | | | | | BUG=476719 Review URL: https://codereview.chromium.org/1770943003 Cr-Commit-Position: refs/heads/master@{#380317}
* Fix (at least some) of Android Swarming /w GNagrieve2016-03-091-0/+1
| | | | | | | | | | Verified this fixes android_webview_unittests BUG=589318 Review URL: https://codereview.chromium.org/1774193004 Cr-Commit-Position: refs/heads/master@{#380183}
* Make the status reporting of devices in status_check more verbose.bpastene2016-03-091-9/+18
| | | | | | | | | | | | | | | Replaces the unknowns we keep seeing with either usb_failure, missing, or offline. missing: Shows up in lsusb but not adb. usb_failure: Shows up in lsusb but can't reset the device via /dev/bus. offline: Doesn't show up in lsusb or adb. BUG=592666 Review URL: https://codereview.chromium.org/1768423002 Cr-Commit-Position: refs/heads/master@{#380028}
* Created separate shards for suspicious testcases.hzl2016-03-092-5/+37
| | | | | | | | | In case a crashing testcase stops following testcases in the shard from running, created a separate shard for suspicious crashing testcase. BUG=581086 Review URL: https://codereview.chromium.org/1703863003 Cr-Commit-Position: refs/heads/master@{#379987}
* Raise exception when all devices fail to list tests.hzl2016-03-081-2/+5
| | | | | | | | | | When devices fail to list tests, instead of treating the result of list tests, which is None as an empty list, we should raise an exception. BUG=585585 Review URL: https://codereview.chromium.org/1716863002 Cr-Commit-Position: refs/heads/master@{#379925}
* Fix Android incremental install failing to set permissionsjohnme2016-03-082-4/+7
| | | | | | | | | | | | | | | | | | | On Marshmallow devices, permissions such as WRITE_EXTERNAL_STORAGE must be set for gtest suites to run properly (otherwise you can get strange results like "0 tests ran"). _InstallInternal in build/android/devil/android/device_utils.py will automatically set the right permissions, but only if the permissions parameter is None. But do_install in build/android/incremental_install/installer.py was setting permissions to (), hence _InstallInternal wasn't granting the permissions. This patch fixes that. BUG=none Review URL: https://codereview.chromium.org/1738843003 Cr-Commit-Position: refs/heads/master@{#379826}
* Fix typo in build/android/resource_sizes.pyperezju2016-03-081-1/+1
| | | | | | | | BUG=555668 Review URL: https://codereview.chromium.org/1770883002 Cr-Commit-Position: refs/heads/master@{#379817}
* Fix incremental install for Android Nagrieve2016-03-071-8/+16
| | | | | | | | BUG=590394 Review URL: https://codereview.chromium.org/1772843002 Cr-Commit-Position: refs/heads/master@{#379664}
* [Android] Add RUN_INSTRUMENTATION to the permission blacklist.jbudorick2016-03-071-0/+1
| | | | | | | | | | chromium port of https://codereview.chromium.org/1768253002/ BUG= Review URL: https://codereview.chromium.org/1768263002 Cr-Commit-Position: refs/heads/master@{#379612}
* [Android] Log initial exception on lint failure.jbudorick2016-03-071-3/+5
| | | | | | | | BUG=583661 Review URL: https://codereview.chromium.org/1767103002 Cr-Commit-Position: refs/heads/master@{#379594}
* Specify blink_platform_unittests.isolate in gtest_test_instance.py.jbroman2016-03-071-0/+2
| | | | | | | | | This allows running blink_platform_unittests on a local device without explicitly specifying the isolate path. Review URL: https://codereview.chromium.org/1766153003 Cr-Commit-Position: refs/heads/master@{#379515}
* Replaced apk_command_line with adb_cast_shell_command_line.sanfin2016-03-022-1/+25
| | | | | | | | | | | The new script uses the adb_command_line.py script in build/android and is therefore compatible with the --device argument. BUG= internal b/26406816 Review URL: https://codereview.chromium.org/1750393003 Cr-Commit-Position: refs/heads/master@{#378780}
* Allow pushing all command line files to user buildskraush2016-03-013-1/+33
| | | | | | | | | | | | | | This change alters the command line file handling in python to always push, restore, and delete them in /data/local/tmp rather than /data/local on user builds. This will allow user builds to run all tests. BUG=561796 Review URL: https://codereview.chromium.org/1688003004 Cr-Commit-Position: refs/heads/master@{#378562}
* Extend emulator launch the wait time to be 60syolandyan2016-03-011-2/+2
| | | | | | | | | | | Mistakenly not glient synced in my last CL which reverted the CL that extend the emulator launch wait time to be 60s BUG= Review URL: https://codereview.chromium.org/1751083002 Cr-Commit-Position: refs/heads/master@{#378542}
* Fix a bug of not specifying android command pathyolandyan2016-03-011-15/+17
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/1752493002 Cr-Commit-Position: refs/heads/master@{#378347}
* Make _incremental targets work with emma_coverage=trueagrieve2016-02-291-0/+1
| | | | | | | | BUG=587986 Review URL: https://codereview.chromium.org/1711253002 Cr-Commit-Position: refs/heads/master@{#378251}
* [Android] Add known devices file argument to perf test runner.rnephew2016-02-263-7/+19
| | | | | | | | | | | Recent changes make the path to the known devices file configurable. This change makes the perf test runner aware of this. BUG=590229 Review URL: https://codereview.chromium.org/1736283002 Cr-Commit-Position: refs/heads/master@{#377926}
* Extend each emulator wait time to 1 minyolandyan2016-02-261-4/+4
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/1736193002 Cr-Commit-Position: refs/heads/master@{#377809}
* Reland: md5check'ify lint.pyagrieve2016-02-261-23/+40
| | | | | | | | | | | | | | When locally building chrome_public_apk_incremental and making a non-public change to a content_java file, this brings ninja time: 24s -> 7s BUG=None TBR=jbudorick Review URL: https://codereview.chromium.org/1733893003 Cr-Commit-Position: refs/heads/master@{#377758}
* Remove //tools/profile_chrome from Chromium repozhenw2016-02-262-1/+11
| | | | | | | | | | | | | | The previous CL (https://crrev.com/1726783002/) copied //tools/profile_chrome to Catapult. This CL removes //tools/profile_chrome in Chromium repo and updates the dependency. BUG=catapult:#1937 TBR=thakis@chromium.org Review URL: https://codereview.chromium.org/1738563002 Cr-Commit-Position: refs/heads/master@{#377733}
* Add wait option in avd.py to wait for emulators to finish bootingyolandyan2016-02-252-68/+101
| | | | | | | | | | | | | | Emulators on bots are spawn in the background process so it requires support from the launch script to wait for all the emulators to finish booting Related CL: 1729053002 BUG= Review URL: https://codereview.chromium.org/1733733002 Cr-Commit-Position: refs/heads/master@{#377694}
* Revert of md5check'ify lint.py (patchset #2 id:20001 of ↵agrieve2016-02-251-38/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/1730143003/ ) Reason for revert: Breaking bots. Original issue's description: > md5check'ify lint.py > > When locally building chrome_public_apk_incremental and making a > non-public change to a content_java file, this brings ninja time: > > 24s -> 7s > > BUG=None > > Committed: https://crrev.com/b84f70aeb94075c8f6ec85a8211b231a9125ba41 > Cr-Commit-Position: refs/heads/master@{#377614} TBR=jbudorick@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=None Review URL: https://codereview.chromium.org/1739903002 Cr-Commit-Position: refs/heads/master@{#377650}
* md5check'ify lint.pyagrieve2016-02-251-23/+38
| | | | | | | | | | | | | When locally building chrome_public_apk_incremental and making a non-public change to a content_java file, this brings ninja time: 24s -> 7s BUG=None Review URL: https://codereview.chromium.org/1730143003 Cr-Commit-Position: refs/heads/master@{#377614}
* Revert of [Android] Switch instrumentation tests to platform mode. (patchset ↵jbudorick2016-02-243-9/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | #11 id:200001 of https://codereview.chromium.org/1387933005/ ) Reason for revert: CRONET Original issue's description: > [Android] Switch instrumentation tests to platform mode. > > BUG=428729 > > Committed: https://crrev.com/e5648339683b69d8eba1b7899b4e2593d2f83de0 > Cr-Commit-Position: refs/heads/master@{#377324} TBR=mikecase@chromium.org,rnephew@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=428729 Review URL: https://codereview.chromium.org/1738543002 Cr-Commit-Position: refs/heads/master@{#377417}
* [Android] Switch instrumentation tests to platform mode.jbudorick2016-02-243-9/+9
| | | | | | | | BUG=428729 Review URL: https://codereview.chromium.org/1387933005 Cr-Commit-Position: refs/heads/master@{#377324}