From dc6cf4b683c6b8af678dc7f50c069879013f9fa5 Mon Sep 17 00:00:00 2001 From: Sundeep Ghuman Date: Wed, 8 Mar 2017 16:18:29 -0800 Subject: [PATCH] Necessary AccessPoint visibility change for WifiNetworkDetailsFragment. Refactor constant name in WifiTracker for readability. Bug: 34713316 Test: runtest --path frameworks/base/packages/SettingsLib/tests/integ/src/com/android/settingslib/wifi/AccessPointTest.java runtest --path frameworks/base/packages/SettingsLib/tests/integ/src/com/android/settingslib/wifi/WifiTrackerTest.java Change-Id: I41b0d69303d0452e2c2c22bcbddc34ae3932e99e --- .../src/com/android/settingslib/wifi/AccessPoint.java | 2 +- .../src/com/android/settingslib/wifi/WifiTracker.java | 5 +++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/packages/SettingsLib/src/com/android/settingslib/wifi/AccessPoint.java b/packages/SettingsLib/src/com/android/settingslib/wifi/AccessPoint.java index 6fe581eab86dd..0ba3f7c229419 100644 --- a/packages/SettingsLib/src/com/android/settingslib/wifi/AccessPoint.java +++ b/packages/SettingsLib/src/com/android/settingslib/wifi/AccessPoint.java @@ -848,7 +848,7 @@ public class AccessPoint implements Comparable { return false; } - boolean update(WifiConfiguration config, WifiInfo info, NetworkInfo networkInfo) { + public boolean update(WifiConfiguration config, WifiInfo info, NetworkInfo networkInfo) { boolean reorder = false; if (info != null && isInfoForThisAccessPoint(config, info)) { reorder = (mInfo == null); diff --git a/packages/SettingsLib/src/com/android/settingslib/wifi/WifiTracker.java b/packages/SettingsLib/src/com/android/settingslib/wifi/WifiTracker.java index 8421c2c9dee3b..0771dd88028ef 100644 --- a/packages/SettingsLib/src/com/android/settingslib/wifi/WifiTracker.java +++ b/packages/SettingsLib/src/com/android/settingslib/wifi/WifiTracker.java @@ -471,7 +471,8 @@ public class WifiTracker { accessPoint.update(connectionConfig, mLastInfo, mLastNetworkInfo); } if (mIncludeSaved) { - // If saved network not present in scan result then set its Rssi to MAX_VALUE + // If saved network not present in scan result then set its Rssi to + // UNREACHABLE_RSSI boolean apFound = false; for (ScanResult result : results) { if (result.SSID.equals(accessPoint.getSsidStr())) { @@ -480,7 +481,7 @@ public class WifiTracker { } } if (!apFound) { - accessPoint.setRssi(Integer.MAX_VALUE); + accessPoint.setRssi(AccessPoint.UNREACHABLE_RSSI); } accessPoints.add(accessPoint); apMap.put(accessPoint.getSsidStr(), accessPoint);