tree: 714e04a2f6d3907fa188bcbcd0fa9aa00c77cda1 [path history] [tgz]
  1. res/
  2. res_autofill_assistant/
  3. res_chromium/
  4. res_download/
  5. res_night/
  6. res_template/
  7. res_vr/
  8. src/
  9. strings/
  10. AndroidManifest.xml
  11. AndroidManifest_monochrome.xml
  12. AndroidManifest_trichrome_chrome.xml
  13. AndroidManifest_trichrome_library.xml
  14. apk_for_test.flags
  15. ChromeVersionConstants.java.version
  16. DEPS
  17. monochrome_public_apk.proguard_flags.expected
  18. OWNERS
  19. proguard.flags
  20. README.md
  21. ResourceId.template
  22. trichrome.flags
  23. version_strings.xml.template
chrome/android/java/README.md

//chrome/android/java/*proguard_flags.expected files

Proguard flags

Proguard is used in the build to obfuscate and minify Java code.

Proguard flags (also known as configs or rules) are used to specify which parts of Java code should not be optimized/obfuscated/modified by Proguard.

For example, the following rule specifies that all public classes with a public static void main(java.lang.String[]) method should not be modifed.

-keepclasseswithmembers public class * {
    public static void main(java.lang.String[]);
}

What are *.proguard_flags.expected files?

monochrome_public_apk.proguard_flags.expected contains all proguard configs used when building MonochromePublic.apk, and is generated by the proguard() build step.

Why do we care about proguard flag discrepancies?

Some configs are explicitly added (ex) while others are pulled in implicitly by GN deps (ex. aar_prebuilt() deps). In the later case, these config changes aren't reviewed and can sometimes result in crashes and/or size regressions. Example of where this has happened before:

  • Updating a prebuilt that supplies a Proguard config with a rule that is too broad (i.e. a rule that matches more classes than the intended ones)

Having checked-in versions of the Proguard configs used allows us to identify and address these issues earlier.

What is the build error telling me?

The build error is indicating that your CL has caused proguard rules to be added/removed/changed and that the expected file needs to be updated.

Fixing build failures

  1. Ensure your args.gn contains these args:
enable_chrome_android_internal = false
is_java_debug = false
  1. Run:
autoninja -C $CHROMIUM_OUTPUT_DIR monochrome_public_apk
  1. Run the command suggested in the error message to copy the contents of the generated proguard config file to the expected config file.

  2. Add the updated .expected file to your CL

  3. Afterwards, you can revert the args.gn changes suggested above and build normally

Trybot failures

On the android-binary-size trybot we set check_android_configuration=true which causes any differences between the expected and generated Proguard configs to fail the build.

Without this argument the error message is shown but doesn't fail the build.

Troubleshooting

Trybots fail but you can't reproduce locally

  • If a public target is failing, double check to make sure you've set enable_chrome_android_internal=false

Can't find the file suggested by the error message

  • Make sure is_java_debug=false

Updating the file doesn't fix the error

  • Make sure you're building monochrome_public_apk

Otherwise, please file a bug at crbug.com/new and/or message estevenson@.