diff options
| author | roldenburg <roldenburg@google.com> | 2017-10-11 16:26:51 -0700 |
|---|---|---|
| committer | Eric Erfanian <erfanian@google.com> | 2017-10-12 14:29:53 -0700 |
| commit | 2cec38000c225d15a7b895db28541f4be2f2df7b (patch) | |
| tree | c32641930e09271ac587a767288818baf393b08c /java | |
| parent | d0f207f160a4abac488de29df82a842b6e3a353d (diff) | |
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
Diffstat (limited to 'java')
| -rw-r--r-- | java/com/android/incallui/call/DialerCall.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/java/com/android/incallui/call/DialerCall.java b/java/com/android/incallui/call/DialerCall.java index 621ad21f8..6829203ff 100644 --- a/java/com/android/incallui/call/DialerCall.java +++ b/java/com/android/incallui/call/DialerCall.java @@ -857,7 +857,7 @@ public class DialerCall implements VideoTechListener, StateChangedListener, Capa } public boolean isVideoCall() { - return getVideoTech().isTransmittingOrReceiving(); + return getVideoTech().isTransmittingOrReceiving() || VideoProfile.isVideo(getVideoState()); } public boolean hasReceivedVideoUpgradeRequest() { |