Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.

Commit b4158f9

Browse files
gaaclarkegaaclarke
committed
A partial revert of the work done to accomodate Android Background Channels.
1 parent 97c2483 commit b4158f9

File tree

4 files changed

+21
-79
lines changed

4 files changed

+21
-79
lines changed

‎packages/android_intent/android/src/test/java/io/flutter/plugins/androidintent/MethodCallHandlerImplTest.java

Lines changed: 5 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -55,11 +55,7 @@ public void startListening_registersChannel() {
5555
methodCallHandler.startListening(messenger);
5656

5757
verify(messenger, times(1))
58-
// FLUTTER_STABLE_CONDITIONAL_IF_NOT_STABLE
59-
.setMessageHandler(eq(CHANNEL_NAME), any(BinaryMessageHandler.class), eq(null));
60-
// FLUTTER_STABLE_CONDITIONAL_ELSE
61-
// .setMessageHandler(eq(CHANNEL_NAME), any(BinaryMessageHandler.class));
62-
// FLUTTER_STABLE_CONDITIONAL_ENDIF
58+
.setMessageHandler(eq(CHANNEL_NAME), any(BinaryMessageHandler.class));
6359
}
6460

6561
@Test
@@ -71,15 +67,9 @@ public void startListening_unregistersExistingChannel() {
7167
methodCallHandler.startListening(secondMessenger);
7268

7369
// Unregisters the first and then registers the second.
74-
// FLUTTER_STABLE_CONDITIONAL_IF_NOT_STABLE
75-
verify(firstMessenger, times(1)).setMessageHandler(CHANNEL_NAME, null, null);
70+
verify(firstMessenger, times(1)).setMessageHandler(CHANNEL_NAME, null);
7671
verify(secondMessenger, times(1))
77-
.setMessageHandler(eq(CHANNEL_NAME), any(BinaryMessageHandler.class), eq(null));
78-
// FLUTTER_STABLE_CONDITIONAL_ELSE
79-
// verify(firstMessenger, times(1)).setMessageHandler(CHANNEL_NAME, null);
80-
// verify(secondMessenger, times(1))
81-
// .setMessageHandler(eq(CHANNEL_NAME), any(BinaryMessageHandler.class));
82-
// FLUTTER_STABLE_CONDITIONAL_ENDIF
72+
.setMessageHandler(eq(CHANNEL_NAME), any(BinaryMessageHandler.class));
8373
}
8474

8575
@Test
@@ -89,11 +79,7 @@ public void stopListening_unregistersExistingChannel() {
8979

9080
methodCallHandler.stopListening();
9181

92-
// FLUTTER_STABLE_CONDITIONAL_IF_NOT_STABLE
93-
verify(messenger, times(1)).setMessageHandler(CHANNEL_NAME, null, null);
94-
// FLUTTER_STABLE_CONDITIONAL_ELSE
95-
// verify(messenger, times(1)).setMessageHandler(CHANNEL_NAME, null);
96-
// FLUTTER_STABLE_CONDITIONAL_ENDIF
82+
verify(messenger, times(1)).setMessageHandler(CHANNEL_NAME, null);
9783
}
9884

9985
@Test
@@ -102,11 +88,7 @@ public void stopListening_doesNothingWhenUnset() {
10288

10389
methodCallHandler.stopListening();
10490

105-
// FLUTTER_STABLE_CONDITIONAL_IF_NOT_STABLE
106-
verify(messenger, never()).setMessageHandler(CHANNEL_NAME, null, null);
107-
// FLUTTER_STABLE_CONDITIONAL_ELSE
108-
// verify(messenger, never()).setMessageHandler(CHANNEL_NAME, null);
109-
// FLUTTER_STABLE_CONDITIONAL_ENDIF
91+
verify(messenger, never()).setMessageHandler(CHANNEL_NAME, null);
11092
}
11193

11294
@Test

‎packages/camera/camera/android/src/test/java/io/flutter/plugins/camera/DartMessengerTest.java

Lines changed: 4 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@
1212

