summaryrefslogtreecommitdiff
path: root/java/com/android/incallui/videotech/ims/ImsVideoTech.java
diff options
context:
space:
mode:
authorSekine Yasuaki <yasuaki.sekine@sony.com>2017-07-07 17:01:56 +0900
committertakeshi tanigawa <takeshi.tanigawa@sony.com>2019-04-02 15:13:57 +0900
commitda2c0ae3ea67708ff564c362bee6f69f0a7e19a0 (patch)
tree0c23b621e3c83f73434fe8598e6148da92246041 /java/com/android/incallui/videotech/ims/ImsVideoTech.java
parent381dffd9a95b5c8e3d01a91c676dcfb04cbbd204 (diff)
Fix error case which session change request failed when pausing video
When a user clicks the pause video button, the camera close request and the session modify request are executed at the same time. However, the session modify request fails occasionally by a limitation of the Network side. In that case, the video session is continued with the camera closed state and a user cannot open the camera again due to pause video button disabled. Remove camera close request triggered by the pause video button clicked because it is handled appropriately according to the result of the session modify request. Also, enable the pause video button and show error message to a user if receiving the error result of the session modify request. Test: manual Bug: 69235524 Change-Id: I9a2dde755a6c28edfb0ce962b55ac8a6e907ca97
Diffstat (limited to 'java/com/android/incallui/videotech/ims/ImsVideoTech.java')
-rw-r--r--java/com/android/incallui/videotech/ims/ImsVideoTech.java1
1 files changed, 1 insertions, 0 deletions
diff --git a/java/com/android/incallui/videotech/ims/ImsVideoTech.java b/java/com/android/incallui/videotech/ims/ImsVideoTech.java
index 5b733d612..1d4fe769a 100644
--- a/java/com/android/incallui/videotech/ims/ImsVideoTech.java
+++ b/java/com/android/incallui/videotech/ims/ImsVideoTech.java
@@ -227,6 +227,7 @@ public class ImsVideoTech implements VideoTech {
call.getVideoCall()
.sendSessionModifyRequest(
new VideoProfile(unpausedVideoState & ~VideoProfile.STATE_TX_ENABLED));
+ setSessionModificationState(SessionModificationState.WAITING_FOR_RESPONSE);
}
@Override