Android: Remove enable_turbine flag (reland)

Now that r8 has been updated to remove unnecessary bridge methods and
turbine is turned back on for all targets, and robolectric has been
moved into android_deps, we no longer need this enable_turbine flag. Now
we can assume that all non-prebuilt java targets have a turbine header.

This is a reland of https://crrev.com/c/2219670.
Need to wait until https://crrev.com/c/2219111 lands first.

No code change except to remove now-obsolete third_party changes.

Tbr: agrieve@chromium.org
Bug: 1056299
Change-Id: I1c31234ff3d6e009428664e9327a6138e91c4e22
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2220246Reviewed-by: Peter Wen <wnwen@chromium.org>
Commit-Queue: Peter Wen <wnwen@chromium.org>
Auto-Submit: Peter Wen <wnwen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#772894}

build/config/android/internal_rules.gni

@@ -3143,9 +3143,6 @@ if (enable_java_templates) {
   #    ones.
   #  deps: Dependencies for this target.
   #  testonly: True iff target should only be used for tests.
-  #  enable_turbine: If exists then will be used to determine whether to run
-  #    turbine or not. Useful for disabling turbine headers for problematic
-  #    targets.
   #  chromium_code: Optional. Whether this is Chromium-specific code. If not
   #    provided, this is determined automatically, based on the location of
   #    the source files (i.e. anything under third_party/ is not
@@ -3313,11 +3310,6 @@ if (enable_java_templates) {
         _final_jar_path = invoker.final_jar_path
       }
 
-      _enable_turbine = _has_sources
-      if (defined(invoker.enable_turbine)) {
-        _enable_turbine = invoker.enable_turbine
-      }
-
       # For static libraries, the javac jar output is created at the intermediate
       # path so that it can be processed by another target and moved to the final
       # spot that the .build_config knows about. Technically this should be done
@@ -3326,7 +3318,7 @@ if (enable_java_templates) {
       _build_config_jar_path = _final_jar_path
       _final_ijar_path = get_path_info(_final_jar_path, "dir") + "/" +
                          get_path_info(_final_jar_path, "name")
-      if (_enable_turbine) {
+      if (_has_sources) {
         _final_ijar_path += ".turbine.jar"
       } else {
         _final_ijar_path += ".ijar.jar"
@@ -3592,26 +3584,20 @@ if (enable_java_templates) {
         _annotation_processor_deps = invoker.annotation_processor_deps
       }
 
-      if (_enable_turbine) {
-        compile_java_helper(_header_target_name) {
-          forward_variables_from(invoker, _compile_java_forward_variables)
-          use_turbine = true
-          output_jar_path = _final_ijar_path
-          generated_jar_path = _generated_jar_path
-          deps = _annotation_processor_deps
-        }
+      compile_java_helper(_header_target_name) {
+        forward_variables_from(invoker, _compile_java_forward_variables)
+        use_turbine = true
+        output_jar_path = _final_ijar_path
+        generated_jar_path = _generated_jar_path
+        deps = _annotation_processor_deps
       }
 
       _compile_java_target = "${_main_target_name}__compile_java"
       compile_java_helper(_compile_java_target) {
         forward_variables_from(invoker, _compile_java_forward_variables)
         output_jar_path = _javac_jar_path
-        if (_enable_turbine) {
-          deps = [ ":$_header_target_name" ]
-          generated_jar_path = _generated_jar_path
-        } else {
-          deps = _annotation_processor_deps
-        }
+        deps = [ ":$_header_target_name" ]
+        generated_jar_path = _generated_jar_path
       }
       if (_enable_errorprone) {
         _compile_java_errorprone_target = "${_main_target_name}__errorprone"
@@ -3624,12 +3610,8 @@ if (enable_java_templates) {
             }
             javac_args += invoker.errorprone_args
           }
-          if (_enable_turbine) {
-            deps = [ ":$_header_target_name" ]
-            generated_jar_path = _generated_jar_path
-          } else {
-            deps = _annotation_processor_deps
-          }
+          deps = [ ":$_header_target_name" ]
+          generated_jar_path = _generated_jar_path
           output_jar_path = "$target_out_dir/$target_name.errorprone.stamp"
         }
         _analysis_public_deps += [ ":$_compile_java_errorprone_target" ]
@@ -3637,7 +3619,7 @@ if (enable_java_templates) {
     }  # _has_sources
 
     if (_is_prebuilt || _has_sources) {
-      if (!_enable_turbine) {
+      if (_is_prebuilt) {
         generate_interface_jar(_header_target_name) {
           # Always used the unfiltered .jar to create the interface jar so that
           # other targets will resolve filtered classes when depending on