summaryrefslogtreecommitdiffstats
path: root/build/android
Commit message (Collapse)AuthorAgeFilesLines
...
* Reland of Add increment install native libraries first in lookup pathsagrieve2016-02-242-9/+9
| | | | | | | | | | | This fixes a subtle bug where the wrong libchromium_android_linker.so is used when it already exists in the system library path. BUG=587584 Review URL: https://codereview.chromium.org/1729973002 Cr-Commit-Position: refs/heads/master@{#377290}
* Revert of Add increment install native libraries first in lookup paths ↵agrieve2016-02-241-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | (patchset #1 id:1 of https://codereview.chromium.org/1726703002/ ) Reason for revert: Breaks incremental install for M devices. Original issue's description: > Add increment install native libraries first in lookup paths > > This fixes a subtle bug where the wrong libchromium_android_linker.so is used > when it already exists in the system library path. > > BUG=587584 > > Committed: https://crrev.com/86af989585dd6aac45d3087456ef7cafb37afa86 > Cr-Commit-Position: refs/heads/master@{#377074} TBR=hanxi@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=587584 Review URL: https://codereview.chromium.org/1731593003 Cr-Commit-Position: refs/heads/master@{#377171}
* Add increment install native libraries first in lookup pathsagrieve2016-02-231-3/+3
| | | | | | | | | | | This fixes a subtle bug where the wrong libchromium_android_linker.so is used when it already exists in the system library path. BUG=587584 Review URL: https://codereview.chromium.org/1726703002 Cr-Commit-Position: refs/heads/master@{#377074}
* Fix javac sometimes not create all outputs with enable_incremental_javacagrieve2016-02-231-0/+9
| | | | | | | | | | | | | | | | | The problem is that sometimes there are .apks that don't have any resources (e.g. test apks), and so there is a javac.py call with a single .srcjar, which has no sources in it. Ninja is expecting the call to create a .jar.pdb file, so just create an empty file to appease it. Without this, running "ninja chrome_public_test_apk" multiple times causes ninja to do the same work over and over again, rather than saying there's no work to do BUG=none Review URL: https://codereview.chromium.org/1706363003 Cr-Commit-Position: refs/heads/master@{#376984}
* [android/pylib] Do not mangle benchmark name in the chartjson outputperezju2016-02-221-15/+0
| | | | | | | | | | | | As a side effect of fixing crbug.com/525220, there is no longer need to swap the benchmark name with the step name in the chartjson with the test results. BUG=568029 Review URL: https://codereview.chromium.org/1683083007 Cr-Commit-Position: refs/heads/master@{#376741}
* Fix generate_emma_html.py failing due to 0-length .em filesagrieve2016-02-191-0/+7
| | | | | | | | | | | This also makes it return non-zero when the .html is not actually created. BUG=587983 Review URL: https://codereview.chromium.org/1711983002 Cr-Commit-Position: refs/heads/master@{#376348}
* Tell _incremental targets to fail fast when load_library_from_apk=trueagrieve2016-02-182-2/+15
| | | | | | | | BUG=587584 Review URL: https://codereview.chromium.org/1709543003 Cr-Commit-Position: refs/heads/master@{#376165}
* Make the change of umcompress_shared_libraries trigger rebuildmichaelbai2016-02-171-1/+3
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/1705903002 Cr-Commit-Position: refs/heads/master@{#375972}
* Import _strptime early to prevent race condition.stip2016-02-171-0/+5
| | | | | | | | BUG=584730 Review URL: https://codereview.chromium.org/1704993002 Cr-Commit-Position: refs/heads/master@{#375927}
* Reland #3 of Tweak GN deps of instrumentation apksagrieve2016-02-161-2/+0
| | | | | | | | | | | | | Reland of https://codereview.chromium.org/1692383002/ This improves build concurrency a little bit. TBR=jbudorick BUG=none Review URL: https://codereview.chromium.org/1702663002 Cr-Commit-Position: refs/heads/master@{#375669}
* Fix Android lint.py behavior when run in C locale (in GN builds).lambroslambrou2016-02-161-2/+2
| | | | | | | | | | | | | | | | | | | | Android GN builds appear to run Python scripts in an environment where the locale is 'C'. The Android linter typically produces XML output containing strings with non-ASCII characters (such as translated string resources). The Python 'print' statement defaults to 'ascii' encoding and throws a UnicodeEncodeError when printing such an item. This exception is caught and (incorrectly) interpreted as a problem with parsing the XML file, which causes lint.py to write out the entire XML file. This CL fixes the problem by specifying utf-8 when printing these objects. BUG=583098 TEST=GN build of remoting_apk no longer produces 1000s of lines of output. Review URL: https://codereview.chromium.org/1701463002 Cr-Commit-Position: refs/heads/master@{#375652}
* [Android] Fix generated scripts for junit tests.mikecase2016-02-162-1/+14
| | | | | | | | | | | | | | | | Adding scripts to make it easy to run junit test. These are named out/bin/run_<junit_suite_name>. Moving the helper scripts that our test runner uses to run the junit test suites. These scripts are not meant to be run except by the test runner. Moving them into out/bin/helper/<script_names> so people don't accidently try to run them. BUG=511999 Review URL: https://codereview.chromium.org/1674353004 Cr-Commit-Position: refs/heads/master@{#375603}
* Fix logcat regex in asan stack trace symbolizer.bpastene2016-02-161-1/+1
| | | | | | | | BUG=549348 Review URL: https://codereview.chromium.org/1698283002 Cr-Commit-Position: refs/heads/master@{#375589}
* [Android] Trim extraneous logging in the android test runner.jbudorick2016-02-164-10/+9
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/1704453002 Cr-Commit-Position: refs/heads/master@{#375584}
* Revert of Make instrumentation_test_apk depend on their apk_under_test. ↵jbudorick2016-02-121-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | (patchset #2 id:70001 of https://codereview.chromium.org/1690983002/ ) Reason for revert: https://code.google.com/p/chromium/issues/detail?id=586641 Original issue's description: > Reland #2 of Tweak GN deps of instrumentation apks > > Reland of https://codereview.chromium.org/1687723002/ > > This improves build concurrency a little bit. > > BUG=none TBR=agrieve@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/1692383002 Cr-Commit-Position: refs/heads/master@{#375272}
* Reland #2 of Tweak GN deps of instrumentation apksagrieve2016-02-121-2/+0
| | | | | | | | | | | | Reland of https://codereview.chromium.org/1687723002/ This improves build concurrency a little bit. BUG=none Review URL: https://codereview.chromium.org/1690983002 Cr-Commit-Position: refs/heads/master@{#375252}
* Android Add java_group() template to GNagrieve2016-02-121-3/+15
| | | | | | | | | | | | | | | | | One drawback of the custom java_* and android_* GN templates is that group() does not work as you'd expect. For example, if you have a dependency like: java_binary() -> group() -> java_library(), then the java_library() won't actually be added to the java_binary() when compiled (because write_build_config requires that all deps also produce build_configs) This adds java_group() in order to fill that gap. BUG=586506 Review URL: https://codereview.chromium.org/1681223007 Cr-Commit-Position: refs/heads/master@{#375249}
* Simplify adb_gdb to accept only --output-directory / CHROMIUM_OUTPUT_DIRagrieve2016-02-121-65/+13
| | | | | | | | | | | No longer respects setting CHROMIUM_OUT_DIR / --out-dir plus BUILDTYPE / --release / --debug. BUG=573345 Review URL: https://codereview.chromium.org/1659413003 Cr-Commit-Position: refs/heads/master@{#375230}
* Android Make --output-dir mandatory for tombstones and friendsagrieve2016-02-123-2/+28
| | | | | | | | | | | | | | | Friends: * third_party/android_platform/development/scripts/stack * build/android/asan_symbolize.py Mandatory: Required, unless CHROMIUM_OUTPUT_DIR or CHROMIUM_OUT_DIR are set or when the current working directory is an output directory (contains a build.ninja file) BUG=573345 Review URL: https://codereview.chromium.org/1670223002 Cr-Commit-Position: refs/heads/master@{#375222}
* Remove pylib.flag_changerzhenw2016-02-121-175/+0
| | | | | | | | | | | This CL removes pylib.flag_changer. flag_changer has been moved to catapult. BUG=583785 Review URL: https://codereview.chromium.org/1691683004 Cr-Commit-Position: refs/heads/master@{#375199}
* [Android] Make provision devices wait longer to detect charging errors.rnephew2016-02-121-1/+1
| | | | | | | | BUG=567786 Review URL: https://codereview.chromium.org/1688113005 Cr-Commit-Position: refs/heads/master@{#375072}
* [Android] Detect if device is not charging properly.rnephew2016-02-114-6/+64
| | | | | | | | | | | Reboot device during provisioning if it is not charging correctly. After rebooting if it is still not charging correctly, blacklist. BUG=567786 Review URL: https://codereview.chromium.org/1672543002 Cr-Commit-Position: refs/heads/master@{#375007}
* Create wrapper scripts that set --output-directoryagrieve2016-02-112-0/+74
| | | | | | | | | | | | | | | For: - build/android/adb_gdb* - build/android/asan_symbolize.py - build/android/tombstones.py - third_party/android_platform/development/scripts/stack - tools/perf/run_benchmark BUG=573345 Review URL: https://codereview.chromium.org/1663103004 Cr-Commit-Position: refs/heads/master@{#374964}
* Use flag_changer in catapult devil.androidzhenw2016-02-113-2/+177
| | | | | | | | | | | | | | | This CL updates the dependency on pylib.flag_changer to Catapult's devil.android.flag_changer in Chromium repo. - The previous CL added devil.android.flag_changer in catapult: https://crrev.com/1675773002/ - The next CL will remove pylib.flag_changer. BUG=583785 Review URL: https://codereview.chromium.org/1677313002 Cr-Commit-Position: refs/heads/master@{#374936}
* [Android] Lazily import lighttpd_server.jbudorick2016-02-111-1/+1
| | | | | | | | | BUG= TBR=mikecase@chromium.org Review URL: https://codereview.chromium.org/1688173002 Cr-Commit-Position: refs/heads/master@{#374903}
* test_runner.py Add --enable-concurrent-adb to --fast-modeagrieve2016-02-101-2/+3
| | | | | | | | | | Missed this one first time around. BUG=none Review URL: https://codereview.chromium.org/1684193002 Cr-Commit-Position: refs/heads/master@{#374752}
* Android Add _incremental targets for instrumentation testsagrieve2016-02-1013-105/+204
| | | | | | | | | | | Switches gtests to explicitly passing path to install script rather than --incremental-install + heuristics. BUG=575975 Review URL: https://codereview.chromium.org/1680233002 Cr-Commit-Position: refs/heads/master@{#374699}
* Add java-side support for _incremental instrumentation testsagrieve2016-02-103-37/+93
| | | | | | | | | | Build rule and test runner changes to follow. BUG=575975 Review URL: https://codereview.chromium.org/1684583003 Cr-Commit-Position: refs/heads/master@{#374695}
* [Android] Change resource_sizes.py to get static initializers the same as ↵rnephew2016-02-101-34/+48
| | | | | | | | | | size.py BUG=546012 Review URL: https://codereview.chromium.org/1644773005 Cr-Commit-Position: refs/heads/master@{#374679}
* test_runner.py Add --fast-local-dev flagagrieve2016-02-091-0/+14
| | | | | | | | | | | As a shorthand for: --verbose --num-retries=0 --enable-device-cache \ --skip-clear-data --extract-test-list-from-filter BUG=None Review URL: https://codereview.chromium.org/1661253002 Cr-Commit-Position: refs/heads/master@{#374420}
* Reland of Add --output-directory to stack tool buildbot step (patchset #1 ↵agrieve2016-02-091-1/+4
| | | | | | | | | | | id:1 of https://codereview.chromium.org/1677193002/ ) TBR=jbudorick@chromium.org,flackr@chromium.org BUG=573345 Review URL: https://codereview.chromium.org/1681143002 Cr-Commit-Position: refs/heads/master@{#374419}
* Android Incremental Install Always use the same device cacheagrieve2016-02-092-13/+25
| | | | | | | | | | Having two scripts that cache the same thing in two different spots is a bad recipe. BUG=none Review URL: https://codereview.chromium.org/1679633002 Cr-Commit-Position: refs/heads/master@{#374415}
* [Android] Make pylib.remote imports tolerate failure.jbudorick2016-02-092-7/+23
| | | | | | | | BUG=580129 Review URL: https://codereview.chromium.org/1673393003 Cr-Commit-Position: refs/heads/master@{#374413}
* Add dependency_manager and catapult_base to devil_py GN target.jbudorick2016-02-091-0/+2
| | | | | | | | | BUG=580129 TBR=agrieve@chromium.org Review URL: https://codereview.chromium.org/1684513002 Cr-Commit-Position: refs/heads/master@{#374315}
* [Android] Add --logcat-output-file option.mikecase2016-02-093-3/+52
| | | | | | | | | | | This option will make it much easier to add logcats to individual steps in recipes. BUG=583798 Review URL: https://codereview.chromium.org/1663933003 Cr-Commit-Position: refs/heads/master@{#374295}
* Adds --skip-clear-data flag to android's test_runner.pyagrieve2016-02-086-7/+17
| | | | | | | | | | Meant to be used to speed up local development. Mostly applicable when running instrumentation tests against _incremental .apks. BUG=540857 Review URL: https://codereview.chromium.org/1570043004 Cr-Commit-Position: refs/heads/master@{#374211}
* Create dist jars only for instrumentation apks; use ijars for themagrieve2016-02-082-4/+15
| | | | | | | | | | | | | | | | | Since the dist jar is used only for extracting the list of tests, there's no need to create one for every apk. Using interface jars rather than normal jars speeds up test calculation time for ChromePublicTestApk 17s -> 5s on my machine. Even better would be to address the TODO and create the test list at build time, but might as well pick this lower-hanging fruit :). BUG=none Review URL: https://codereview.chromium.org/1679623002 Cr-Commit-Position: refs/heads/master@{#374181}
* Revert of Add --output-directory to stack tool buildbot step (patchset #1 ↵flackr2016-02-081-6/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | id:1 of https://codereview.chromium.org/1674683002/ ) Reason for revert: This has is failing 'Run stack tool with logcat dump' step on several android bots: https://build.chromium.org/p/chromium.gpu/builders/Android%20Debug%20%28Nexus%206%29/builds/6322 https://build.chromium.org/p/chromium.gpu/builders/Android%20Debug%20%28Nexus%205%29/builds/5689 Original issue's description: > Add --output-directory to stack tool buildbot step > > BUG=573345 > > Committed: https://crrev.com/ec0616bfa85b1af5369766ee9dad95e69cfe1f7a > Cr-Commit-Position: refs/heads/master@{#374136} 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=573345 Review URL: https://codereview.chromium.org/1677193002 Cr-Commit-Position: refs/heads/master@{#374154}
* Add --output-directory to stack tool buildbot stepagrieve2016-02-081-3/+6
| | | | | | | | BUG=573345 Review URL: https://codereview.chromium.org/1674683002 Cr-Commit-Position: refs/heads/master@{#374136}
* [Android][gn] Add the test runner as a data dependency of test_runner_script.jbudorick2016-02-061-0/+18
| | | | | | | | BUG=580129 Review URL: https://codereview.chromium.org/1672623002 Cr-Commit-Position: refs/heads/master@{#374004}
* Add --output-directory flag to tombstones.py, & friendsagrieve2016-02-061-0/+6
| | | | | | | | | | | | | | Friends: * third_party/android_platform/development/scripts/stack * build/android/asan_symbolize.py Will make it mandatory in stack as in a follow-up. BUG=573345 Review URL: https://codereview.chromium.org/1660693002 Cr-Commit-Position: refs/heads/master@{#373991}
* Remove the disabled webkit_unit_test gtest filterpeter2016-02-061-25/+0
| | | | | | | | | | None of these tests exist anymore. BUG=159935, 241730, 320005 Review URL: https://codereview.chromium.org/1678443003 Cr-Commit-Position: refs/heads/master@{#373981}
* wait for device before restarting its adbdbpastene2016-02-051-0/+1
| | | | | | | | BUG=583614 Review URL: https://codereview.chromium.org/1670833002 Cr-Commit-Position: refs/heads/master@{#373873}
* [Android] Convert TabsTest and InfoBarTest to EmbeddedTestServer.jbudorick2016-02-052-16/+30
| | | | | | | | | | | This is a reland of https://codereview.chromium.org/1639253010/ BUG=488195 TBR=agrieve@chromium.org,maruel@chromium.org,mef@chromium.org,yfriedman@chromium.org Review URL: https://codereview.chromium.org/1671923002 Cr-Commit-Position: refs/heads/master@{#373841}
* Add --output-directory flag to build/android/asan_symbolize.pyagrieve2016-02-051-0/+7
| | | | | | | | BUG=573345 Review URL: https://codereview.chromium.org/1666593002 Cr-Commit-Position: refs/heads/master@{#373832}
* Revert of [Android] Convert TabsTest and InfoBarTest to EmbeddedTestServer. ↵mvanouwerkerk2016-02-052-30/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | (patchset #4 id:60001 of https://codereview.chromium.org/1639253010/ ) Reason for revert: I suspect this CL caused the breakage described in crbug.com/584630 Original issue's description: > [Android] Convert TabsTest and InfoBarTest to EmbeddedTestServer. > > BUG=488195 > TBR=maruel@chromium.org > > Committed: https://crrev.com/91bb9b3d4964cb8611231722a4538afccbd9fd09 > Cr-Commit-Position: refs/heads/master@{#373588} TBR=yfriedman@chromium.org,agrieve@chromium.org,maruel@chromium.org,mef@chromium.org,jbudorick@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=488195 Review URL: https://codereview.chromium.org/1671603003 Cr-Commit-Position: refs/heads/master@{#373789}
* [Android] Convert TabsTest and InfoBarTest to EmbeddedTestServer.jbudorick2016-02-042-16/+30
| | | | | | | | | BUG=488195 TBR=maruel@chromium.org Review URL: https://codereview.chromium.org/1639253010 Cr-Commit-Position: refs/heads/master@{#373588}
* Never auto-detect the output directory in adb_gdbagrieve2016-02-041-3/+17
| | | | | | | | | | | | | | | | | | | Instead, require one of: --output-directory --out-dir CHROMIUM_OUTPUT_DIR CHROMIUM_OUT_DIR CWD=output-dir This should reduce developer confusion resulting from the wrong output directory being auto-detected. TBR=pasko BUG=573345 Review URL: https://codereview.chromium.org/1555783002 Cr-Commit-Position: refs/heads/master@{#373580}
* [Android] Add output_file property to logcat monitor.mikecase2016-02-041-0/+4
| | | | | | | | | | | | | Adding this for convienance to access logcat output if you happen to want to do things with the files (like merge them) after recording. This way to just don't have to store the file names alongside the logcat_monitors. BUG= Review URL: https://codereview.chromium.org/1668703002 Cr-Commit-Position: refs/heads/master@{#373569}
* test_runner.py Fix error message being intertwined with log messagesagrieve2016-02-041-7/+7
| | | | | | | | | | | | | | | | | | | | Before change: I 0.025s Main Using device cache: /usr/local/... Could not find apk or executable for base_unittestsI 0.026s Main Wrote device cache: /usr/... E 0.028s Main Unrecognized error occurred. After change: I 0.025s Main Using device cache: /usr/local/... C 0.025s Main Could not find apk or executable for base_unittests I 0.026s Main Wrote device cache: /usr/local/... BUG=None Review URL: https://codereview.chromium.org/1666983002 Cr-Commit-Position: refs/heads/master@{#373559}
generated by cgit v1.1 at 2025-03-26 06:18:21 +0000