Chromium Code Reviews| Index: chrome/browser/extensions/api/mdns/mdns_api.cc |
| diff --git a/chrome/browser/extensions/api/mdns/mdns_api.cc b/chrome/browser/extensions/api/mdns/mdns_api.cc |
| index 3e30670324c763e66641baa42387f333a1003f8a..c3c98dfef47312ebfdab0c92d3acb7414d0ca509 100644 |
| --- a/chrome/browser/extensions/api/mdns/mdns_api.cc |
| +++ b/chrome/browser/extensions/api/mdns/mdns_api.cc |
| @@ -89,6 +89,7 @@ void MDnsAPI::UpdateMDnsListeners(const EventListenerInfo& details) { |
| for (EventListenerMap::ListenerList::const_iterator it = listeners.begin(); |
| it != listeners.end(); ++it) { |
| base::DictionaryValue* filter = ((*it)->filter.get()); |
| + // TODO(imcheng): Figure out what this is loop for, maybe remove it. |
|
Alpha Left Google
2014/01/15 22:34:13
Don't add this TODO. It has no use to do code.
imcheng
2014/01/15 22:51:37
Ok, looks like this loop does nothing so I will re
|
| for (base::DictionaryValue::Iterator iter(*filter); |
| !iter.IsAtEnd(); iter.Advance()) { |
| } |
| @@ -146,6 +147,8 @@ void MDnsAPI::OnDnsSdEvent(const std::string& service_type, |
| event->restrict_to_browser_context = profile_; |
| event->filter_info.SetServiceType(service_type); |
| + VLOG(1) << "Broadcasting OnServiceList event: " << event.get(); |
| + |
| // TODO(justinlin): To avoid having listeners without filters getting all |
| // events, modify API to have this event require filters. |
| extensions::ExtensionSystem::Get(profile_)->event_router()-> |