Add await to callsites of BasicMessageChannel.send by victorsanni · Pull Request #182868 · flutter/flutter

victorsanni · 2026-02-24T22:51:33Z

Impl:

  /// Sends the specified [message] to the platform plugins on this channel.
  ///
  /// Returns a [Future] which completes to the received response, which may
  /// be null.
  Future<T?> send(T message) async {
    return codec.decodeMessage(await binaryMessenger.send(name, codec.encodeMessage(message)));
  }

Part of #181513

gemini-code-assist

Code Review

This pull request adds await to callsites of BasicMessageChannel.send, which is now an async method. The changes are generally correct and necessary. However, I've found a couple of issues in the benchmark file dev/benchmarks/platform_channels_benchmarks/lib/main.dart. One change introduces a performance regression by serializing parallel operations, and another uses an async/await anti-pattern. I've provided suggestions to fix these issues.

gemini-code-assist · 2026-02-24T22:56:04Z

+    await basicStandard.send(payload).then((Object? result) {
      _runBasicStandardParallelRecurse(basicStandard, counter, count, completer, payload);
    });


Adding await inside this loop makes it sequential, which defeats the purpose of the _runBasicStandardParallel benchmark. Each iteration now waits for the send operation to complete before starting the next, removing the parallelism.

To restore the parallel behavior, the await should be removed.

basicStandard.send(payload).then((Object? result) { _runBasicStandardParallelRecurse(basicStandard, counter, count, completer, payload); });

Use await Future.forEach() or Future.wait instead.

gemini-code-assist · 2026-02-24T22:56:04Z

+    await basicStandard.send(payload).then((Object? result) {
      _runBasicStandardParallelRecurse(basicStandard, counter, count, completer, payload);
    });


Using await with .then() is an anti-pattern in modern Dart as it can be expressed more clearly using async/await. This also avoids creating a Future<Future<void>> which is likely not intended.

await basicStandard.send(payload); _runBasicStandardParallelRecurse(basicStandard, counter, count, completer, payload);

dkwingsmt · 2026-02-24T23:09:43Z

This future should have a .catch that reports error.

Is the current impl satisfactory?

dkwingsmt

Generally ok.

dkwingsmt · 2026-02-25T06:08:46Z

