Use EmptyVideoTech instead of null RcsVideoShare VideoTech impl when unavailable
Bug: 67005309,71880949
Test: VideoTechManagerTest
PiperOrigin-RevId: 182091664
Change-Id: I0be4d4f9740e2bb678e772ec8526ccf176c334eb
diff --git a/java/com/android/dialer/enrichedcall/RcsVideoShareFactory.java b/java/com/android/dialer/enrichedcall/RcsVideoShareFactory.java
index 38f91da..faea3bc 100644
--- a/java/com/android/dialer/enrichedcall/RcsVideoShareFactory.java
+++ b/java/com/android/dialer/enrichedcall/RcsVideoShareFactory.java
@@ -23,6 +23,7 @@
/** Interface for creating new RCS video tech instances. */
public interface RcsVideoShareFactory {
+ @NonNull
VideoTech newRcsVideoShare(
@NonNull EnrichedCallManager enrichedCallManager,
@NonNull VideoTechListener videoTechListener,
diff --git a/java/com/android/dialer/enrichedcall/stub/StubEnrichedCallModule.java b/java/com/android/dialer/enrichedcall/stub/StubEnrichedCallModule.java
index 6ee1a86..93e1579 100644
--- a/java/com/android/dialer/enrichedcall/stub/StubEnrichedCallModule.java
+++ b/java/com/android/dialer/enrichedcall/stub/StubEnrichedCallModule.java
@@ -18,6 +18,7 @@
import com.android.dialer.enrichedcall.EnrichedCallManager;
import com.android.dialer.enrichedcall.RcsVideoShareFactory;
+import com.android.incallui.videotech.empty.EmptyVideoTech;
import dagger.Module;
import dagger.Provides;
import javax.inject.Singleton;
@@ -35,7 +36,7 @@
@Provides
@Singleton
static RcsVideoShareFactory providesRcsVideoShareFactory() {
- return (enrichedCallManager, videoTechListener, number) -> null;
+ return (enrichedCallManager, videoTechListener, number) -> new EmptyVideoTech();
}
private StubEnrichedCallModule() {}
diff --git a/java/com/android/incallui/call/DialerCall.java b/java/com/android/incallui/call/DialerCall.java
index dd7535e..d57271a 100644
--- a/java/com/android/incallui/call/DialerCall.java
+++ b/java/com/android/incallui/call/DialerCall.java
@@ -1664,9 +1664,7 @@
EnrichedCallComponent.get(call.context).getEnrichedCallManager(),
call,
phoneNumber);
- if (rcsVideoShare != null) {
- videoTechs.add(rcsVideoShare);
- }
+ videoTechs.add(rcsVideoShare);
videoTechs.add(
new DuoVideoTech(