[web] Don't run goldctl init more than once

testing/skia_gold_client/lib/skia_gold_client.dart

@@ -55,10 +55,6 @@ class SkiaGoldClient {
   String get _keysPath => path.join(workDirectory.path, 'keys.json');
   String get _failuresPath => path.join(workDirectory.path, 'failures.json');
 
-  /// Indicates whether the `goldctl` tool has been initialized for the current
-  /// test context.
-  bool _isInitialized = false;
-
   /// Indicates whether the client has already been authorized to communicate
   /// with the Skia Gold backend.
   bool get _isAuthorized {
@@ -121,10 +117,6 @@ class SkiaGoldClient {
   /// The `imgtest` command collects and uploads test results to the Skia Gold
   /// backend, the `init` argument initializes the current test.
   Future<void> _imgtestInit() async {
-    if (_isInitialized) {
-      return;
-    }
-
     final File keys = File(_keysPath);
     final File failures = File(_failuresPath);
 
@@ -165,7 +157,6 @@ class SkiaGoldClient {
         ..writeln('stderr: ${result.stderr}');
       throw Exception(buf.toString());
     }
-    _isInitialized = true;
   }
 
   /// Executes the `imgtest add` command in the `goldctl` tool.
@@ -226,7 +217,7 @@ class SkiaGoldClient {
     int pixelDeltaThreshold,
     double maxDifferentPixelsRate,
   ) async {
-    await _imgtestInit();
+    await _callOnce(_imgtestInit);
 
     final List<String> imgtestCommand = <String>[
       _goldctl,
@@ -253,10 +244,6 @@ class SkiaGoldClient {
   /// The `imgtest` command collects and uploads test results to the Skia Gold
   /// backend, the `init` argument initializes the current tryjob.
   Future<void> _tryjobInit() async {
-    if (_isInitialized) {
-      return;
-    }
-
     final File keys = File(_keysPath);
     final File failures = File(_failuresPath);
 
@@ -300,7 +287,6 @@ class SkiaGoldClient {
         ..writeln('stderr: ${result.stderr}');
       throw Exception(buf.toString());
     }
-    _isInitialized = true;
   }
 
   /// Executes the `imgtest add` command in the `goldctl` tool for tryjobs.
@@ -319,7 +305,7 @@ class SkiaGoldClient {
     int pixelDeltaThreshold,
     double differentPixelsRate,
   ) async {
-    await _tryjobInit();
+    await _callOnce(_tryjobInit);
 
     final List<String> tryjobCommand = <String>[
       _goldctl,
@@ -495,5 +481,12 @@ class SkiaGoldClient {
   }
 }
 
+Future<void>? _oneResult;
+Future<void> _callOnce(Future<void> Function() callback) async {
+  // If a call has already been made, return the result of that call.
+  _oneResult ??= callback();
+  return _oneResult;
+}
+
 /// Used to make HttpRequests during testing.
 class SkiaGoldHttpOverrides extends HttpOverrides { }