From fe68c6b586e96aff48259f78f82568c59260a1ef Mon Sep 17 00:00:00 2001 From: Googler Date: Tue, 6 Feb 2018 12:36:27 -0800 Subject: Always do filtering in execution (even after doing it in analysis) Filtering only in analysis was neglecting the possibility of resources being in filesets, the contents of which are not available in analysis. As such, we must *always* filter in execution, even though it's usually just going to be a no-op. Also, add some documentation of same. RELNOTES: none PiperOrigin-RevId: 184722564 --- .../build/android/Aapt2ResourcePackagingAction.java | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) (limited to 'src/tools/android/java/com/google/devtools/build/android/Aapt2ResourcePackagingAction.java') diff --git a/src/tools/android/java/com/google/devtools/build/android/Aapt2ResourcePackagingAction.java b/src/tools/android/java/com/google/devtools/build/android/Aapt2ResourcePackagingAction.java index ac072dbf4a..408f7314e6 100644 --- a/src/tools/android/java/com/google/devtools/build/android/Aapt2ResourcePackagingAction.java +++ b/src/tools/android/java/com/google/devtools/build/android/Aapt2ResourcePackagingAction.java @@ -87,15 +87,10 @@ public class Aapt2ResourcePackagingAction { final Path compiledResources = Files.createDirectories(tmp.resolve("compiled")); final Path linkedOut = Files.createDirectories(tmp.resolve("linked")); - final List densities; - if (options.densities.isEmpty()) { - // aapt2 always needs to filter on densities, as the resource filtering from analysis is - // disregarded. - // TODO(b/70335064): Remove this once we never filter in analysis when building for aapt2. - densities = options.densitiesForManifest; - } else { - densities = options.densities; - } + + // TODO(b/72995408): Remove the densitiesForManifest option once it is no longer being passed. + final List densities = + options.densities.isEmpty() ? options.densitiesForManifest : options.densities; profiler.recordEndOf("setup").startTask("merging"); -- cgit v1.2.3