Revert "Reland "Cronet: Simplify Android GN logic for srcjars by not using them""

This reverts commit 282d95d7.

Reason for revert: Still has a missing dep:
https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8928671892382176864/+/steps/compile__with_patch_/0/stdout

Original change's description:
> Reland "Cronet: Simplify Android GN logic for srcjars by not using them"
> 
> No need to zip up the generated source just to unzip it again :P
> 
> This reverts commit f4e7bed3.
> 
> Reason for reland: Fixed missing deps in __final_jni template
> 
> TBR=pauljensen@chromium.org,agrieve@chromium.org
> 
> Bug: 908479
> Change-Id: I1177d550eb33737f47192bd70db5358c6392598c
> Reviewed-on: https://chromium-review.googlesource.com/c/1351930
> Commit-Queue: agrieve <agrieve@chromium.org>
> Reviewed-by: agrieve <agrieve@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#611108}

TBR=pauljensen@chromium.org,agrieve@chromium.org

Change-Id: I5f10f94e023f7413fb19d85624013fbddcb732a5
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 908479
Reviewed-on: https://chromium-review.googlesource.com/c/1352516Reviewed-by: agrieve <agrieve@chromium.org>
Commit-Queue: agrieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#611329}

build/config/android/rules.gni

@@ -423,9 +423,6 @@ if (enable_java_templates) {
       deps = [
         "${invoker.target}__build_config",
       ]
-      if (defined(invoker.deps)) {
-        deps += invoker.deps
-      }
       inputs = [
         _build_config,
       ]
@@ -1298,7 +1295,6 @@ if (enable_java_templates) {
     _outer_target_name = target_name
     generate_jni_registration(_jni_srcjar_target) {
       target = ":$_outer_target_name"
-      deps = _deps
     }
 
     java_library_impl(_java_binary_target_name) {
@@ -2438,7 +2434,6 @@ if (enable_java_templates) {
         if (defined(invoker.jni_sources_blacklist)) {
           sources_blacklist = invoker.jni_sources_blacklist
         }
-        deps = _deps
       }
       _srcjar_deps += [ ":${_template_name}__final_jni" ]
     }

components/cronet/android/BUILD.gn

@@ -14,7 +14,6 @@ import("//third_party/protobuf/proto_library.gni")
 import("//url/features.gni")
 
 _jni_registration_header = "$target_gen_dir/cronet_jni_registration.h"
-_templates_dir = "$target_gen_dir/templates"
 
 declare_args() {
   # In integrated mode, CronetEngine will use the shared network task runner by
@@ -110,10 +109,13 @@ java_cpp_template("integrated_mode_state") {
   }
 }
 
-_generated_api_version_java = "$_templates_dir/org/chromium/net/ApiVersion.java"
-_api_level = read_file("api_version.txt", "value")
+_generated_api_version_java_dir =
+    "$target_gen_dir/templates/cronet_api_version_java"
+_generated_api_version_java =
+    "$_generated_api_version_java_dir/org/chromium/net/ApiVersion.java"
 
-process_version("api_version") {
+process_version("cronet_api_version_java") {
+  _api_level = read_file("api_version.txt", "value")
   template_file = "api/src/org/chromium/net/ApiVersion.template"
   sources = [
     "//chrome/VERSION",
@@ -126,10 +128,24 @@ process_version("api_version") {
   output = _generated_api_version_java
 }
 
+zip("cronet_api_version_srcjar") {
+  inputs = [
+    _generated_api_version_java,
+  ]
+  output = "$target_gen_dir/$target_name.srcjar"
+  base_dir = _generated_api_version_java_dir
+  deps = [
+    ":cronet_api_version_java",
+  ]
+}
+
+_generated_impl_version_java_dir =
+    "$target_gen_dir/templates/cronet_impl_version_java"
 _generated_impl_version_java =
-    "$_templates_dir/org/chromium/net/impl/ImplVersion.java"
+    "$_generated_impl_version_java_dir/org/chromium/net/impl/ImplVersion.java"
 
-process_version("impl_version") {
+process_version("cronet_impl_version_java") {
+  _api_level = read_file("api_version.txt", "value")
   template_file = "java/src/org/chromium/net/impl/ImplVersion.template"
   sources = [
     "//chrome/VERSION",
@@ -142,6 +158,17 @@ process_version("impl_version") {
   output = _generated_impl_version_java
 }
 
+zip("cronet_impl_version_srcjar") {
+  inputs = [
+    _generated_impl_version_java,
+  ]
+  output = "$target_gen_dir/$target_name.srcjar"
+  base_dir = _generated_impl_version_java_dir
+  deps = [
+    ":cronet_impl_version_java",
+  ]
+}
+
 _cronet_version_header_include_dir = "$target_gen_dir/cronet_version_header"
 
 source_set("cronet_static") {
@@ -227,6 +254,8 @@ shared_library("cronet") {
   configs += [ ":hide_all_but_jni_onload_and_cronet" ]
 }
 
+cronet_api_srcjar_deps = [ ":cronet_api_version_srcjar" ]
+
 # cronet_api_java.jar defines Cronet API.
 android_library("cronet_api_java") {
   java_files = [
@@ -250,16 +279,17 @@ android_library("cronet_api_java") {
     "api/src/org/chromium/net/UploadDataSink.java",
     "api/src/org/chromium/net/UrlRequest.java",
     "api/src/org/chromium/net/UrlResponseInfo.java",
-    _generated_api_version_java,
   ]
 
   deps = [
-    ":api_version",
     "//third_party/android_deps:android_support_annotations_java",
   ]
+
+  srcjar_deps = cronet_api_srcjar_deps
 }
 
 cronet_impl_common_java_srcjar_deps = [
+  ":cronet_impl_version_srcjar",
   ":effective_connection_type_java",
   ":http_cache_type_java",
   ":integrated_mode_state",
@@ -287,14 +317,12 @@ android_library("cronet_impl_common_base_java") {
     "java/src/org/chromium/net/impl/UrlResponseInfoImpl.java",
     "java/src/org/chromium/net/impl/UserAgent.java",
     "java/src/org/chromium/net/impl/VersionSafeCallbacks.java",
-    _generated_impl_version_java,
   ]
 
   # Adding deps here won't include those deps in the cronet_impl_common_java.jar.
   # Please add to cronet_impl_common_java_deps_to_package instead.
   deps = [
     ":cronet_api_java",
-    ":impl_version",
     "//third_party/android_deps:android_support_annotations_java",
   ]
   deps += cronet_impl_common_java_deps_to_package
@@ -560,19 +588,14 @@ template("jar_src") {
 }
 
 jar_src("jar_cronet_api_source") {
-  src_search_dirs = [
-    "api/src",
-    _templates_dir,
-  ]
+  src_search_dirs = [ "api/src" ]
   source_deps = [ ":cronet_api_java" ]
+  srcjar_deps = cronet_api_srcjar_deps
   jar_path = "$_package_dir/cronet_api-src.jar"
 }
 
 jar_src("jar_cronet_impl_common_java_source") {
-  src_search_dirs = [
-    "java/src",
-    _templates_dir,
-  ]
+  src_search_dirs = [ "java/src" ]
   source_deps = [ ":cronet_impl_common_base_java" ]
   srcjar_deps = cronet_impl_common_java_srcjar_deps
   jar_path = "$_package_dir/cronet_impl_common_java-src.jar"