commit 287730e764d00542e61562562de57a2e52e74d47
author Hugo Benichi <hugobenichi@google.com> Fri Apr 28 13:29:21 2017 +0900
committer Hugo Benichi <hugobenichi@google.com> Tue May 02 15:26:33 2017 +0900
tree 972702da794b573565b084770eda1a73381bcd2f
parent 3ec710f6f03eb1edd05c0a48a992f79a6ac7f925

NsdManager: add test coverage for discoverServices()

Test: added new tests
Bug: 37013369, 33298084
Change-Id: If6c1113ee1bd4690db90abd5a61c3e229c827aec

core/java/android/net/nsd/NsdManager.java

diff --git a/core/java/android/net/nsd/NsdManager.java b/core/java/android/net/nsd/NsdManager.java
index 4c33c36..3fd9f19 100644
--- a/core/java/android/net/nsd/NsdManager.java
+++ b/core/java/android/net/nsd/NsdManager.java
@@ -372,6 +372,8 @@
                     ((DiscoveryListener) listener).onServiceLost((NsdServiceInfo) message.obj);
                     break;
                 case STOP_DISCOVERY_FAILED:
+                    // TODO: failure to stop discovery should be internal and retried internally, as
+                    // the effect for the client is indistinguishable from STOP_DISCOVERY_SUCCEEDED
                     removeListener(message.arg2);
                     ((DiscoveryListener) listener).onStopDiscoveryFailed(getNsdServiceInfoType(ns),
                             message.arg1);