- Feb 20, 2024
-
-
Shrinidhi Hegde authored
Added high impact rollback as a last resort after low impact rollbacks are tried. Also removed reference to automatic rollback deny list, since we will be using impact level instead of the deny list going forward. Bug: 291137901 Test: Unit tests Change-Id: I4d75dd513b3f43c179baf93ece832f9007173ff4
-
- Feb 13, 2024
-
-
Sandro Montanari authored
-
Inseob Kim authored
-
Inseob Kim authored
caeab81a Change-Id: Ie5da143d8db3baa93b98cc8fd93d139e54048adc
-
Spandan Das authored
-
- Feb 12, 2024
-
-
Christopher Ferris authored
-
Treehugger Robot authored
Merge "Remove jaggies@google.com from services/core/java/com/android/server/locksettings/OWNERS" into main
-
Treehugger Robot authored
-
Treehugger Robot authored
-
Treehugger Robot authored
-
Treehugger Robot authored
Merge "Remove jaggies@google.com from services/core/java/com/android/server/biometrics/OWNERS" into main
-
Shaquille Johnson authored
-
Treehugger Robot authored
-
Song Chun Fan authored
-
Pablo Gamito authored
* changes: Update tracing OWNERS file to depend on Winscope OWNERS file Add ProtoLog owners files
-
Pablo Gamito authored
Test: n/a Change-Id: I838fefec09e868b702ad036c0692fd0e93a2a12d
-
Pablo Gamito authored
Test: n/a Change-Id: I6e8a3f288a231f44e0df20b52f0bbf6a0ca08460
-
Treehugger Robot authored
-
Treehugger Robot authored
-
Spandan Das authored
This ensures that we do not run into circular depencies when UNBUNDLED_BUILD_TARGET_SDK_WITH_API_FINGERPRINT=true. When this variable is true, apps compile with a min_sdk_version equivalent to the sha of the apis (i.e. the java_genrule `api_fingerprint`). If the api stubs are allowed to depend on android apps, we will run into circular deps Implementation details 1. Modify the deps mutator of combined_apis to add a dependency on the stub java modules 2. Do a graph walk in GenerateAndroidBuildActions and ensure that no child is an android_app. This will be determined by casting to AndroidLibraryDependency. Re (2): this does a graph walk via ctx.WalkDeps, but should not be computationally intensive because (a) We do this for a single module (i.e. combined_apis) (b) The transitive closure is shallow since it starts from stubs Test: presubmits Test: locally, added an app to libs of `service-sdksandbox` and verified that the error is raised Bug: 315016205 Change-Id: Iaf35f03171d13d75e75de6e0e744fcf34e3294fd
-
Naomi Musgrave authored
Bug: 323008518 Change-Id: I56d84ad8d48f694e1d2a59d187402fe78195a76d Test: Build
-
- Feb 11, 2024
-
-
Sam Saccone authored
-
Owner Cleanup Bot authored
This suggested change is automatically generated based on group memberships and affiliations. If this change is unnecessary or in error, vote CR -1 and the bot will abandon it. Vote CR +1/2 to approve this change. See the owner's recent activity for context: https://android-review.googlesource.com/q/jaggies@google.com To report an issue, file a bug in the Infra>Codereview component. Change-Id: I6213bcb47c9439e88ab7515eb32af285d84dc866
-
Owner Cleanup Bot authored
This suggested change is automatically generated based on group memberships and affiliations. If this change is unnecessary or in error, vote CR -1 and the bot will abandon it. Vote CR +1/2 to approve this change. See the owner's recent activity for context: https://android-review.googlesource.com/q/jaggies@google.com To report an issue, file a bug in the Infra>Codereview component. Change-Id: I511edf8c6821b530b46f4133dc647f8cae6cfa76
-
Owner Cleanup Bot authored
This suggested change is automatically generated based on group memberships and affiliations. If this change is unnecessary or in error, vote CR -1 and the bot will abandon it. Vote CR +1/2 to approve this change. See the owner's recent activity for context: https://android-review.googlesource.com/q/jaggies@google.com To report an issue, file a bug in the Infra>Codereview component. Change-Id: I93dbbae3e2956ee46825e24b826e4839075f3113
-
Owner Cleanup Bot authored
This suggested change is automatically generated based on group memberships and affiliations. If this change is unnecessary or in error, vote CR -1 and the bot will abandon it. Vote CR +1/2 to approve this change. See the owner's recent activity for context: https://android-review.googlesource.com/q/jaggies@google.com To report an issue, file a bug in the Infra>Codereview component. Change-Id: I8e5df95fab63e382e3e2cbcbd319b2417f1f09a7
-
Owner Cleanup Bot authored
This suggested change is automatically generated based on group memberships and affiliations. If this change is unnecessary or in error, vote CR -1 and the bot will abandon it. Vote CR +1/2 to approve this change. See the owner's recent activity for context: https://android-review.googlesource.com/q/jaggies@google.com To report an issue, file a bug in the Infra>Codereview component. Change-Id: I2c2e9a1ea804baa6c3541cbd9e720a30233af179
-
- Feb 09, 2024
-
-
Christopher Ferris authored
When trying to decode a file, using a stream can lead to a pathological case where the entire file is read into memory. If a large file is encountered, the entire file will be read into memory and result in different types of crashes. So instead of using a stream, use a file descriptor to prevent this case. Bug: 309868782 Test: Put a large file on the system. Start the files app and Test: observe no crashes. Change-Id: I59cbab80af68eb3da4b46df81a5c26bf041778d8
-
Nadia Agueeva authored
-
Ryan Tao authored
-
Treehugger Robot authored
-
Shunkai Yao authored
-
Song Chun Fan authored
Change-Id: I1434ba6d39b3af255fd3f3c6a3f89a00b624248f BUG: 324148141 Test: n/a
-
Maciej Żenczykowski authored
-
Richard MacGregor authored
-
Treehugger Robot authored
-
Ryan Tao authored
Change-Id: I121d8cee49829f120440e9628aa319a5c664aee3
-
Varun Narisetty authored
Change-Id: I5a372543f22002762c16044d2228c07d94825529
-
Natalie Alowayed authored
Change-Id: I73f932819c5e195e3186f1fd2806279df7f67aee
-
Julie Pan authored
-