| commit | 6e49735aa8787e872c0530110be2c24715e805f5 | [log] [tgz] |
|---|---|---|
| author | Donald Chai <dchai@google.com> | Sun May 19 21:07:50 2019 -0700 |
| committer | Donald Chai <dchai@google.com> | Wed Jun 05 19:42:42 2019 -0700 |
| tree | 96aa153163bfac849b50b323af7f163bd466e2b6 | |
| parent | e7ac92cd7d47c53feb70e0ef3fd820affee87863 [diff] |
[aapt2] Ensure that attributes have declared namespaces when fixing manifest aapt::ManifestFixer might add attributes from http://schemas.android.com/apk/res/android when none existed previously. When that happens, something needs to ensure that the schema has a declared xmlns prefix, otherwise this: https://github.com/bazelbuild/bazel/blob/bdb3e97c47188ed4e5b472a3b7393e9588c93011/src/tools/android/java/com/google/devtools/build/android/aapt2/ResourceLinker.java#L560 will crash. It's a bit odd that Bazel converts back and forth between protobuf and XML, but as long as the fields exist in Resources.proto, they should be set. Bug: 133004752 Change-Id: I73fb3f713be34c9f81753fde2fd24c4524656b7d Tested: aapt2_tests