From 834fc3a1535214ec38c2186b50f0a20975c5f73c Mon Sep 17 00:00:00 2001
From: Ningyuan Wang <nywang@google.com>
Date: Fri, 21 Apr 2017 21:53:02 +0000
Subject: Revert "Abort ongoing STA scan to prioritize P2P scan"

Let's get it fully tested before checking in.

This reverts commit 75342748e40b4674fa744c8f1904793af571b0d7.

Change-Id: I9288a61f19183c182eaf487c780caa5621230199
---
 service/java/com/android/server/wifi/WifiInjector.java |  2 +-
 .../java/com/android/server/wifi/WificondControl.java  | 18 ------------------
 .../com/android/server/wifi/p2p/WifiP2pNative.java     | 17 +----------------
 3 files changed, 2 insertions(+), 35 deletions(-)

diff --git a/service/java/com/android/server/wifi/WifiInjector.java b/service/java/com/android/server/wifi/WifiInjector.java
index edf396994..817385340 100644
--- a/service/java/com/android/server/wifi/WifiInjector.java
+++ b/service/java/com/android/server/wifi/WifiInjector.java
@@ -161,7 +161,7 @@ public class WifiInjector {
         mWifiP2pMonitor = new WifiP2pMonitor(this);
         mSupplicantP2pIfaceHal = new SupplicantP2pIfaceHal(mWifiP2pMonitor);
         mWifiP2pNative = new WifiP2pNative(SystemProperties.get("wifi.direct.interface", "p2p0"),
-                mSupplicantP2pIfaceHal, mWificondControl);
+                mSupplicantP2pIfaceHal);
 
         // Now get instances of all the objects that depend on the HandlerThreads
         mTrafficPoller =  new WifiTrafficPoller(mContext, mWifiServiceHandlerThread.getLooper(),
diff --git a/service/java/com/android/server/wifi/WificondControl.java b/service/java/com/android/server/wifi/WificondControl.java
index 8be747828..fdd0faffe 100644
--- a/service/java/com/android/server/wifi/WificondControl.java
+++ b/service/java/com/android/server/wifi/WificondControl.java
@@ -465,22 +465,4 @@ public class WificondControl {
         return false;
     }
 
-    /**
-     * Abort ongoing single scan triggered by wificond.
-     * @return Returns true on success.
-     */
-    public boolean abortScan() {
-        if (mWificondScanner == null) {
-            Log.e(TAG, "No valid wificond scanner interface handler");
-            return false;
-        }
-
-        try {
-            return mWificondScanner.abortScan();
-        } catch (RemoteException e1) {
-            Log.e(TAG, "Failed to request abortScan due to remote exception");
-        }
-        return false;
-    }
-
 }
diff --git a/service/java/com/android/server/wifi/p2p/WifiP2pNative.java b/service/java/com/android/server/wifi/p2p/WifiP2pNative.java
index 5de6585f4..bae3faa62 100644
--- a/service/java/com/android/server/wifi/p2p/WifiP2pNative.java
+++ b/service/java/com/android/server/wifi/p2p/WifiP2pNative.java
@@ -21,8 +21,6 @@ import android.net.wifi.p2p.WifiP2pGroup;
 import android.net.wifi.p2p.WifiP2pGroupList;
 import android.net.wifi.p2p.nsd.WifiP2pServiceInfo;
 
-import com.android.server.wifi.WificondControl;
-
 /**
  * Native calls for bring up/shut down of the supplicant daemon and for
  * sending requests to the supplicant daemon
@@ -33,14 +31,11 @@ public class WifiP2pNative {
     private final String mTAG;
     private final String mInterfaceName;
     private final SupplicantP2pIfaceHal mSupplicantP2pIfaceHal;
-    private final WificondControl mWificondControl;
 
-    public WifiP2pNative(String interfaceName, SupplicantP2pIfaceHal p2pIfaceHal,
-            WificondControl wificondcontrol) {
+    public WifiP2pNative(String interfaceName, SupplicantP2pIfaceHal p2pIfaceHal) {
         mTAG = "WifiP2pNative-" + interfaceName;
         mInterfaceName = interfaceName;
         mSupplicantP2pIfaceHal = p2pIfaceHal;
-        mWificondControl = wificondcontrol;
     }
 
     public String getInterfaceName() {
@@ -252,7 +247,6 @@ public class WifiP2pNative {
      * @return boolean value indicating whether operation was successful.
      */
     public boolean p2pFind(int timeout) {
-        abortScan();
         return mSupplicantP2pIfaceHal.find(timeout);
     }
 
@@ -603,13 +597,4 @@ public class WifiP2pNative {
     public boolean saveConfig() {
         return mSupplicantP2pIfaceHal.saveConfig();
     }
-
-    /**
-     * Abort ongoing single scan triggered by wificond.
-     *
-     * @return true on success, false otherwise.
-     */
-    private boolean abortScan() {
-        return mWificondControl.abortScan();
-    }
 }
-- 
cgit v1.2.3