From 7d0342493edef18bcbb40fcc4bd35f23489056a7 Mon Sep 17 00:00:00 2001
From: Yorke Lee <yorkelee@google.com>
Date: Thu, 13 Mar 2014 13:54:45 -0700
Subject: Modify InCallUI to hold/unhold calls through Telecomm

Modify holdClicked to direct hold commands through an instance of
InCallAdapter.

Hardcode all Calls to have the capability Call.Capabilities.HOLD.

Bug: 13169202

Change-Id: I7a3c111eda8338047652eb68f6d7234e172b56e6
---
 .../src/com/android/incallui/CallButtonPresenter.java | 19 ++++++++++++++++---
 .../src/com/android/incallui/CallInfoTranslator.java  |  4 ++++
 .../src/com/android/incallui/InCallServiceImpl.java   |  9 +++++++++
 3 files changed, 29 insertions(+), 3 deletions(-)

(limited to 'InCallUI')

diff --git a/InCallUI/src/com/android/incallui/CallButtonPresenter.java b/InCallUI/src/com/android/incallui/CallButtonPresenter.java
index e2bfa1627..02b0237d8 100644
--- a/InCallUI/src/com/android/incallui/CallButtonPresenter.java
+++ b/InCallUI/src/com/android/incallui/CallButtonPresenter.java
@@ -224,9 +224,22 @@ public class CallButtonPresenter extends Presenter<CallButtonPresenter.CallButto
             return;
         }
 
-        Log.d(this, "holding: " + mCall.getCallId());
-
-        CallCommandClient.getInstance().hold(mCall.getCallId(), checked);
+        // Notify Telecomm that the user hit the hold button.
+        InCallAdapter telecommAdapter = InCallPresenter.getInstance().getTelecommAdapter();
+        if (telecommAdapter != null) {
+            String callId = CallInfoTranslator.getTelecommCallId(mCall);
+            if (callId != null) {
+                if (checked) {
+                    Log.i(this, "Putting the call on hold: " + callId);
+                    telecommAdapter.holdCall(callId);
+                } else {
+                    Log.i(this, "Removing the call from hold: " + callId);
+                    telecommAdapter.unholdCall(callId);
+                }
+            } else {
+                Log.wtf(this, "Telecomm callId not found for call: " + mCall.getCallId());
+            }
+        }
     }
 
     public void mergeClicked() {
diff --git a/InCallUI/src/com/android/incallui/CallInfoTranslator.java b/InCallUI/src/com/android/incallui/CallInfoTranslator.java
index fc4bd4940..697c5ea82 100644
--- a/InCallUI/src/com/android/incallui/CallInfoTranslator.java
+++ b/InCallUI/src/com/android/incallui/CallInfoTranslator.java
@@ -72,6 +72,10 @@ final class CallInfoTranslator {
         call.setState(translateCallState(callInfo.getState()));
         call.setNumber(callInfo.getHandle());
 
+        // TODO: Each CallService needs to provide information what kind of call capabilities they
+        // support. For now, always assume that all calls support hold by default.
+        call.addCapabilities(Call.Capabilities.HOLD);
+
         return call;
     }
 
diff --git a/InCallUI/src/com/android/incallui/InCallServiceImpl.java b/InCallUI/src/com/android/incallui/InCallServiceImpl.java
index 09991d7b6..93986ff57 100644
--- a/InCallUI/src/com/android/incallui/InCallServiceImpl.java
+++ b/InCallUI/src/com/android/incallui/InCallServiceImpl.java
@@ -85,4 +85,13 @@ public class InCallServiceImpl extends android.telecomm.InCallService {
             CallInfoTranslator.removeCall(callId);
         }
     }
+
+    /** {@inheritDoc} */
+    @Override protected void setOnHold(String callId) {
+        Call call = CallInfoTranslator.getCall(callId);
+        if (null != call) {
+            call.setState(Call.State.ONHOLD);
+            CallList.getInstance().onUpdate(call);
+        }
+    }
 }
-- 
cgit v1.2.3