Added VLOG(1) loggings to mdns extensions API code

chrome/browser/extensions/api/mdns/dns_sd_registry.cc

 // 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.
 
 #include "chrome/browser/extensions/api/mdns/dns_sd_registry.h"
 
 #include "base/stl_util.h"
 #include "chrome/browser/extensions/api/mdns/dns_sd_device_lister.h"
 #include "chrome/browser/local_discovery/service_discovery_shared_client.h"
 
@@ skipping 35 matching lines @@
 }
 
 bool DnsSdRegistry::ServiceTypeData::UpdateService(
       bool added, const DnsSdService& service) {
   DnsSdRegistry::DnsSdServiceList::iterator it =
       std::find_if(service_list_.begin(),
                    service_list_.end(),
                    IsSameServiceName(service));
   // Set to true when a service is updated in or added to the registry.
   bool updated_or_added = added;
+  VLOG(1) << "UpdateService: " << service.service_name
+          << ", added: " << added
+          << ", known: " << (it != service_list_.end());
   if (it != service_list_.end()) {
     // If added == true, but we still found the service in our cache, then just
     // update the existing entry, but this should not happen!
     DCHECK(!added);
     if (*it != service) {
       *it = service;
       updated_or_added = true;
     }
   } else if (added) {
     service_list_.push_back(service);
   }
+
+  VLOG(1) << "Updated or added service: " << service.service_name << "? : "

[Vitaly Buka (NO REVIEWS), 2014/01/15 23:08:51]

function never returns between logs, wot can combine them here

[imcheng, 2014/01/15 23:30:18]

Done.

+          << updated_or_added;
   return updated_or_added;
 };
 
 bool DnsSdRegistry::ServiceTypeData::RemoveService(
     const std::string& service_name) {
   for (DnsSdRegistry::DnsSdServiceList::iterator it = service_list_.begin();
        it != service_list_.end(); ++it) {
     if ((*it).service_name == service_name) {
       service_list_.erase(it);
       return true;
@@ skipping 36 matching lines @@
 }
 
 DnsSdDeviceLister* DnsSdRegistry::CreateDnsSdDeviceLister(
     DnsSdDelegate* delegate,
     const std::string& service_type,
     local_discovery::ServiceDiscoverySharedClient* discovery_client) {
   return new DnsSdDeviceLister(discovery_client, delegate, service_type);
 }
 
 void DnsSdRegistry::RegisterDnsSdListener(std::string service_type) {
+  VLOG(1) << "RegisterDnsSdListener: " << service_type
+           << ", registered: " << IsRegistered(service_type);

[Vitaly Buka (NO REVIEWS), 2014/01/15 23:08:51]

missalinged

   if (service_type.empty())
     return;
 
   if (IsRegistered(service_type)) {
     service_data_map_[service_type]->ListenerAdded();
     DispatchApiEvent(service_type);
     return;
   }
 
   scoped_ptr<DnsSdDeviceLister> dns_sd_device_lister(CreateDnsSdDeviceLister(
       this, service_type, service_discovery_client_));
   dns_sd_device_lister->Discover(false);
   linked_ptr<ServiceTypeData> service_type_data(
       new ServiceTypeData(dns_sd_device_lister.Pass()));
   service_data_map_[service_type] = service_type_data;
   DispatchApiEvent(service_type);
 }
 
 void DnsSdRegistry::UnregisterDnsSdListener(std::string service_type) {
+  VLOG(1) << "UnregisterDnsSdListener: " << service_type;
   DnsSdRegistry::DnsSdServiceTypeDataMap::iterator it =
       service_data_map_.find(service_type);
   if (it == service_data_map_.end())
     return;
 
   if (service_data_map_[service_type]->ListenerRemoved())
     service_data_map_.erase(it);
 }
 
 void DnsSdRegistry::ServiceChanged(const std::string& service_type,
                                    bool added,
                                    const DnsSdService& service) {
-  if (!IsRegistered(service_type))
+  VLOG(1) << "ServiceChanged: service_type: " << service_type
+           << ", known: " << IsRegistered(service_type)
+           << ", service: " << service.service_name
+           << ", added: " << added;

[Vitaly Buka (NO REVIEWS), 2014/01/15 23:08:51]

missaligned

[imcheng, 2014/01/15 23:30:18]

Done.

+  if (!IsRegistered(service_type)) {
     return;
+  }
 
   VLOG(1) << "Service changed: " << service.service_name;

[Vitaly Buka (NO REVIEWS), 2014/01/15 23:08:51]

dont need line 171

[imcheng, 2014/01/15 23:30:18]

Done.

   if (service_data_map_[service_type]->UpdateService(added, service)) {
+    VLOG(1) << "Updated service and dispatching event for service type: "
+             << service_type;

[Vitaly Buka (NO REVIEWS), 2014/01/15 23:08:51]

missaligned

[imcheng, 2014/01/15 23:30:18]

Done.

     DispatchApiEvent(service_type);
   } else {
     VLOG(1) << "Failed to find existing service to update: "
             << service.service_name;
   }
 }
 
 void DnsSdRegistry::ServiceRemoved(const std::string& service_type,
                                    const std::string& service_name) {
-  if (!IsRegistered(service_type))
+  VLOG(1) << "ServiceRemoved: service_type: " << service_type
+           << "known: " << IsRegistered(service_type)

[Vitaly Buka (NO REVIEWS), 2014/01/15 23:08:51]

", known:...

[imcheng, 2014/01/15 23:30:18]

Done.

+           << ", service: " << service_name;
+  if (!IsRegistered(service_type)) {
     return;
+  }
 
-  VLOG(1) << "Removing service: " << service_name;
   if (service_data_map_[service_type]->RemoveService(service_name)) {

[Vitaly Buka (NO REVIEWS), 2014/01/15 23:08:51]

bool is_removed = service_data_map_[service_type]->RemoveService(service_name);
VLOG(1) << "ServiceRemoved: success:" << is_removed;
if (is_removed) {
no logging
}else{
no logging
}

[imcheng, 2014/01/15 23:30:18]

Done.

+    VLOG(1) << "Removed service and dispatching event for service type: "
+             << service_type;

[Vitaly Buka (NO REVIEWS), 2014/01/15 23:08:51]

missaligned

[imcheng, 2014/01/15 23:30:18]

Done.

     DispatchApiEvent(service_type);
   } else {
     VLOG(1) << "Failed to remove service: " << service_name;
   }
 }
 
 void DnsSdRegistry::ServicesFlushed(const std::string& service_type) {
-  if (!IsRegistered(service_type))
+  VLOG(1) << "ServicesFlushed: service_type: " << service_type
+           << ", known: " << IsRegistered(service_type);

[Vitaly Buka (NO REVIEWS), 2014/01/15 23:08:51]

missaligned

[imcheng, 2014/01/15 23:30:18]

Done.

+  if (!IsRegistered(service_type)) {
     return;
+  }
 
-  if (service_data_map_[service_type]->ClearServices())
+  if (service_data_map_[service_type]->ClearServices()) {
+    VLOG(1) << "Cleared services and dispatching event for service type: "

[Vitaly Buka (NO REVIEWS), 2014/01/15 23:08:51]

same 

[imcheng, 2014/01/15 23:30:18]

Done.

+             << service_type;
     DispatchApiEvent(service_type);
+  } else {
+    VLOG(1) << "Failed to clear services for service type: " << service_type;
+  }
 }
 
 void DnsSdRegistry::DispatchApiEvent(const std::string& service_type) {
   // TODO(justinlin): Make this MaybeDispatchApiEvent instead and dispatch if a
   // dirty bit is set.
+  VLOG(1) << "DispatchApiEvent: service_type: " << service_type;
   FOR_EACH_OBSERVER(DnsSdObserver, observers_, OnDnsSdEvent(
       service_type, service_data_map_[service_type]->GetServiceList()));
 }
 
 bool DnsSdRegistry::IsRegistered(const std::string& service_type) {
   return service_data_map_.find(service_type) != service_data_map_.end();
 }
 
 }  // namespace extensions