summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorroldenburg <roldenburg@google.com>2017-10-15 03:11:23 +0000
committerandroid-build-merger <android-build-merger@google.com>2017-10-15 03:11:23 +0000
commit886879d36aa418fb4a01f68922ebbfea3125e386 (patch)
tree72ed5afc03ba49aa7a25b7419e5f33708151d370
parent039fa13e82456a4e756364e6a59126f3b1420edc (diff)
parent65aa3b3677537974f2136c702828d70a60fe6361 (diff)
Merge changes Iaf6e217d,Ib22e33de,If9fd6ca9
am: 65aa3b3677 Change-Id: I8fbd63b32623a3cd7cc35051aaa110c88a8b2313
-rw-r--r--java/com/android/dialer/app/calllog/VisualVoicemailUpdateTask.java3
-rw-r--r--java/com/android/dialer/app/list/ListsFragment.java6
-rw-r--r--java/com/android/incallui/videotech/lightbringer/LightbringerTech.java6
3 files changed, 7 insertions, 8 deletions
diff --git a/java/com/android/dialer/app/calllog/VisualVoicemailUpdateTask.java b/java/com/android/dialer/app/calllog/VisualVoicemailUpdateTask.java
index 25444834d..e30623bb2 100644
--- a/java/com/android/dialer/app/calllog/VisualVoicemailUpdateTask.java
+++ b/java/com/android/dialer/app/calllog/VisualVoicemailUpdateTask.java
@@ -121,7 +121,8 @@ class VisualVoicemailUpdateTask implements Worker<VisualVoicemailUpdateTask.Inpu
if (notification.getId() != VisualVoicemailNotifier.NOTIFICATION_ID) {
continue;
}
- if (!notification.getTag().startsWith(VisualVoicemailNotifier.NOTIFICATION_TAG_PREFIX)) {
+ if (TextUtils.isEmpty(notification.getTag())
+ || !notification.getTag().startsWith(VisualVoicemailNotifier.NOTIFICATION_TAG_PREFIX)) {
continue;
}
String uri =
diff --git a/java/com/android/dialer/app/list/ListsFragment.java b/java/com/android/dialer/app/list/ListsFragment.java
index 362997a5a..05c647aec 100644
--- a/java/com/android/dialer/app/list/ListsFragment.java
+++ b/java/com/android/dialer/app/list/ListsFragment.java
@@ -203,7 +203,11 @@ public class ListsFragment extends Fragment implements OnPageChangeListener, Lis
mPrefs.getBoolean(
VisualVoicemailEnabledChecker.PREF_KEY_HAS_ACTIVE_VOICEMAIL_PROVIDER, false));
mViewPager.setAdapter(mAdapter);
- mViewPager.setOffscreenPageLimit(DialtactsPagerAdapter.TAB_COUNT_WITH_VOICEMAIL - 1);
+
+ // This is deliberate. See cl/172018946 for the app startup implications of using 1 here
+ // versus loading more fragments upfront.
+ mViewPager.setOffscreenPageLimit(1);
+
mViewPager.addOnPageChangeListener(this);
showTab(DialtactsPagerAdapter.TAB_INDEX_SPEED_DIAL);
diff --git a/java/com/android/incallui/videotech/lightbringer/LightbringerTech.java b/java/com/android/incallui/videotech/lightbringer/LightbringerTech.java
index 784daa85a..eb2e92e85 100644
--- a/java/com/android/incallui/videotech/lightbringer/LightbringerTech.java
+++ b/java/com/android/incallui/videotech/lightbringer/LightbringerTech.java
@@ -17,7 +17,6 @@
package com.android.incallui.videotech.lightbringer;
import android.content.Context;
-import android.os.Build;
import android.support.annotation.NonNull;
import android.support.annotation.Nullable;
import android.telecom.Call;
@@ -57,11 +56,6 @@ public class LightbringerTech implements VideoTech, LightbringerListener {
@Override
public boolean isAvailable(Context context) {
- if (Build.VERSION.SDK_INT < Build.VERSION_CODES.N) {
- LogUtil.v("LightbringerTech.isAvailable", "upgrade unavailable, only supported on N+");
- return false;
- }
-
if (!ConfigProviderBindings.get(context)
.getBoolean("enable_lightbringer_video_upgrade", true)) {
LogUtil.v("LightbringerTech.isAvailable", "upgrade disabled by flag");