remove dead code, use constants when avilable, fix todo formatting

packages/flutter_tools/lib/src/android/android_sdk.dart

@@ -439,12 +439,6 @@ class AndroidSdk {
       return null;
     }
     return parseJavaVersion(result.stdout);
-//     const String versionOutput = '''
-// openjdk version "11.0.18" 2023-01-17 LTS
-// OpenJDK Runtime Environment Zulu11.62+17-CA (build 11.0.18+10-LTS)
-// OpenJDK 64-Bit Server VM Zulu11.62+17-CA (build 11.0.18+10-LTS, mixed mode)
-// ''';
-//     return AndroidSdk.parseJavaVersion(versionOutput);
   }
 
   // Java versions have 2 formats

packages/flutter_tools/lib/src/android/gradle_utils.dart

@@ -178,7 +178,6 @@ String getGradleVersionForAndroidPlugin(Directory directory, Logger logger) {
 /// [directory] should be and android directory with a build.gradle file.
 Future<String?> getGradleVersion(
     Directory directory, Logger logger, ProcessManager processManager) async {
-  // TODO make constant to share with other places in this file.
   final File propertiesFile = directory
       .childDirectory(gradleDirectoryName)
       .childDirectory(gradleWrapperDirectoryName)
@@ -300,7 +299,7 @@ String _formatParseWarning(String content) {
 bool validateGradleAndAgp(Logger logger,
     {required String? gradleV, required String? agpV}) {
 
-  // TODO are these the correct values we want to support?
+  // TODO(reidbaker): are these the correct values we want to support?
   const String oldestSupportedAgpVersion = '3.3.0';
   const String oldestSupportedGradleVersion = '4.10.1';
 
@@ -571,7 +570,8 @@ String getGradleVersionFor(String androidPluginVersion) {
   if (isWithinVersionRange(androidPluginVersion, min: maxKnownAgpVersion, max: '100.100')) {
     return _maxKnownAndSupportedGradleVersion;
   }
-  // TODO should this be a regular thow so that chris can see these in crash logging.
+  // TODO(reidbaker): should this be a regular thow so that @christopherfujino
+  // can see these in crash logging.
   throwToolExit('Unsupported Android Plugin version: $androidPluginVersion.');
 }
 

packages/flutter_tools/lib/src/project.dart

@@ -555,7 +555,6 @@ class AndroidProject extends FlutterProjectPlatform {
         hostAppGradleRoot, globals.logger, globals.processManager);
     final String? agpVersion =
         gradle.getAgpVersion(hostAppGradleRoot, globals.logger);
-    // TODO update getJavaVersion to take fewer globals.
     final String? javaVersion = globals.androidSdk?.getJavaVersion(
       androidStudio: globals.androidStudio,
       fileSystem: globals.fs,

packages/flutter_tools/test/general.shard/project_test.dart

@@ -6,6 +6,7 @@ import 'package:file/file.dart';
 import 'package:file/memory.dart';
 import 'package:flutter_tools/src/android/android_sdk.dart';
 import 'package:flutter_tools/src/android/android_studio.dart';
+import 'package:flutter_tools/src/android/gradle_utils.dart' as gradle_utils;
 import 'package:flutter_tools/src/base/file_system.dart';
 import 'package:flutter_tools/src/base/logger.dart';
 import 'package:flutter_tools/src/base/os.dart';
@@ -1337,9 +1338,9 @@ void addRootGradleFile(Directory directory,
 void addGradleWrapperFile(Directory directory, String gradleVersion) {
   directory
       .childDirectory('android')
-      .childDirectory('gradle')
-      .childDirectory('wrapper')
-      .childFile('gradle-wrapper.properties')
+      .childDirectory(gradle_utils.gradleDirectoryName)
+      .childDirectory(gradle_utils.gradleWrapperDirectoryName)
+      .childFile(gradle_utils.gradleWrapperPropertiesFilename)
     ..createSync(recursive: true)
     // ignore: unnecessary_string_escapes
     ..writeAsStringSync('''