1313
import android.os.Handler;
1414
import androidx.annotation.NonNull;
15-
import androidx.annotation.Nullable;
1615
import io.flutter.embedding.engine.systemchannels.PlatformChannel;
1716
import io.flutter.plugin.common.BinaryMessenger;
1817
import io.flutter.plugin.common.MethodCall;
@@ -32,15 +31,6 @@ public class DartMessengerTest {
3231
private static class FakeBinaryMessenger implements BinaryMessenger {
3332
private final List<ByteBuffer> sentMessages = new ArrayList<>();
3433

35-
// TODO(aaclarke): Remove when https://github.com/flutter/engine/pull/29147 is on stable.
36-
// FLUTTER_STABLE_CONDITIONAL_IF_NOT_STABLE
37-
@Override
38-
public BinaryMessenger.TaskQueue makeBackgroundTaskQueue() {
39-
return null;
40-
}
41-
// FLUTTER_STABLE_CONDITIONAL_ELSE
42-
// FLUTTER_STABLE_CONDITIONAL_ENDIF
43-
4434
@Override
4535
public void send(@NonNull String channel, ByteBuffer message) {
4636
sentMessages.add(message);
@@ -51,16 +41,14 @@ public void send(@NonNull String channel, ByteBuffer message, BinaryReply callba
5141
send(channel, message);
5242
}
5343

44+
@Override
45+
public void setMessageHandler(@NonNull String channel, BinaryMessageHandler handler) {}
46+
5447
// TODO(aaclarke): Remove when https://github.com/flutter/engine/pull/29147 is on stable.
5548
// FLUTTER_STABLE_CONDITIONAL_IF_NOT_STABLE
5649
@Override
57-
public void setMessageHandler(
58-
@NonNull String channel,
59-
BinaryMessageHandler handler,
60-
@Nullable BinaryMessenger.TaskQueue taskQueue) {}
50+
public void setTaskQueue(@NonNull String channel, TaskQueue handler) {}
6151
// FLUTTER_STABLE_CONDITIONAL_ELSE
62-
// @Override
63-
// public void setMessageHandler(@NonNull String channel, BinaryMessageHandler handler) {}
6452
// FLUTTER_STABLE_CONDITIONAL_ENDIF
6553

6654
List<ByteBuffer> getMessages() {

‎packages/quick_actions/quick_actions/android/src/test/java/io/flutter/plugins/quickactions/QuickActionsTest.java

Lines changed: 7 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -33,15 +33,6 @@ public class QuickActionsTest {
3333
private static class TestBinaryMessenger implements BinaryMessenger {
3434
public MethodCall lastMethodCall;
3535

36-
// TODO(aaclarke): Remove when https://github.com/flutter/engine/pull/29147 is on stable.
37-
// FLUTTER_STABLE_CONDITIONAL_IF_NOT_STABLE
38-
@Override
39-
public BinaryMessenger.TaskQueue makeBackgroundTaskQueue() {
40-
return null;
41-
}
42-
// FLUTTER_STABLE_CONDITIONAL_ELSE
43-
// FLUTTER_STABLE_CONDITIONAL_ENDIF
44-
4536
@Override
4637
public void send(@NonNull String channel, @Nullable ByteBuffer message) {
4738
send(channel, message, null);
@@ -58,20 +49,19 @@ public void send(
5849
}
5950
}
6051

52+
@Override
53+
public void setMessageHandler(@NonNull String channel, @Nullable BinaryMessageHandler handler) {
54+
// Do nothing.
55+
}
56+
6157
// TODO(aaclarke): Remove when https://github.com/flutter/engine/pull/29147 is on stable.
6258
// FLUTTER_STABLE_CONDITIONAL_IF_NOT_STABLE
6359
@Override
64-
public void setMessageHandler(
65-
@NonNull String channel,
66-
@Nullable BinaryMessageHandler handler,
67-
@Nullable BinaryMessenger.TaskQueue taskQueue) {
60+
public void setTaskQueue(
61+
@NonNull String channel, @Nullable BinaryMessenger.TaskQueue taskQueue) {
6862
// Do nothing.
6963
}
7064
// FLUTTER_STABLE_CONDITIONAL_ELSE
71-
// @Override
72-
// public void setMessageHandler(
73-
// @NonNull String channel,
74-
// @Nullable BinaryMessageHandler handler) {}
7565
// FLUTTER_STABLE_CONDITIONAL_ENDIF
7666
}
7767

‎packages/url_launcher/url_launcher/android/src/test/java/io/flutter/plugins/urllauncher/MethodCallHandlerImplTest.java

Lines changed: 5 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -44,11 +44,7 @@ public void startListening_registersChannel() {
4444
methodCallHandler.startListening(messenger);
4545

4646
verify(messenger, times(1))
47-
// FLUTTER_STABLE_CONDITIONAL_IF_NOT_STABLE
48-
.setMessageHandler(eq(CHANNEL_NAME), any(BinaryMessageHandler.class), eq(null));
49-
// FLUTTER_STABLE_CONDITIONAL_ELSE
50-
// .setMessageHandler(eq(CHANNEL_NAME), any(BinaryMessageHandler.class));
51-
// FLUTTER_STABLE_CONDITIONAL_ENDIF
47+
.setMessageHandler(eq(CHANNEL_NAME), any(BinaryMessageHandler.class));
5248
}
5349

5450
@Test
@@ -60,15 +56,9 @@ public void startListening_unregistersExistingChannel() {
6056
methodCallHandler.startListening(secondMessenger);
6157

6258
// Unregisters the first and then registers the second.
63-
// FLUTTER_STABLE_CONDITIONAL_IF_NOT_STABLE
64-
verify(firstMessenger, times(1)).setMessageHandler(CHANNEL_NAME, null, null);
59+
verify(firstMessenger, times(1)).setMessageHandler(CHANNEL_NAME, null);
6560
verify(secondMessenger, times(1))
66-
.setMessageHandler(eq(CHANNEL_NAME), any(BinaryMessageHandler.class), eq(null));
67-
// FLUTTER_STABLE_CONDITIONAL_ELSE
68-
// verify(firstMessenger, times(1)).setMessageHandler(CHANNEL_NAME, null);
69-
// verify(secondMessenger, times(1))
70-
// .setMessageHandler(eq(CHANNEL_NAME), any(BinaryMessageHandler.class));
71-
// FLUTTER_STABLE_CONDITIONAL_ENDIF
61+
.setMessageHandler(eq(CHANNEL_NAME), any(BinaryMessageHandler.class));
7262
}
7363

7464
@Test
@@ -78,11 +68,7 @@ public void stopListening_unregistersExistingChannel() {
7868

7969
methodCallHandler.stopListening();
8070

81-
// FLUTTER_STABLE_CONDITIONAL_IF_NOT_STABLE
82-
verify(messenger, times(1)).setMessageHandler(CHANNEL_NAME, null, null);
83-
// FLUTTER_STABLE_CONDITIONAL_ELSE
84-
// verify(messenger, times(1)).setMessageHandler(CHANNEL_NAME, null);
85-
// FLUTTER_STABLE_CONDITIONAL_ENDIF
71+
verify(messenger, times(1)).setMessageHandler(CHANNEL_NAME, null);
8672
}
8773

8874
@Test
@@ -91,11 +77,7 @@ public void stopListening_doesNothingWhenUnset() {
9177

9278
methodCallHandler.stopListening();
9379

94-
// FLUTTER_STABLE_CONDITIONAL_IF_NOT_STABLE
95-
verify(messenger, never()).setMessageHandler(CHANNEL_NAME, null, null);
96-
// FLUTTER_STABLE_CONDITIONAL_ELSE
97-
// verify(messenger, never()).setMessageHandler(CHANNEL_NAME, null);
98-
// FLUTTER_STABLE_CONDITIONAL_ENDIF
80+
verify(messenger, never()).setMessageHandler(CHANNEL_NAME, null);
9981
}
10082

10183
@Test

0 commit comments

Comments
 (0)