👾 GN(android): Filter R.class files before obfuscating via proguard

build/config/android/internal_rules.gni

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import("//build/config/android/config.gni")
 import("//build/config/sanitizers/sanitizers.gni")
 import("//build/config/zip.gni")
 import("//third_party/ijar/ijar.gni")
 
 assert(is_android)
@@ skipping 585 matching lines @@
 
   _proguard_preprocess =
       defined(invoker.proguard_preprocess) && invoker.proguard_preprocess
   _input_jar_path = invoker.input_jar_path
   _output_jar_path = invoker.output_jar_path
 
   _jar_excluded_patterns = []
   if (defined(invoker.jar_excluded_patterns)) {
     _jar_excluded_patterns = invoker.jar_excluded_patterns
   }
-
   _strip_resource_classes =
       defined(invoker.strip_resource_classes) && invoker.strip_resource_classes
+  _filter_jar = _jar_excluded_patterns != [] || _strip_resource_classes
 
-  if (_jar_excluded_patterns != [] || _strip_resource_classes) {
-    _filter_class_files_input_file = _input_jar_path
-    if (_proguard_preprocess) {
-      _filter_class_files_input_file =
-          "$target_out_dir/$target_name-proguarded.jar"
-    }
+  if (_filter_jar) {
+    _filter_target = "${target_name}__filter"
+    _output_jar_target = _filter_target
+  }
+  if (_proguard_preprocess) {
+    _proguard_target = "${target_name}__proguard_process"
+    _output_jar_target = _proguard_target
+  }
+  if (!_filter_jar && !_proguard_preprocess) {
+    _copy_target = "${target_name}__copy"
+    _output_jar_target = _copy_target
   }
 
-  if (_proguard_preprocess) {
-    _proguard_config_path = invoker.proguard_config
-    _proguard_target = "${target_name}__proguard_process"
-    proguard(_proguard_target) {
+  if (_filter_jar) {
+    _filtered_jar_path = _output_jar_path
+    if (_proguard_preprocess) {
+      _filtered_jar_path = "$target_out_dir/$target_name-filtered.jar"
+    }
+    action(_filter_target) {
+      script = "//build/android/gyp/jar.py"
       forward_variables_from(invoker,
                              [
                                "deps",
                                "public_deps",
                              ])
       inputs = [
         _build_config,
         _input_jar_path,
+      ]
+      outputs = [
+        _filtered_jar_path,
+      ]
+      args = [
+        "--input-jar",
+        rebase_path(_input_jar_path, root_build_dir),
+        "--jar-path",
+        rebase_path(_filtered_jar_path, root_build_dir),
+        "--excluded-classes=$_jar_excluded_patterns",
+      ]
+      if (_strip_resource_classes) {
+        args += [ "--strip-resource-classes-for=@FileArg($_rebased_build_config:javac:resource_packages)" ]
+      }
+    }
+  }
+
+  if (_proguard_preprocess) {
+    _proguard_config_path = invoker.proguard_config
+    proguard(_proguard_target) {
+      if (_filter_jar) {
+        _proguard_input_jar = _filtered_jar_path
+        public_deps = [
+          ":$_filter_target",
+        ]
+      } else {
+        _proguard_input_jar = _input_jar_path
+        public_deps = []
+      }
+      if (defined(invoker.deps)) {
+        deps = invoker.deps
+      }
+      if (defined(invoker.public_deps)) {
+        public_deps += invoker.public_deps
+      }
+      inputs = [
+        _build_config,
         _proguard_config_path,
+        _proguard_input_jar,
       ]
       output_jar_path = _output_jar_path
-      if (defined(_filter_class_files_input_file)) {
-        output_jar_path = _filter_class_files_input_file
-      }
 
       _rebased_input_paths = [ rebase_path(_input_jar_path, root_build_dir) ]
       _rebased_proguard_configs =
           [ rebase_path(_proguard_config_path, root_build_dir) ]
       args = [
         "--input-paths=$_rebased_input_paths",
         "--proguard-configs=$_rebased_proguard_configs",
         "--classpath=@FileArg($_rebased_build_config:javac:classpath)",
       ]
     }
-  } else if (_jar_excluded_patterns == [] && !_strip_resource_classes) {
-    _output_jar_target = "${target_name}__copy_jar"
-    copy(_output_jar_target) {
+  } else if (!_filter_jar) {
+    copy(_copy_target) {
       forward_variables_from(invoker,
                              [
                                "deps",
                                "public_deps",
                              ])
       sources = [
         _input_jar_path,
       ]
       outputs = [
         _output_jar_path,
       ]
     }
   }
 
-  if (_jar_excluded_patterns != [] || _strip_resource_classes) {
-    _output_jar_target = "${target_name}__filter"
-    action(_output_jar_target) {
-      script = "//build/android/gyp/jar.py"
-      if (_proguard_preprocess) {
-        deps = [
-          ":$_proguard_target",
-        ]
-      } else {
-        forward_variables_from(invoker,
-                               [
-                                 "deps",
-                                 "public_deps",
-                               ])
-      }
-      inputs = [
-        _filter_class_files_input_file,
-      ]
-      outputs = [
-        _output_jar_path,
-      ]
-      args = [
-        "--input-jar",
-        rebase_path(_filter_class_files_input_file, root_build_dir),
-        "--jar-path",
-        rebase_path(_output_jar_path, root_build_dir),
-        "--excluded-classes=$_jar_excluded_patterns",
-      ]
-      if (_strip_resource_classes) {
-        args += [ "--strip-resource-classes-for=@FileArg($_rebased_build_config:javac:resource_packages)" ]
-      }
-    }
-  } else if (_proguard_preprocess) {
-    _output_jar_target = _proguard_target
-  }
-
   group(target_name) {
     forward_variables_from(invoker,
                            [
                              "data_deps",
                              "visibility",
                            ])
     public_deps = [
       ":$_output_jar_target",
     ]
   }
@@ skipping 899 matching lines @@
     build_config = _build_config
     input_jar_path = _javac_jar_path
     output_jar_path = _process_prebuilt_jar_path
     if (defined(invoker.proguard_preprocess) && invoker.proguard_preprocess) {
       proguard_preprocess = invoker.proguard_preprocess
       proguard_config = invoker.proguard_config
     }
     deps = [
       ":$_javac_target_name",
     ]
+    if (defined(invoker.deps)) {
+      deps += invoker.deps
+    }
   }
 
   emma_instr(_emma_instr_target_name) {
     visibility = [
       ":$_ijar_target_name",
       ":$_final_target_name",
     ]
 
     forward_variables_from(invoker, [ "emma_instrument" ])
 
@@ skipping 684 matching lines @@
 
     args = [
       "--depfile",
       rebase_path(depfile, root_build_dir),
       "--script-output-path",
       rebase_path(generated_script, root_build_dir),
     ]
     args += test_runner_args
   }
 }