Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1757)

Unified Diff: chrome/browser/extensions/api/mdns/mdns_api.cc

Issue 668983003: Enable chrome.mdns for Chrome Apps (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Open up access to chrome.mdns API to all packaged apps Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 572ae1eec7b08e50906f96f94aeda0314b75d1c0..6384ba7c5b57aa891a627200ea700c280c9c975e 100644
--- a/chrome/browser/extensions/api/mdns/mdns_api.cc
+++ b/chrome/browser/extensions/api/mdns/mdns_api.cc
@@ -15,21 +15,6 @@ namespace extensions {
namespace mdns = api::mdns;
-namespace {
-
-// Whitelisted mDNS service types.
-const char kCastServiceType[] = "_googlecast._tcp.local";
-const char kPrivetServiceType[] = "_privet._tcp.local";
-const char kTestServiceType[] = "_testing._tcp.local";
-
-bool IsServiceTypeWhitelisted(const std::string& service_type) {
- return service_type == kCastServiceType ||
- service_type == kPrivetServiceType ||
- service_type == kTestServiceType;
-}
-
-} // namespace
-
MDnsAPI::MDnsAPI(content::BrowserContext* context) : browser_context_(context) {
DCHECK(browser_context_);
EventRouter::Get(context)
@@ -114,13 +99,11 @@ void MDnsAPI::UpdateMDnsListeners(const EventListenerInfo& details) {
DnsSdRegistry* registry = dns_sd_registry();
for (std::set<std::string>::iterator it = added_service_types.begin();
it != added_service_types.end(); ++it) {
- if (IsServiceTypeWhitelisted(*it))
- registry->RegisterDnsSdListener(*it);
+ registry->RegisterDnsSdListener(*it);
mark a. foltz 2014/10/31 06:29:15 To preserve existing behavior, I think extensions
Red Daly 2014/11/17 22:50:01 Done.
}
for (std::set<std::string>::iterator it = removed_service_types.begin();
it != removed_service_types.end(); ++it) {
- if (IsServiceTypeWhitelisted(*it))
- registry->UnregisterDnsSdListener(*it);
+ registry->UnregisterDnsSdListener(*it);
}
service_types_ = new_service_types;

Powered by Google App Engine
This is Rietveld 408576698