From e249c00a3ca1993bf429ca93c82a4d70ec210400 Mon Sep 17 00:00:00 2001
From: Doris Ling <dling@google.com>
Date: Thu, 4 May 2017 14:33:52 -0700
Subject: [PATCH] Limit the summary text for App info->Permissions to one line.

- the summary text is the list of permissions that the app has. It is
dynamic, and when the list is long, setting the summary will expand
the text view height and make the page launch looks janky. We cannot
pre-set the summary to certain line, since the permissions list for each
app is unknown beforehand. To avoid the jank, limit the summary to 1
line and truncate it if necessary.

Change-Id: I1acb1ac9f46c55808ce03626e53cf71502231ab2
Fix: 36413664
Test: make RunSettingsRoboTests
---
 res/xml/installed_app_details_ia.xml          |  2 +-
 .../widget/SingleLineSummaryPreference.java   | 42 +++++++++++
 .../SingleLineSummaryPreferenceTest.java      | 69 +++++++++++++++++++
 3 files changed, 112 insertions(+), 1 deletion(-)
 create mode 100644 src/com/android/settings/widget/SingleLineSummaryPreference.java
 create mode 100644 tests/robotests/src/com/android/settings/widget/SingleLineSummaryPreferenceTest.java

diff --git a/res/xml/installed_app_details_ia.xml b/res/xml/installed_app_details_ia.xml
index 538205f8343..5f8e8660587 100644
--- a/res/xml/installed_app_details_ia.xml
+++ b/res/xml/installed_app_details_ia.xml
@@ -40,7 +40,7 @@
         android:title="@string/notifications_label"
         android:selectable="true"/>
 
-    <Preference
+    <com.android.settings.widget.SingleLineSummaryPreference
         android:key="permission_settings"
         android:title="@string/permissions_label"
         android:summary="@string/summary_placeholder"
diff --git a/src/com/android/settings/widget/SingleLineSummaryPreference.java b/src/com/android/settings/widget/SingleLineSummaryPreference.java
new file mode 100644
index 00000000000..36d9e80818b
--- /dev/null
+++ b/src/com/android/settings/widget/SingleLineSummaryPreference.java
@@ -0,0 +1,42 @@
+/*
+ * Copyright (C) 2017 The Android Open Source Project
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file
+ * except in compliance with the License. You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software distributed under the
+ * License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the specific language governing
+ * permissions and limitations under the License.
+ */
+
+package com.android.settings.widget;
+
+import android.content.Context;
+import android.support.v7.preference.Preference;
+import android.support.v7.preference.PreferenceViewHolder;
+import android.text.TextUtils.TruncateAt;
+import android.util.AttributeSet;
+import android.widget.TextView;
+
+/**
+ * A preference whose summary text will only span one single line.
+ */
+public class SingleLineSummaryPreference extends Preference {
+
+    public SingleLineSummaryPreference(Context context, AttributeSet attrs) {
+        super(context, attrs);
+    }
+
+    @Override
+    public void onBindViewHolder(PreferenceViewHolder holder) {
+        super.onBindViewHolder(holder);
+        TextView summary = (TextView) holder.findViewById(android.R.id.summary);
+        if (summary != null) {
+            summary.setSingleLine();
+            summary.setEllipsize(TruncateAt.END);
+        }
+    }
+}
diff --git a/tests/robotests/src/com/android/settings/widget/SingleLineSummaryPreferenceTest.java b/tests/robotests/src/com/android/settings/widget/SingleLineSummaryPreferenceTest.java
new file mode 100644
index 00000000000..d4c53923d31
--- /dev/null
+++ b/tests/robotests/src/com/android/settings/widget/SingleLineSummaryPreferenceTest.java
@@ -0,0 +1,69 @@
+/*
+ * Copyright (C) 2017 The Android Open Source Project
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License
+ */
+package com.android.settings.widget;
+
+import static org.mockito.Mockito.spy;
+import static org.mockito.Mockito.verify;
+import static org.mockito.Mockito.when;
+
+import android.content.Context;
+import android.support.v7.preference.PreferenceViewHolder;
+import android.view.LayoutInflater;
+import android.view.View;
+import android.widget.LinearLayout;
+import android.widget.TextView;
+import com.android.settings.SettingsRobolectricTestRunner;
+import com.android.settings.TestConfig;
+import com.android.settings.widget.SingleLineSummaryPreference;
+import org.junit.Before;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.mockito.Mock;
+import org.mockito.MockitoAnnotations;
+import org.robolectric.RuntimeEnvironment;
+import org.robolectric.annotation.Config;
+
+@RunWith(SettingsRobolectricTestRunner.class)
+@Config(manifest = TestConfig.MANIFEST_PATH, sdk = TestConfig.SDK_VERSION)
+public class SingleLineSummaryPreferenceTest {
+
+    @Mock private TextView mSummary;
+
+    private Context mContext;
+    private PreferenceViewHolder mHolder;
+    private SingleLineSummaryPreference mPreference;
+
+    @Before
+    public void setUp() {
+        MockitoAnnotations.initMocks(this);
+
+        mContext = RuntimeEnvironment.application;
+        mPreference = new SingleLineSummaryPreference(mContext, null);
+        LayoutInflater inflater = LayoutInflater.from(mContext);
+        final View view = inflater.inflate(mPreference.getLayoutResource(),
+            new LinearLayout(mContext), false);
+        mHolder = spy(PreferenceViewHolder.createInstanceForTests(view));
+        when(mHolder.findViewById(android.R.id.summary)).thenReturn(mSummary);
+    }
+
+    @Test
+    public void onBindViewHolder_shouldSetSingleLine() {
+        mPreference.onBindViewHolder(mHolder);
+        final TextView summary = (TextView) mHolder.findViewById(android.R.id.summary);
+
+        verify(mSummary).setSingleLine();
+    }
+}