Limit number of service instances passed to onServiceList listeners.

chrome/browser/extensions/api/mdns/mdns_api.h

Index: chrome/browser/extensions/api/mdns/mdns_api.h
diff --git a/chrome/browser/extensions/api/mdns/mdns_api.h b/chrome/browser/extensions/api/mdns/mdns_api.h
index f232d3b07b95432a04137b46d5e01da57ec901cf..65e6c6fe8b88b81b1168f634ba7245c6e4e80ecf 100644
--- a/chrome/browser/extensions/api/mdns/mdns_api.h
+++ b/chrome/browser/extensions/api/mdns/mdns_api.h
@@ -13,6 +13,7 @@
 #include "chrome/browser/extensions/api/mdns/dns_sd_registry.h"
 #include "extensions/browser/browser_context_keyed_api_factory.h"
 #include "extensions/browser/event_router.h"
+#include "extensions/browser/extension_function.h"
 
 namespace content {
 class BrowserContext;
@@ -67,6 +68,18 @@ class MDnsAPI : public BrowserContextKeyedAPI,
   // Update the current list of service types and update the registry.
   void UpdateMDnsListeners(const EventListenerInfo& details);
 
+  // Write a message to the consoles of extensions listening to a given service
+  // type.
+  void WriteToConsole(const std::string& service_type,
+                      content::ConsoleMessageLevel level,
+                      const std::string& message);
+
+  // Finds all all the valid listeners of the mdns.onServiceList event and
+  // outputs them to |listeners| if |listeners| is non-null.  Also outputs the

[mark a. foltz, 2015/04/06 23:01:43]

There is no parameter named |listeners|.

[Red Daly, 2015/04/07 18:08:14]

Done.

+  // corresponding mDNS service types to |service_types| when |service_types| is
+  // non-null.
+  std::set<const EventListener*> GetValidOnServiceListListeners();

[mark a. foltz, 2015/04/06 23:01:43]

This is just used for iteration, return a const_iterator instead.

[Red Daly, 2015/04/07 18:08:14]

N/A with new interface, unless you think it makes sense to take a
const_iterator<std::string>* extension_ids argument instead of a
std::set<std::string>* extension_ids argument.

+
   // Ensure methods are only called on UI thread.
   base::ThreadChecker thread_checker_;
   content::BrowserContext* const browser_context_;