fix: replace homegrown mutex with the synchronized package

lib/src/imperative.dart

@@ -141,35 +141,19 @@ class TrayIcon {
     _logger.fine("disposed", null, _disposedAt);
   }
 
-  /// Mutex guard for [_isReal]
-  bool _makingReal = false;
-
-  final _waitingUntilMadeReal = Queue<Completer<void>>();
+  final _lock = Lock();
 
   /// Ensures the icon has been constructed in native code.
   ///
   /// This is deferred until usage, because constructors can't be `async`.
   Future<void> _ensureIsReal() async {
-    if (_makingReal) {
-      final completer = Completer();
-      _waitingUntilMadeReal.addLast(completer);
-      await completer.future;
-    }
-    _makingReal = true;
-    try {
+    await _lock.synchronized(() async {
       if (!_isReal) {
         await _plugin.addTray(_id);
         _isReal = true;
-        final njobs = _waitingUntilMadeReal.length;
-        while (_waitingUntilMadeReal.isNotEmpty) {
-          _waitingUntilMadeReal.removeFirst().complete();
-        }
-        _logger
-            .fine("made real (created in native code). unblocked $njobs tasks");
+        _logger.fine("made real (created in native code).");
       }
-    } finally {
-      _makingReal = false;
-    }
+    });
   }
 
   /// Tests if this instance has already been disposed of.

lib/src/plugin.dart

@@ -2,14 +2,15 @@
 /// However, note that trying to interfere with icons managed by a [TrayIcon]´
 /// will break stuff.
 import 'dart:async';
-import 'dart:collection';
 import 'dart:math';
 import 'dart:typed_data';
 
 import 'package:flutter/foundation.dart';
 import 'package:flutter/services.dart';
 import 'package:flutter/widgets.dart';
 import 'package:logging/logging.dart';
+import 'package:synchronized/extension.dart';
+import 'package:synchronized/synchronized.dart';
 
 import 'image.dart';
 import 'stock_icon.dart';

pubspec.yaml

@@ -17,6 +17,7 @@ dependencies:
     sdk: flutter
   logging: ^1.1.1
   path: ^1.8.2
+  synchronized: ^3.0.1
 
 dev_dependencies:
   flutter_lints: ^2.0.1