diff options
author | Eric Rowe <erowe@google.com> | 2015-05-04 13:13:36 -0700 |
---|---|---|
committer | Eric Rowe <erowe@google.com> | 2015-05-04 13:13:36 -0700 |
commit | 1c22b23fbb4de9be1d590e33fbd24df99f63ccd9 (patch) | |
tree | d77db2de9430f1f1a63ace935f5068a76f8909fa /src/com/android/tradefed/targetprep | |
parent | 26d987949f75a95ab29f848ee3063609925ddf48 (diff) | |
download | tradefederation-1c22b23fbb4de9be1d590e33fbd24df99f63ccd9.tar.gz |
Only disable wifi when ssid is set
Change-Id: I9e2c7b3665805d0ef4669a1cb63688c154260f9c
Diffstat (limited to 'src/com/android/tradefed/targetprep')
-rw-r--r-- | src/com/android/tradefed/targetprep/DeviceSetup2.java | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/com/android/tradefed/targetprep/DeviceSetup2.java b/src/com/android/tradefed/targetprep/DeviceSetup2.java index e6c2cd679..d93a65cd0 100644 --- a/src/com/android/tradefed/targetprep/DeviceSetup2.java +++ b/src/com/android/tradefed/targetprep/DeviceSetup2.java @@ -357,7 +357,9 @@ public class DeviceSetup2 implements ITargetPreparer, ITargetCleaner { return; } - if (mDisconnectWifiAfterTest && device.isWifiEnabled()) { + // Only try to disconnect if wifi ssid is set since isWifiEnabled() is a heavy operation + // which should be avoided when possible + if (mDisconnectWifiAfterTest && mWifiSsid != null && device.isWifiEnabled()) { boolean result = device.disconnectFromWifi(); if (result) { CLog.i("Successfully disconnected from wifi network on %s", |