| commit | 2cec38000c225d15a7b895db28541f4be2f2df7b | [log] [tgz] |
|---|---|---|
| author | roldenburg <roldenburg@google.com> | Wed Oct 11 16:26:51 2017 -0700 |
| committer | Eric Erfanian <erfanian@google.com> | Thu Oct 12 14:29:53 2017 -0700 |
| tree | c32641930e09271ac587a767288818baf393b08c | |
| parent | d0f207f160a4abac488de29df82a842b6e3a353d [diff] [blame] |
Fix for voice screen showing for a short time when making a video call Author: tomoyuki.nishikawa@sony.com This change was inspired by http://aosp/508735 Even though VideoProfile.isVideo(getVideoState() returns true when starting a video call, ImsVideoTech is not ready until call.getVideoCall() returns not null (see https://android.googlesource.com/platform/packages/apps/Dialer/+/master/java/com/android/incallui/videotech/ims/ImsVideoTech.java#66) During the time between VideoProfile.isVideo(getVideoState() being true but call.getVideoCall() returning null, we are incorrectly showing the phone call UI instead of the video call UI. To fix this, we need to trust the VideoProfile check in addition to the VideoTech check. We still need to check VideoTech for other video options aside from just IMS VT. Bug: 67712159 Test: manual PiperOrigin-RevId: 171892356 Change-Id: Ibe1f9de60aaa4adc971401bee0277961cfe037e4
diff --git a/java/com/android/incallui/call/DialerCall.java b/java/com/android/incallui/call/DialerCall.java index 621ad21..6829203 100644 --- a/java/com/android/incallui/call/DialerCall.java +++ b/java/com/android/incallui/call/DialerCall.java
@@ -857,7 +857,7 @@ } public boolean isVideoCall() { - return getVideoTech().isTransmittingOrReceiving(); + return getVideoTech().isTransmittingOrReceiving() || VideoProfile.isVideo(getVideoState()); } public boolean hasReceivedVideoUpgradeRequest() {