Limit number of service instances passed to onServiceList listeners.

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

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_API_MDNS_MDNS_API_H_
 #define CHROME_BROWSER_EXTENSIONS_API_MDNS_MDNS_API_H_
 
 #include <set>
 #include <string>
 
 #include "base/memory/scoped_ptr.h"
 #include "base/threading/thread_checker.h"
 #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;
 }
 
 namespace extensions {
 
 class DnsSdRegistry;
 
 // MDnsAPI is instantiated with the profile and will listen for extensions that
@@ skipping 34 matching lines @@
   static const char* service_name() {
     return "MDnsAPI";
   }
 
   static const bool kServiceIsCreatedWithBrowserContext = true;
   static const bool kServiceIsNULLWhileTesting = true;
 
   // 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_;
   // Lazily created on first access and destroyed with this API class.
   scoped_ptr<DnsSdRegistry> dns_sd_registry_;
   // Current set of service types registered with the registry.
   std::set<std::string> service_types_;
 
   DISALLOW_COPY_AND_ASSIGN(MDnsAPI);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_MDNS_MDNS_API_H_