Remove bogus error from client_cert_resolver.cc

chromeos/network/client_cert_resolver.cc

Index: chromeos/network/client_cert_resolver.cc
diff --git a/chromeos/network/client_cert_resolver.cc b/chromeos/network/client_cert_resolver.cc
index d16f033c590e2701e809bb75fef63891705d0fde..20de5f82f020733ac7c3993183b526f875db49d2 100644
--- a/chromeos/network/client_cert_resolver.cc
+++ b/chromeos/network/client_cert_resolver.cc
@@ -334,15 +334,10 @@ void ClientCertResolver::ResolveNetworks(
     // In any case, don't check this network again in NetworkListChanged.
     resolved_networks_.insert(network->path());
 
-    // If this network is not managed, it cannot have a ClientCertPattern.

[stevenjb, 2014/07/11 21:13:56]

It doesn't look like we really need to check whether or not this is managed
here? Clearly this is no longer a valid test for that.

[pneubeck (no reviews), 2014/07/12 09:07:10]

Right. This was an efficient test for the presence of a policy in the past.

-    if (network->guid().empty())
+    // If this network is not configured, it cannot have a ClientCertPattern.
+    if (network->profile_path().empty())
       continue;
 
-    if (network->profile_path().empty()) {
-      LOG(ERROR) << "Network " << network->path()
-                 << " has a GUID but not profile path";
-      continue;
-    }
     const base::DictionaryValue* policy =
         managed_network_config_handler_->FindPolicyByGuidAndProfile(
             network->guid(), network->profile_path());