-    basicStandard.send(payload).then((Object? result) {
-      _runBasicStandardParallelRecurse(basicStandard, counter, count, completer, payload);
-    });
+    await Future.wait([


Future.wait is meant to run tasks in parallel and wait for the completion of all of them.

You should replace the entire for loop with something like (psuedocode):

await Future.wait(List.generate(parallel, (_) => () async { await basicStandard.send(payload); await _runBasicStandardParallelRecurse(basicStandard, counter, count, completer, payload); }));

victorsanni · 2026-02-25T19:28:02Z

+  await Future.wait(
+    Iterable.generate(parallel, (_) async {
+      await basicStandard.send(payload);
+      // ignore: unawaited_futures


Ignoring because recursive method is intended to run in parallel

Are you sure we can't have await here? I think this still runs in parallel even if you use await, since Iterable.generate runs everything at the same time. In other words, each (_) async { ... } runs in parallel.

dkwingsmt · 2026-02-27T18:41:36Z

-      _runBasicStandardParallelRecurse(basicStandard, counter, count, completer, payload);
-    });
+    await basicStandard.send(payload);
+    _runBasicStandardParallelRecurse(basicStandard, counter, count, completer, payload);


await (or return)

dkwingsmt · 2026-02-27T21:35:21Z

@@ -68,11 +67,12 @@ Future<double> _runBasicStandardParallel(
  final completer = Completer<int>();
  final counter = _Counter();
  watch.start();
-  for (var i = 0; i < parallel; ++i) {
-    basicStandard.send(payload).then((Object? result) {
-      _runBasicStandardParallelRecurse(basicStandard, counter, count, completer, payload);
-    });
-  }
+  await Future.wait(
+    Iterable.generate(parallel, (_) async {
+      await basicStandard.send(payload);
+      await _runBasicStandardParallelRecurse(basicStandard, counter, count, completer, payload);
+    }),
+  );


I'm sorry but I just realized that we might have been doing something wrong. _runBasicStandardParallelRecurse is using its completer parameter as the async control, and shouldn't need additional Future return value.

(I suspect that this entire function is basically reinventing Future.wait from scratch.)

In short, I think the original implementation is probably completely correct and we shouldn't change it at all. It doesn't have anything that violate the lint either, right? All futures either has .then or is awaited.

…/flutter#182868)

Roll Flutter from 46fb7210422d to d3dd7744e81f (33 revisions) flutter/flutter@46fb721...d3dd774 2026-03-04 15619084+vashworth@users.noreply.github.com Show warning when plugins do not support SwiftPM (flutter/flutter#182506) 2026-03-04 15619084+vashworth@users.noreply.github.com Give guided message when project is not compatible with SwiftPM (flutter/flutter#182394) 2026-03-04 danny@tuppeny.com Pass --web-define through to web runner when using --machine mode (flutter/flutter#183228) 2026-03-04 104147021+MohammedTarigg@users.noreply.github.com Improve SwiftPM minimum platform mismatch diagnostics (flutter/flutter#182375) 2026-03-04 vegorov@google.com Use dart::bin::SetupDartIo to setup dart:io (flutter/flutter#176714) 2026-03-04 engine-flutter-autoroll@skia.org Roll Skia from 3197848b14ad to ada0b7628c79 (5 revisions) (flutter/flutter#183221) 2026-03-04 engine-flutter-autoroll@skia.org Roll Skia from fe9e9f22c531 to 3197848b14ad (15 revisions) (flutter/flutter#183198) 2026-03-04 34465683+rkishan516@users.noreply.github.com refactor: remove material in reorderable_list_test, scroll_notification_test, scroll_physics_test, shortcuts_test, sliver_floating_header_test, snapshot_widget_test (flutter/flutter#182698) 2026-03-04 34465683+rkishan516@users.noreply.github.com refactor: remove material in pop_scope_test, route_notification_message_test, two_dimensional_utils, two_dimensional_viewport_test (flutter/flutter#182699) 2026-03-04 737941+loic-sharma@users.noreply.github.com Add dev/benchmarks/README.md (flutter/flutter#182976) 2026-03-03 jason-simmons@users.noreply.github.com Roll RapidJSON to a branch based on the current upstream head (flutter/flutter#183048) 2026-03-03 flar@google.com [Impeller] Update comments to reflect new info about 2-pass rendering (flutter/flutter#183050) 2026-03-03 47866232+chunhtai@users.noreply.github.com Add vmservices for accessibilityEvaluation (flutter/flutter#182791) 2026-03-03 engine-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from 0dCDM2oORHwDf_pyb... to JJw5EJ87vLGqFVl4h... (flutter/flutter#183177) 2026-03-03 lukas.klingsbo@gmail.com Support mixed color spaces in `Color.lerp` (flutter/flutter#182934) 2026-03-03 36861262+QuncCccccc@users.noreply.github.com Add warning when there is a widget with color between `Material` and `ListTile` (flutter/flutter#181402) 2026-03-03 41930132+hellohuanlin@users.noreply.github.com [ios]uitest for admob banner in scrollable list gesture issue (flutter/flutter#183128) 2026-03-03 engine-flutter-autoroll@skia.org Roll Packages from faa4e22 to 9083bc9 (4 revisions) (flutter/flutter#183164) 2026-03-03 15619084+vashworth@users.noreply.github.com Build App and native asset frameworks for Add to App FlutterPluginRegistrant (flutter/flutter#183136) 2026-03-03 engine-flutter-autoroll@skia.org Roll Skia from f886711f180d to fe9e9f22c531 (4 revisions) (flutter/flutter#183155) 2026-03-03 engine-flutter-autoroll@skia.org Roll Dart SDK from e86dbe9aa742 to c597ef90d2dc (2 revisions) (flutter/flutter#183147) 2026-03-03 codefu@google.com fix: bump matcher (flutter/flutter#183167) 2026-03-02 58529443+srujzs@users.noreply.github.com Use isA to test for exceptions (flutter/flutter#183129) 2026-03-02 dpxhfxywang@163.com [two_dimensional_scrollables] Fix tableview janks when first row/column pinned (flutter/flutter#180563) 2026-03-02 victorsanniay@gmail.com Add await to callsites of BasicMessageChannel.send (flutter/flutter#182868) 2026-03-02 137456488+flutter-pub-roller-bot@users.noreply.github.com Roll pub packages (flutter/flutter#183133) 2026-03-02 robert.ancell@canonical.com Improve FFI code for windowing (flutter/flutter#183098) 2026-03-02 kevinjchisholm@google.com [workflow] Update the changelog merge action to fetch the stable branch (flutter/flutter#183132) 2026-03-02 engine-flutter-autoroll@skia.org Roll Skia from e180358b7a7a to f886711f180d (2 revisions) (flutter/flutter#183130) 2026-03-02 jacksongardner@google.com Merge changelog from 3.41.3. (flutter/flutter#183131) 2026-03-02 50985133+SpiralMomentum@users.noreply.github.com Make TextDecoration final and unify maskValue across platforms (flutter/flutter#183070) 2026-03-02 137456488+flutter-pub-roller-bot@users.noreply.github.com Roll pub packages (flutter/flutter#182640) 2026-03-02 15619084+vashworth@users.noreply.github.com Enable SwiftPM by default on master and beta (flutter/flutter#182923) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC bmparr@google.com,stuartmorgan@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: ...

Impl: ```dart /// Sends the specified [message] to the platform plugins on this channel. /// /// Returns a [Future] which completes to the received response, which may /// be null. Future<T?> send(T message) async { return codec.decodeMessage(await binaryMessenger.send(name, codec.encodeMessage(message))); } ``` Part of flutter#181513

…r#11182) Roll Flutter from 46fb7210422d to d3dd7744e81f (33 revisions) flutter/flutter@46fb721...d3dd774 2026-03-04 15619084+vashworth@users.noreply.github.com Show warning when plugins do not support SwiftPM (flutter/flutter#182506) 2026-03-04 15619084+vashworth@users.noreply.github.com Give guided message when project is not compatible with SwiftPM (flutter/flutter#182394) 2026-03-04 danny@tuppeny.com Pass --web-define through to web runner when using --machine mode (flutter/flutter#183228) 2026-03-04 104147021+MohammedTarigg@users.noreply.github.com Improve SwiftPM minimum platform mismatch diagnostics (flutter/flutter#182375) 2026-03-04 vegorov@google.com Use dart::bin::SetupDartIo to setup dart:io (flutter/flutter#176714) 2026-03-04 engine-flutter-autoroll@skia.org Roll Skia from 3197848b14ad to ada0b7628c79 (5 revisions) (flutter/flutter#183221) 2026-03-04 engine-flutter-autoroll@skia.org Roll Skia from fe9e9f22c531 to 3197848b14ad (15 revisions) (flutter/flutter#183198) 2026-03-04 34465683+rkishan516@users.noreply.github.com refactor: remove material in reorderable_list_test, scroll_notification_test, scroll_physics_test, shortcuts_test, sliver_floating_header_test, snapshot_widget_test (flutter/flutter#182698) 2026-03-04 34465683+rkishan516@users.noreply.github.com refactor: remove material in pop_scope_test, route_notification_message_test, two_dimensional_utils, two_dimensional_viewport_test (flutter/flutter#182699) 2026-03-04 737941+loic-sharma@users.noreply.github.com Add dev/benchmarks/README.md (flutter/flutter#182976) 2026-03-03 jason-simmons@users.noreply.github.com Roll RapidJSON to a branch based on the current upstream head (flutter/flutter#183048) 2026-03-03 flar@google.com [Impeller] Update comments to reflect new info about 2-pass rendering (flutter/flutter#183050) 2026-03-03 47866232+chunhtai@users.noreply.github.com Add vmservices for accessibilityEvaluation (flutter/flutter#182791) 2026-03-03 engine-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from 0dCDM2oORHwDf_pyb... to JJw5EJ87vLGqFVl4h... (flutter/flutter#183177) 2026-03-03 lukas.klingsbo@gmail.com Support mixed color spaces in `Color.lerp` (flutter/flutter#182934) 2026-03-03 36861262+QuncCccccc@users.noreply.github.com Add warning when there is a widget with color between `Material` and `ListTile` (flutter/flutter#181402) 2026-03-03 41930132+hellohuanlin@users.noreply.github.com [ios]uitest for admob banner in scrollable list gesture issue (flutter/flutter#183128) 2026-03-03 engine-flutter-autoroll@skia.org Roll Packages from faa4e22 to 9083bc9 (4 revisions) (flutter/flutter#183164) 2026-03-03 15619084+vashworth@users.noreply.github.com Build App and native asset frameworks for Add to App FlutterPluginRegistrant (flutter/flutter#183136) 2026-03-03 engine-flutter-autoroll@skia.org Roll Skia from f886711f180d to fe9e9f22c531 (4 revisions) (flutter/flutter#183155) 2026-03-03 engine-flutter-autoroll@skia.org Roll Dart SDK from e86dbe9aa742 to c597ef90d2dc (2 revisions) (flutter/flutter#183147) 2026-03-03 codefu@google.com fix: bump matcher (flutter/flutter#183167) 2026-03-02 58529443+srujzs@users.noreply.github.com Use isA to test for exceptions (flutter/flutter#183129) 2026-03-02 dpxhfxywang@163.com [two_dimensional_scrollables] Fix tableview janks when first row/column pinned (flutter/flutter#180563) 2026-03-02 victorsanniay@gmail.com Add await to callsites of BasicMessageChannel.send (flutter/flutter#182868) 2026-03-02 137456488+flutter-pub-roller-bot@users.noreply.github.com Roll pub packages (flutter/flutter#183133) 2026-03-02 robert.ancell@canonical.com Improve FFI code for windowing (flutter/flutter#183098) 2026-03-02 kevinjchisholm@google.com [workflow] Update the changelog merge action to fetch the stable branch (flutter/flutter#183132) 2026-03-02 engine-flutter-autoroll@skia.org Roll Skia from e180358b7a7a to f886711f180d (2 revisions) (flutter/flutter#183130) 2026-03-02 jacksongardner@google.com Merge changelog from 3.41.3. (flutter/flutter#183131) 2026-03-02 50985133+SpiralMomentum@users.noreply.github.com Make TextDecoration final and unify maskValue across platforms (flutter/flutter#183070) 2026-03-02 137456488+flutter-pub-roller-bot@users.noreply.github.com Roll pub packages (flutter/flutter#182640) 2026-03-02 15619084+vashworth@users.noreply.github.com Enable SwiftPM by default on master and beta (flutter/flutter#182923) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC bmparr@google.com,stuartmorgan@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: ...

Impl: ```dart /// Sends the specified [message] to the platform plugins on this channel. /// /// Returns a [Future] which completes to the received response, which may /// be null. Future<T?> send(T message) async { return codec.decodeMessage(await binaryMessenger.send(name, codec.encodeMessage(message))); } ``` Part of flutter#181513

…r#11182) Roll Flutter from 46fb7210422d to d3dd7744e81f (33 revisions) flutter/flutter@46fb721...d3dd774 2026-03-04 15619084+vashworth@users.noreply.github.com Show warning when plugins do not support SwiftPM (flutter/flutter#182506) 2026-03-04 15619084+vashworth@users.noreply.github.com Give guided message when project is not compatible with SwiftPM (flutter/flutter#182394) 2026-03-04 danny@tuppeny.com Pass --web-define through to web runner when using --machine mode (flutter/flutter#183228) 2026-03-04 104147021+MohammedTarigg@users.noreply.github.com Improve SwiftPM minimum platform mismatch diagnostics (flutter/flutter#182375) 2026-03-04 vegorov@google.com Use dart::bin::SetupDartIo to setup dart:io (flutter/flutter#176714) 2026-03-04 engine-flutter-autoroll@skia.org Roll Skia from 3197848b14ad to ada0b7628c79 (5 revisions) (flutter/flutter#183221) 2026-03-04 engine-flutter-autoroll@skia.org Roll Skia from fe9e9f22c531 to 3197848b14ad (15 revisions) (flutter/flutter#183198) 2026-03-04 34465683+rkishan516@users.noreply.github.com refactor: remove material in reorderable_list_test, scroll_notification_test, scroll_physics_test, shortcuts_test, sliver_floating_header_test, snapshot_widget_test (flutter/flutter#182698) 2026-03-04 34465683+rkishan516@users.noreply.github.com refactor: remove material in pop_scope_test, route_notification_message_test, two_dimensional_utils, two_dimensional_viewport_test (flutter/flutter#182699) 2026-03-04 737941+loic-sharma@users.noreply.github.com Add dev/benchmarks/README.md (flutter/flutter#182976) 2026-03-03 jason-simmons@users.noreply.github.com Roll RapidJSON to a branch based on the current upstream head (flutter/flutter#183048) 2026-03-03 flar@google.com [Impeller] Update comments to reflect new info about 2-pass rendering (flutter/flutter#183050) 2026-03-03 47866232+chunhtai@users.noreply.github.com Add vmservices for accessibilityEvaluation (flutter/flutter#182791) 2026-03-03 engine-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from 0dCDM2oORHwDf_pyb... to JJw5EJ87vLGqFVl4h... (flutter/flutter#183177) 2026-03-03 lukas.klingsbo@gmail.com Support mixed color spaces in `Color.lerp` (flutter/flutter#182934) 2026-03-03 36861262+QuncCccccc@users.noreply.github.com Add warning when there is a widget with color between `Material` and `ListTile` (flutter/flutter#181402) 2026-03-03 41930132+hellohuanlin@users.noreply.github.com [ios]uitest for admob banner in scrollable list gesture issue (flutter/flutter#183128) 2026-03-03 engine-flutter-autoroll@skia.org Roll Packages from faa4e22 to 9083bc9 (4 revisions) (flutter/flutter#183164) 2026-03-03 15619084+vashworth@users.noreply.github.com Build App and native asset frameworks for Add to App FlutterPluginRegistrant (flutter/flutter#183136) 2026-03-03 engine-flutter-autoroll@skia.org Roll Skia from f886711f180d to fe9e9f22c531 (4 revisions) (flutter/flutter#183155) 2026-03-03 engine-flutter-autoroll@skia.org Roll Dart SDK from e86dbe9aa742 to c597ef90d2dc (2 revisions) (flutter/flutter#183147) 2026-03-03 codefu@google.com fix: bump matcher (flutter/flutter#183167) 2026-03-02 58529443+srujzs@users.noreply.github.com Use isA to test for exceptions (flutter/flutter#183129) 2026-03-02 dpxhfxywang@163.com [two_dimensional_scrollables] Fix tableview janks when first row/column pinned (flutter/flutter#180563) 2026-03-02 victorsanniay@gmail.com Add await to callsites of BasicMessageChannel.send (flutter/flutter#182868) 2026-03-02 137456488+flutter-pub-roller-bot@users.noreply.github.com Roll pub packages (flutter/flutter#183133) 2026-03-02 robert.ancell@canonical.com Improve FFI code for windowing (flutter/flutter#183098) 2026-03-02 kevinjchisholm@google.com [workflow] Update the changelog merge action to fetch the stable branch (flutter/flutter#183132) 2026-03-02 engine-flutter-autoroll@skia.org Roll Skia from e180358b7a7a to f886711f180d (2 revisions) (flutter/flutter#183130) 2026-03-02 jacksongardner@google.com Merge changelog from 3.41.3. (flutter/flutter#183131) 2026-03-02 50985133+SpiralMomentum@users.noreply.github.com Make TextDecoration final and unify maskValue across platforms (flutter/flutter#183070) 2026-03-02 137456488+flutter-pub-roller-bot@users.noreply.github.com Roll pub packages (flutter/flutter#182640) 2026-03-02 15619084+vashworth@users.noreply.github.com Enable SwiftPM by default on master and beta (flutter/flutter#182923) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC bmparr@google.com,stuartmorgan@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: ...

Add await to callsites of BasicMessageChannel.send

7123f7e

victorsanni requested a review from dkwingsmt February 24, 2026 22:51

github-actions Bot added framework flutter/packages/flutter repository. See also f: labels. a: accessibility Accessibility, e.g. VoiceOver or TalkBack. (aka a11y) d: examples Sample code and demos f: routes Navigator, Router, and related APIs. labels Feb 24, 2026

gemini-code-assist Bot reviewed Feb 24, 2026

View reviewed changes

victorsanni requested a review from hannah-hyj February 24, 2026 22:58

dkwingsmt reviewed Feb 24, 2026

View reviewed changes

Comment thread examples/flutter_view/lib/main.dart Outdated

dkwingsmt reviewed Feb 24, 2026

View reviewed changes

victorsanni added 2 commits February 24, 2026 17:43

Apply review suggestions

d0a51b4

Merge branch 'master' into send-awaits

6b28c39

victorsanni requested a review from dkwingsmt February 25, 2026 01:44

dkwingsmt reviewed Feb 25, 2026

View reviewed changes

victorsanni added 3 commits February 25, 2026 10:16

Fix linux analyze

de66e54

Merge branch 'master' into send-awaits

7116123

Add ignores

aa6dc0e

victorsanni commented Feb 25, 2026

View reviewed changes

victorsanni requested a review from dkwingsmt February 25, 2026 19:28

victorsanni added 4 commits February 25, 2026 17:29

Merge branch 'master' into send-awaits

53b534d

Add error reporting to unawaited futures

ae9b87c

Merge branch 'master' into send-awaits

30f5325

Merge branch 'master' into send-awaits

d21bde8

dkwingsmt reviewed Feb 27, 2026

View reviewed changes

victorsanni added 2 commits February 27, 2026 11:20

add await

9e1ab26

Merge branch 'master' into send-awaits

f6dc5e6

dkwingsmt reviewed Feb 27, 2026

View reviewed changes

victorsanni added 2 commits February 27, 2026 13:50

Revert platform_channel_benchmarks/lib/main.dart

1fbf29d

Merge branch 'master' into send-awaits

215ec99

engine-flutter-autoroll mentioned this pull request Mar 3, 2026

Roll Flutter from 46fb7210422d to 6593592c450e (11 revisions) flutter/packages#11170

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 3, 2026

3b618442a Add await to callsites of BasicMessageChannel.send (flutter…

c747458

…/flutter#182868)

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 3, 2026

3b618442a Add await to callsites of BasicMessageChannel.send (flutter…

769ad81

…/flutter#182868)

This was referenced Mar 3, 2026

Roll Flutter from 46fb7210422d to 915a972eb2d6 (12 revisions) flutter/packages#11174

Closed

Roll Flutter from 46fb7210422d to eb0b756f6e05 (29 revisions) flutter/packages#11176

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 4, 2026

3b618442a Add await to callsites of BasicMessageChannel.send (flutter…

4d5571b

…/flutter#182868)

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 4, 2026

3b618442a Add await to callsites of BasicMessageChannel.send (flutter…

30df1a1

…/flutter#182868)

engine-flutter-autoroll mentioned this pull request Mar 4, 2026

Roll Flutter from 46fb7210422d to eb0b756f6e05 (29 revisions) flutter/packages#11177

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 4, 2026

3b618442a Add await to callsites of BasicMessageChannel.send (flutter…

06a1447

…/flutter#182868)

engine-flutter-autoroll mentioned this pull request Mar 4, 2026

Roll Flutter from 46fb7210422d to 6ec6dd191477 (30 revisions) flutter/packages#11178

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 4, 2026

3b618442a Add await to callsites of BasicMessageChannel.send (flutter…

dc9f9ea

…/flutter#182868)

engine-flutter-autoroll mentioned this pull request Mar 4, 2026

Roll Flutter from 46fb7210422d to b5afa98fc52b (31 revisions) flutter/packages#11179

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 4, 2026

3b618442a Add await to callsites of BasicMessageChannel.send (flutter…

c40caf8

…/flutter#182868)

engine-flutter-autoroll mentioned this pull request Mar 4, 2026

Roll Flutter from 46fb7210422d to 9895434fc2a4 (32 revisions) flutter/packages#11181

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 4, 2026

3b618442a Add await to callsites of BasicMessageChannel.send (flutter…

89c7fca

…/flutter#182868)

engine-flutter-autoroll mentioned this pull request Mar 4, 2026

Roll Flutter from 46fb7210422d to d3dd7744e81f (33 revisions) flutter/packages#11182

Merged

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 5, 2026

3b618442a Add await to callsites of BasicMessageChannel.send (flutter…

c50a51d

…/flutter#182868)

engine-flutter-autoroll mentioned this pull request Mar 5, 2026

Roll Flutter from 46fb7210422d to 308649c0cd1b (43 revisions) flutter/packages#11183

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 5, 2026

3b618442a Add await to callsites of BasicMessageChannel.send (flutter…

d4b99d0

…/flutter#182868)

engine-flutter-autoroll mentioned this pull request Mar 5, 2026

Roll Flutter from 46fb7210422d to 308649c0cd1b (43 revisions) flutter/packages#11184

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 5, 2026

3b618442a Add await to callsites of BasicMessageChannel.send (flutter…

b6dee60

…/flutter#182868)

engine-flutter-autoroll mentioned this pull request Mar 5, 2026

Roll Flutter from 46fb7210422d to 3c323fb06507 (44 revisions) flutter/packages#11186

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 5, 2026

3b618442a Add await to callsites of BasicMessageChannel.send (flutter…

b918e39

…/flutter#182868)

engine-flutter-autoroll mentioned this pull request Mar 5, 2026

Roll Flutter from 46fb7210422d to 1488fd4783de (45 revisions) flutter/packages#11187

Closed

victorsanni deleted the send-awaits branch May 5, 2026 18:13

Uh oh!

Conversation

victorsanni commented Feb 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

gemini-code-assist Bot Feb 24, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Feb 24, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

dkwingsmt left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

victorsanni commented Feb 24, 2026 •

edited

Loading