Codesearch: fix cronet/webview missing dependencies. To fix this, I tried two options. - Option 1: simply remove the obj/ prefix check, whose peak memory is 14.8G, because it stores unnecessary dependencies. - Option 2: something like this CL, whose peak memory is 11.8G. The original solution's peak memory is 12.9G. All the solutions are generating same number of clang units on my gLinux. Bug: 476847943 Bug: 460271270 Change-Id: I50a3068cec40a7a3a053f2e8d38ed50091ced836 Reviewed-on: https://chromium-review.googlesource.com/c/infra/infra/+/7488875 Commit-Queue: Marc Jin <jmarc@google.com> Reviewed-by: Fumitoshi Ukai <ukai@google.com> Cr-Commit-Position: refs/heads/main@{#78471}
Welcome to the Chrome Infra repository!
Wondering where to start? Check out General Chrome Infrastructure documentation. In particular, to check out this repo and the rest of the infrastructure code, follow the instructions here. The rest of this page is specific to this repo.
sys.path modifications.git push origin <updated hash>:deployedgit push commandIf you've added a new module, run your tests with test.py:
*_test.py files to this directory.Double-check that your tests are getting picked up when you want them to be: ./test.py test <path-to-package>.
Tests still not getting picked up by test.py? Double-check to make sure you have __init__.py files in each directory of your module so Python recognizes it as a package.
The preferred style is PEP8 with two-space indent; that is, the Chromium Python style, except functions use lowercase_with_underscores. Use yapf (git cl format) to autoformat new code.