Reverts "add run_gradle_lock_files_check.dart for new PR's that modify gradle files (flutter#169245)" (flutter#170770)

<!-- start_original_pr_link -->
Reverts: flutter#169245
<!-- end_original_pr_link -->
<!-- start_initiating_author -->
Initiated by: matanlurey
<!-- end_initiating_author -->
<!-- start_revert_reason -->
Reason for reverting: Increased the time of the test task
`gradle_java8_compile_test` from about 5 minutes per run to over 30m,
which hits the timeout causing tree failures.


https://ci.chromium.org/ui/p/flutter/builders/luci.flutter.prod/Linux%20gradle_java8_compile_test
<!-- end_revert_reason -->
<!-- start_original_pr_author -->
Original PR Author: ash2moon
<!-- end_original_pr_author -->

<!-- start_reviewers -->
Reviewed By: {matanlurey}
<!-- end_reviewers -->

<!-- start_revert_body -->
This change reverts the following previous change:
This commit checks if there have been any changes to the gradle files
and if gradle lock files need updating. If so, it will alert then PR
owner to update their PR with the new gradle lock file.s

Fixes flutter#163228

Example test run:
https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8712427303492230593/+/u/run_gradle_java8_compile_test/stdout?format=raw

This was added as an existing test under Linux gradle_java8_compile_test
because benchmarking results showed on average about 3min 30s to run on
my local M2 Mac. SInce it is not > 10min to run, placing it inside an
existing test case was ideal.

## Pre-launch Checklist

- [x] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [x] I read the [Tree Hygiene] wiki page, which explains my
responsibilities.
- [x] I read and followed the [Flutter Style Guide], including [Features
we expect every widget to implement].
- [x] I signed the [CLA].
- [x] I listed at least one issue that this PR fixes in the description
above.
- [x] I updated/added relevant documentation (doc comments with `///`).
- [x] I added new tests to check the change I am making, or this PR is
[test-exempt].
- [x] I followed the [breaking change policy] and added [Data Driven
Fixes] where supported.
- [x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel
on [Discord].

<!-- Links -->
[Contributor Guide]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#overview
[Tree Hygiene]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md
[test-exempt]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#tests
[Flutter Style Guide]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md
[Features we expect every widget to implement]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md#features-we-expect-every-widget-to-implement
[CLA]: https://cla.developers.google.com/
[flutter/tests]: https://github.com/flutter/tests
[breaking change policy]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#handling-breaking-changes
[Discord]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Chat.md
[Data Driven Fixes]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Data-driven-Fixes.md

<!-- end_revert_body -->

Co-authored-by: auto-submit[bot] <[email protected]>

Author: 2 people

.ci.yaml

@@ -909,10 +909,6 @@ targets:
       - .ci.yaml
       - engine/**
       - DEPS
-      - /**/*.gradle
-      - /**/*.gradle.kts
-      - /**/gradle-wrapper.properties
-      - /**/*.lockfile
 
   - name: Linux gradle_plugin_bundle_test
     recipe: devicelab/devicelab_drone

dev/devicelab/bin/tasks/gradle_java8_compile_test.dart

@@ -8,19 +8,11 @@ import 'package:flutter_devicelab/framework/apk_utils.dart';
 import 'package:flutter_devicelab/framework/framework.dart';
 import 'package:flutter_devicelab/framework/task_result.dart';
 import 'package:flutter_devicelab/framework/utils.dart';
-import 'package:flutter_devicelab/tasks/gradle_lock_files_check.dart';
 import 'package:path/path.dart' as path;
 
 Future<void> main() async {
   await task(() async {
     try {
-      section('Gradle lockfiles check');
-      try {
-        await runGradleLockFilesCheck();
-      } catch (e) {
-        return TaskResult.failure(e.toString());
-      }
-
       await runPluginProjectTest((FlutterPluginProject pluginProject) async {
         section('check main plugin file exists');
         final File pluginMainKotlinFile = File(

dev/devicelab/lib/framework/utils.dart

@@ -324,18 +324,6 @@ Future<void> forceQuitRunningProcesses() async {
   _runningProcesses.clear();
 }
 
-/// Type definition of the [exec] function.
-typedef ExecFunction =
-    Future<int> Function(
-      String executable,
-      List<String> arguments, {
-      Map<String, String>? environment,
-      bool canFail,
-      String? workingDirectory,
-      StringBuffer? output,
-      StringBuffer? stderr,
-    });
-
 /// Executes a command and returns its exit code.
 Future<int> exec(
   String executable,
@@ -436,20 +424,6 @@ Future<void> forwardStandardStreams(
   return Future.wait<void>(<Future<void>>[stdoutDone.future, stderrDone.future]);
 }
 
-/// Type definition of the [eval] function.
-typedef EvalFunction =
-    Future<String> Function(
-      String executable,
-      List<String> arguments, {
-      Map<String, String>? environment,
-      bool canFail,
-      String? workingDirectory,
-      StringBuffer? stdout,
-      StringBuffer? stderr,
-      bool printStdout,
-      bool printStderr,
-    });
-
 /// Executes a command and returns its standard output as a String.
 ///
 /// For logging purposes, the command's output is also printed out by default.