From f29d22e97bbbd6e3201c0347e48503e40309253e Mon Sep 17 00:00:00 2001
From: zachh <zachh@google.com>
Date: Fri, 29 Sep 2017 11:11:01 -0700
Subject: Warmup shared prefs in googledialer when strict mode is enabled.

This avoid some spurious strict mode violations.

Test: observed no strict mode notification launching google dialer
PiperOrigin-RevId: 170505918
Change-Id: Ie48671dcd306ea18b041a0bd7989b419bab903f5
---
 .../contacts/common/list/PhoneNumberListAdapter.java    | 17 +++++++++++++----
 1 file changed, 13 insertions(+), 4 deletions(-)

(limited to 'java/com/android/contacts/common')

diff --git a/java/com/android/contacts/common/list/PhoneNumberListAdapter.java b/java/com/android/contacts/common/list/PhoneNumberListAdapter.java
index 741e606ac..8cf54b96c 100644
--- a/java/com/android/contacts/common/list/PhoneNumberListAdapter.java
+++ b/java/com/android/contacts/common/list/PhoneNumberListAdapter.java
@@ -42,6 +42,7 @@ import com.android.contacts.common.preference.ContactsPreferences;
 import com.android.contacts.common.util.Constants;
 import com.android.dialer.common.LogUtil;
 import com.android.dialer.compat.CompatUtils;
+import com.android.dialer.configprovider.ConfigProviderBindings;
 import com.android.dialer.contactphoto.ContactPhotoManager.DefaultImageRequest;
 import com.android.dialer.dialercontact.DialerContact;
 import com.android.dialer.enrichedcall.EnrichedCallCapabilities;
@@ -500,14 +501,22 @@ public class PhoneNumberListAdapter extends ContactEntryListAdapter {
     if (getDirectorySearchMode() == DirectoryListLoader.SEARCH_MODE_NONE) {
       return;
     }
-    final int numExtendedDirectories = mExtendedDirectories.size();
+    int numExtendedDirectories = mExtendedDirectories.size();
+
+    if (ConfigProviderBindings.get(getContext()).getBoolean("p13n_ranker_should_enable", false)) {
+      // Suggested results wasn't formulated as an extended directory, so manually
+      // increment the count here when the feature is enabled. Suggestions are
+      // only shown when the ranker is enabled.
+      numExtendedDirectories++;
+    }
+
     if (getPartitionCount() == cursor.getCount() + numExtendedDirectories) {
       // already added all directories;
       return;
     }
-    //
+
     mFirstExtendedDirectoryId = Long.MAX_VALUE;
-    if (numExtendedDirectories > 0) {
+    if (!mExtendedDirectories.isEmpty()) {
       // The Directory.LOCAL_INVISIBLE is not in the cursor but we can't reuse it's
       // "special" ID.
       long maxId = Directory.LOCAL_INVISIBLE;
@@ -527,7 +536,7 @@ public class PhoneNumberListAdapter extends ContactEntryListAdapter {
       }
       // Extended directories ID's cannot collide with base directories
       mFirstExtendedDirectoryId = maxId + 1;
-      for (int i = 0; i < numExtendedDirectories; i++) {
+      for (int i = 0; i < mExtendedDirectories.size(); i++) {
         final long id = mFirstExtendedDirectoryId + i;
         final DirectoryPartition directory = mExtendedDirectories.get(i);
         if (getPartitionByDirectoryId(id) == -1) {
-- 
cgit v1.2.3