Skip to content

Commit 32ffe84

Browse files
committed
Ensure async consistency
1 parent 8cd072c commit 32ffe84

File tree

4 files changed

+10
-9
lines changed

4 files changed

+10
-9
lines changed

flutter/lib/src/native/java/android_replay_recorder.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ class AndroidReplayRecorder extends ScheduledScreenshotRecorder {
2727
@override
2828
Future<void> start() async {
2929
final spawningWorker = _AndroidNativeReplayWorker.spawn();
30-
super.start();
30+
await super.start();
3131
_worker = await spawningWorker;
3232
}
3333

flutter/lib/src/native/java/sentry_native_java.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ class SentryNativeJava extends SentryNativeChannel {
3737
height: (call.arguments['height'] as num).toDouble(),
3838
frameRate: call.arguments['frameRate'] as int);
3939

40-
_replayRecorder?.onConfigurationChanged(config);
40+
await _replayRecorder?.onConfigurationChanged(config);
4141
break;
4242
case 'ReplayRecorder.stop':
4343
hub.configureScope((s) {

flutter/lib/src/replay/scheduled_recorder.dart

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ class ScheduledScreenshotRecorder extends ReplayScreenshotRecorder {
4242
_callback = callback;
4343
}
4444

45-
void start() async {
45+
Future<void> start() async {
4646
assert(() {
4747
// The following fails if callback hasn't been provided
4848
// in the constructor nor set with a setter.
@@ -52,22 +52,23 @@ class ScheduledScreenshotRecorder extends ReplayScreenshotRecorder {
5252

5353
options.log(SentryLevel.debug, "$logName: starting capture");
5454
_status = _Status.running;
55-
_restartScheduler();
55+
await _restartScheduler();
5656
}
5757

58-
void onConfigurationChanged(ScheduledScreenshotRecorderConfig config) async {
58+
Future<void> onConfigurationChanged(
59+
ScheduledScreenshotRecorderConfig config) async {
5960
super.config = config;
6061
options.log(SentryLevel.debug,
6162
"$logName: onConfigurationChanged (${config.width}x${config.height} @ ${config.frameRate} Hz).");
6263

63-
_restartScheduler();
64+
await _restartScheduler();
6465
}
6566

6667
Future<void> _stopScheduler() {
6768
return _scheduler?.stop() ?? Future.value();
6869
}
6970

70-
void _restartScheduler() async {
71+
Future<void> _restartScheduler() async {
7172
await _stopScheduler();
7273

7374
if (super.config == null) {
@@ -114,7 +115,7 @@ class ScheduledScreenshotRecorder extends ReplayScreenshotRecorder {
114115
Future<void> resume() async {
115116
if (_status == _Status.paused) {
116117
_status = _Status.running;
117-
_restartScheduler();
118+
await _restartScheduler();
118119
// _idleFrameFiller.resume();
119120
}
120121
}

flutter/test/replay/scheduled_recorder_test.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ class _Fixture {
5959
static Future<_Fixture> create(WidgetTester tester) async {
6060
final fixture = _Fixture._(tester);
6161
await pumpTestElement(tester);
62-
fixture.sut.start();
62+
await fixture.sut.start();
6363
return fixture;
6464
}
6565

0 commit comments

Comments
 (0)