Add src_files to src_jar GN template

base/BUILD.gn

 # Copyright (c) 2013 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.
 
 # HOW TO WRITE CONDITIONALS IN THIS FILE
 # ======================================
 #
 # In many other places, one would write a conditional that expresses all the
 # cases when a source file is used or unused, and then either add or subtract
 # it from the sources list in that case
@@ skipping 2235 matching lines @@
   generate_jar_jni("android_runtime_jni_headers") {
     jni_package = "base"
     classes = [ "java/lang/Runtime.class" ]
   }
 
   # GYP: //base.gyp:base_java
   android_library("base_java") {
     srcjar_deps = [
       ":base_android_java_enums_srcjar",
       ":base_build_config_gen",
+      ":base_java_srcjar",

[agrieve, 2016/09/20 00:09:41]

Putting these files into a srcjar will make them uneditable in Android Studio
(https://chromium.googlesource.com/chromium/src.git/+/master/docs/android_stud...)

I think it's also the case that you don't really need these to be zipped, you
just need the list of sources. If this is true, I'd recommend using the .sources
file that's written as a part of gn gen here:
https://cs.chromium.org/chromium/src/build/config/android/internal_rules.gni?...

[kapishnikov, 2016/09/20 14:59:31]

That is a very good point. Thanks for the idea of using ".sources" file. I will
change it.

       ":base_native_libraries_gen",
     ]
 
     deps = [
       "//third_party/android_tools:android_support_annotations_java",
       "//third_party/android_tools:android_support_multidex_java",
       "//third_party/jsr-305:jsr_305_javalib",
     ]
 
-    java_files = [
+    # New versions of BuildConfig.java and NativeLibraries.java
+    # (with the actual correct values) will be created when creating an apk.
+    jar_excluded_patterns = [
+      "*/BuildConfig.class",
+      "*/NativeLibraries.class",
+      "*/NativeLibraries##*.class",
+    ]
+  }
+
+  # Creates srcjar that contains sources for "base_java".
+  zip("base_java_srcjar") {
+    inputs = [
       "android/java/src/org/chromium/base/ActivityState.java",
       "android/java/src/org/chromium/base/AnimationFrameTimeHistogram.java",
       "android/java/src/org/chromium/base/ApiCompatibilityUtils.java",
       "android/java/src/org/chromium/base/ApkAssets.java",
       "android/java/src/org/chromium/base/ApplicationStatus.java",
       "android/java/src/org/chromium/base/BaseChromiumApplication.java",
       "android/java/src/org/chromium/base/BaseSwitches.java",
       "android/java/src/org/chromium/base/BuildInfo.java",
       "android/java/src/org/chromium/base/Callback.java",
       "android/java/src/org/chromium/base/CollectionUtil.java",
@@ skipping 43 matching lines @@
       "android/java/src/org/chromium/base/library_loader/LibraryLoader.java",
       "android/java/src/org/chromium/base/library_loader/Linker.java",
       "android/java/src/org/chromium/base/library_loader/LoaderErrors.java",
       "android/java/src/org/chromium/base/library_loader/ModernLinker.java",
       "android/java/src/org/chromium/base/library_loader/NativeLibraryPreloader.java",
       "android/java/src/org/chromium/base/library_loader/ProcessInitException.java",
       "android/java/src/org/chromium/base/metrics/RecordHistogram.java",
       "android/java/src/org/chromium/base/metrics/RecordUserAction.java",
       "android/java/src/org/chromium/base/multidex/ChromiumMultiDexInstaller.java",
     ]
-
-    # New versions of BuildConfig.java and NativeLibraries.java
-    # (with the actual correct values) will be created when creating an apk.
-    jar_excluded_patterns = [
-      "*/BuildConfig.class",
-      "*/NativeLibraries.class",
-      "*/NativeLibraries##*.class",
-    ]
+    output = "$target_gen_dir/$target_name.srcjar"
+    base_dir = rebase_path("//base/android/java/src")
   }
 
   # GYP: //base.gyp:base_javatests
   android_library("base_javatests") {
     testonly = true
     deps = [
       ":base_java",
       ":base_java_test_support",
     ]
     java_files = [
@@ skipping 108 matching lines @@
     testonly = true
     deps = [
       ":base_java",
     ]
     java_files = [
       "test/android/java/src/org/chromium/base/ContentUriTestUtils.java",
       "test/android/java/src/org/chromium/base/TestSystemMessageHandler.java",
     ]
   }
 }