From 7aa48391a30920007ecb42d6a1d47075fc2a9b63 Mon Sep 17 00:00:00 2001
From: Andrew Lee <anwlee@google.com>
Date: Mon, 28 Apr 2014 16:29:38 -0700
Subject: Always show the call state text view.

Before this was flashing in and out because of the change in
visibility, which was causing some shifting and UI jank. Now
the call state label will always take up space in the layout.

Also, moved where the elapsed time view is so that it is
properly aligned to mesh with new changes.

Change-Id: Id8b395b18ebc1038310721d878622c6b0c15c295
---
 InCallUI/res/layout/primary_call_info.xml          | 42 ++++++++++++----------
 .../src/com/android/incallui/CallCardFragment.java | 11 ++----
 2 files changed, 27 insertions(+), 26 deletions(-)

diff --git a/InCallUI/res/layout/primary_call_info.xml b/InCallUI/res/layout/primary_call_info.xml
index 18cc70b7c..7c0a1811e 100644
--- a/InCallUI/res/layout/primary_call_info.xml
+++ b/InCallUI/res/layout/primary_call_info.xml
@@ -55,14 +55,32 @@
                     android:gravity="left"
                     android:ellipsize="end" />
 
-                <TextView android:id="@+id/name"
+
+                <LinearLayout android:id="@+id/stateAndName"
                     android:layout_width="match_parent"
-                    android:paddingEnd="@dimen/call_banner_name_number_right_padding"
                     android:layout_height="wrap_content"
-                    android:fontFamily="sans-serif-light"
-                    android:textAppearance="?android:attr/textAppearanceLarge"
-                    android:textSize="@dimen/call_name_text_size"
-                    android:singleLine="true" />
+                    android:orientation="horizontal" >
+
+                    <TextView android:id="@+id/name"
+                        android:layout_width="wrap_content"
+                        android:layout_height="wrap_content"
+                        android:paddingEnd="@dimen/call_banner_name_number_right_padding"
+                        android:fontFamily="sans-serif-light"
+                        android:textAppearance="?android:attr/textAppearanceLarge"
+                        android:textSize="@dimen/call_name_text_size"
+                        android:singleLine="true" />
+
+                    <!-- Elapsed time indication for a call in progress. -->
+                    <TextView android:id="@+id/elapsedTime"
+                        android:gravity="right"
+                        android:layout_width="wrap_content"
+                        android:layout_height="wrap_content"
+                        android:textAppearance="?android:attr/textAppearanceMedium"
+                        android:textColor="@color/incall_call_banner_text_color"
+                        android:singleLine="true"
+                        android:visibility="invisible" />
+
+                </LinearLayout>
 
             </LinearLayout>
 
@@ -93,18 +111,6 @@
                     android:layout_marginStart="6dp" />
 
             </LinearLayout>
-
-            <!-- Elapsed time indication for a call in progress. -->
-            <TextView android:id="@+id/elapsedTime"
-                android:layout_alignParentEnd="true"
-                android:layout_centerVertical="true"
-                android:layout_width="wrap_content"
-                android:layout_height="wrap_content"
-                android:textAppearance="?android:attr/textAppearanceMedium"
-                android:textColor="@color/incall_call_banner_text_color"
-                android:singleLine="true"
-                android:visibility="invisible" />
-
             <!-- Call type indication: a special label and/or branding
                  for certain kinds of calls (like "Internet call" for a SIP call.) -->
             <TextView android:id="@+id/callTypeLabel"
diff --git a/InCallUI/src/com/android/incallui/CallCardFragment.java b/InCallUI/src/com/android/incallui/CallCardFragment.java
index 9f5b0527f..e08c6207f 100644
--- a/InCallUI/src/com/android/incallui/CallCardFragment.java
+++ b/InCallUI/src/com/android/incallui/CallCardFragment.java
@@ -250,15 +250,10 @@ public class CallCardFragment extends BaseFragment<CallCardPresenter, CallCardPr
         Log.v(this, "gateway " + gatewayLabel + gatewayNumber);
 
         // Update the call state label.
-        if (!TextUtils.isEmpty(callStateLabel)) {
-            mCallStateLabel.setVisibility(View.VISIBLE);
-            mCallStateLabel.setText(callStateLabel);
+        mCallStateLabel.setText(callStateLabel);
 
-            if (Call.State.INCOMING == state) {
-                setBluetoothOn(bluetoothOn);
-            }
-        } else {
-            mCallStateLabel.setVisibility(View.GONE);
+        if (Call.State.INCOMING == state) {
+            setBluetoothOn(bluetoothOn);
         }
     }
 
-- 
cgit v1.2.3