From ac4037a47512e6a8508a06225ee47a7939c84605 Mon Sep 17 00:00:00 2001
From: Nate Jiang <qiangjiang@google.com>
Date: Wed, 15 Apr 2020 17:03:01 -0700
Subject: Stop persist numAssociation

As we start tp persist most recently connected network for scan
optimizition, stop persist numAssociation

Bug: 148535554
Test: atest com.android.server.wifi
Change-Id: I8a8ec3db42a0a9887e2662c18f930e6be43b6506
---
 service/java/com/android/server/wifi/util/XmlUtil.java               | 5 -----
 .../src/com/android/server/wifi/NetworkListStoreDataTest.java        | 5 -----
 tests/wifitests/src/com/android/server/wifi/WifiConfigStoreTest.java | 1 -
 .../src/com/android/server/wifi/WifiConfigurationTestUtil.java       | 2 +-
 4 files changed, 1 insertion(+), 12 deletions(-)

diff --git a/service/java/com/android/server/wifi/util/XmlUtil.java b/service/java/com/android/server/wifi/util/XmlUtil.java
index 1984e4123..eb204e73d 100644
--- a/service/java/com/android/server/wifi/util/XmlUtil.java
+++ b/service/java/com/android/server/wifi/util/XmlUtil.java
@@ -342,7 +342,6 @@ public class XmlUtil {
         public static final String XML_TAG_METERED_HINT = "MeteredHint";
         public static final String XML_TAG_METERED_OVERRIDE = "MeteredOverride";
         public static final String XML_TAG_USE_EXTERNAL_SCORES = "UseExternalScores";
-        public static final String XML_TAG_NUM_ASSOCIATION = "NumAssociation";
         public static final String XML_TAG_CREATOR_UID = "CreatorUid";
         public static final String XML_TAG_CREATOR_NAME = "CreatorName";
         public static final String XML_TAG_LAST_UPDATE_UID = "LastUpdateUid";
@@ -504,7 +503,6 @@ public class XmlUtil {
             XmlUtil.writeNextValue(out, XML_TAG_METERED_OVERRIDE, configuration.meteredOverride);
             XmlUtil.writeNextValue(
                     out, XML_TAG_USE_EXTERNAL_SCORES, configuration.useExternalScores);
-            XmlUtil.writeNextValue(out, XML_TAG_NUM_ASSOCIATION, configuration.numAssociation);
             XmlUtil.writeNextValue(out, XML_TAG_CREATOR_UID, configuration.creatorUid);
             XmlUtil.writeNextValue(out, XML_TAG_CREATOR_NAME, configuration.creatorName);
             XmlUtil.writeNextValue(out, XML_TAG_LAST_UPDATE_UID, configuration.lastUpdateUid);
@@ -674,9 +672,6 @@ public class XmlUtil {
                         case XML_TAG_USE_EXTERNAL_SCORES:
                             configuration.useExternalScores = (boolean) value;
                             break;
-                        case XML_TAG_NUM_ASSOCIATION:
-                            configuration.numAssociation = (int) value;
-                            break;
                         case XML_TAG_CREATOR_UID:
                             configuration.creatorUid = (int) value;
                             break;
diff --git a/tests/wifitests/src/com/android/server/wifi/NetworkListStoreDataTest.java b/tests/wifitests/src/com/android/server/wifi/NetworkListStoreDataTest.java
index d04076aa7..80dfd7ae5 100644
--- a/tests/wifitests/src/com/android/server/wifi/NetworkListStoreDataTest.java
+++ b/tests/wifitests/src/com/android/server/wifi/NetworkListStoreDataTest.java
@@ -95,7 +95,6 @@ public class NetworkListStoreDataTest extends WifiBaseTest {
                     + "<boolean name=\"MeteredHint\" value=\"false\" />\n"
                     + "<int name=\"MeteredOverride\" value=\"2\" />\n"
                     + "<boolean name=\"UseExternalScores\" value=\"false\" />\n"
-                    + "<int name=\"NumAssociation\" value=\"0\" />\n"
                     + "<int name=\"CreatorUid\" value=\"%d\" />\n"
                     + "<string name=\"CreatorName\">%s</string>\n"
                     + "<int name=\"LastUpdateUid\" value=\"-1\" />\n"
@@ -151,7 +150,6 @@ public class NetworkListStoreDataTest extends WifiBaseTest {
                     + "<boolean name=\"MeteredHint\" value=\"false\" />\n"
                     + "<int name=\"MeteredOverride\" value=\"0\" />\n"
                     + "<boolean name=\"UseExternalScores\" value=\"false\" />\n"
-                    + "<int name=\"NumAssociation\" value=\"0\" />\n"
                     + "<int name=\"CreatorUid\" value=\"%d\" />\n"
                     + "<string name=\"CreatorName\">%s</string>\n"
                     + "<int name=\"LastUpdateUid\" value=\"-1\" />\n"
@@ -228,7 +226,6 @@ public class NetworkListStoreDataTest extends WifiBaseTest {
                     + "<boolean name=\"MeteredHint\" value=\"false\" />\n"
                     + "<int name=\"MeteredOverride\" value=\"0\" />\n"
                     + "<boolean name=\"UseExternalScores\" value=\"false\" />\n"
-                    + "<int name=\"NumAssociation\" value=\"0\" />\n"
                     + "<int name=\"CreatorUid\" value=\"%d\" />\n"
                     + "<string name=\"CreatorName\">%s</string>\n"
                     + "<int name=\"LastUpdateUid\" value=\"-1\" />\n"
@@ -291,7 +288,6 @@ public class NetworkListStoreDataTest extends WifiBaseTest {
                     + "<boolean name=\"MeteredHint\" value=\"false\" />\n"
                     + "<int name=\"MeteredOverride\" value=\"0\" />\n"
                     + "<boolean name=\"UseExternalScores\" value=\"false\" />\n"
-                    + "<int name=\"NumAssociation\" value=\"0\" />\n"
                     + "<int name=\"CreatorUid\" value=\"%d\" />\n"
                     + "<string name=\"CreatorName\">%s</string>\n"
                     + "<int name=\"LastUpdateUid\" value=\"-1\" />\n"
@@ -528,7 +524,6 @@ public class NetworkListStoreDataTest extends WifiBaseTest {
                         + "<boolean name=\"NoInternetAccessExpected\" value=\"false\" />\n"
                         + "<boolean name=\"MeteredHint\" value=\"false\" />\n"
                         + "<boolean name=\"UseExternalScores\" value=\"false\" />\n"
-                        + "<int name=\"NumAssociation\" value=\"0\" />\n"
                         + "<int name=\"CreatorUid\" value=\"%d\" />\n"
                         + "<null name=\"CreatorName\" />\n"
                         + "<int name=\"LastUpdateUid\" value=\"-1\" />\n"
diff --git a/tests/wifitests/src/com/android/server/wifi/WifiConfigStoreTest.java b/tests/wifitests/src/com/android/server/wifi/WifiConfigStoreTest.java
index 834165471..671076bf5 100644
--- a/tests/wifitests/src/com/android/server/wifi/WifiConfigStoreTest.java
+++ b/tests/wifitests/src/com/android/server/wifi/WifiConfigStoreTest.java
@@ -107,7 +107,6 @@ public class WifiConfigStoreTest extends WifiBaseTest {
                     + "<boolean name=\"MeteredHint\" value=\"false\" />\n"
                     + "<int name=\"MeteredOverride\" value=\"0\" />\n"
                     + "<boolean name=\"UseExternalScores\" value=\"false\" />\n"
-                    + "<int name=\"NumAssociation\" value=\"0\" />\n"
                     + "<int name=\"CreatorUid\" value=\"%d\" />\n"
                     + "<string name=\"CreatorName\">%s</string>\n"
                     + "<int name=\"LastUpdateUid\" value=\"-1\" />\n"
diff --git a/tests/wifitests/src/com/android/server/wifi/WifiConfigurationTestUtil.java b/tests/wifitests/src/com/android/server/wifi/WifiConfigurationTestUtil.java
index e41bb5901..b991104f3 100644
--- a/tests/wifitests/src/com/android/server/wifi/WifiConfigurationTestUtil.java
+++ b/tests/wifitests/src/com/android/server/wifi/WifiConfigurationTestUtil.java
@@ -635,7 +635,7 @@ public class WifiConfigurationTestUtil {
         assertEquals(expected.meteredHint, actual.meteredHint);
         assertEquals(expected.meteredOverride, actual.meteredOverride);
         assertEquals(expected.useExternalScores, actual.useExternalScores);
-        assertEquals(expected.numAssociation, actual.numAssociation);
+        assertEquals(0, actual.numAssociation);
         assertEquals(expected.creatorUid, actual.creatorUid);
         assertEquals(expected.creatorName, actual.creatorName);
         assertEquals(expected.lastUpdateUid, actual.lastUpdateUid);
-- 
cgit v1.2.3