Use Name not SSID in CopyIdentifyingProperties

chromeos/network/shill_property_util.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 CHROMEOS_NETWORK_SHILL_PROPERTY_UTIL_H_
 #define CHROMEOS_NETWORK_SHILL_PROPERTY_UTIL_H_
 
 #include <string>
 
 #include "base/memory/scoped_ptr.h"
@@ skipping 24 matching lines @@
 // |shill_dictionary|. If parsing fails or the field doesn't exist, returns
 // NULL.
 scoped_ptr<NetworkUIData> GetUIDataFromProperties(
     const base::DictionaryValue& shill_dictionary);
 
 // Sets the UIData property in |shill_dictionary| to the serialization of
 // |ui_data|.
 void SetUIData(const NetworkUIData& ui_data,
                base::DictionaryValue* shill_dictionary);
 
-// Copy configuration properties required by Shill to identify a network.
-// Only WiFi, VPN, Ethernet and EthernetEAP are supported. WiMax and Cellular
-// are not supported. Returns true only if all required properties could be
-// copied.
-bool CopyIdentifyingProperties(const base::DictionaryValue& service_properties,

[pneubeck (no reviews), 2013/09/14 16:04:25]

please keep this function as it was. I made it visible in this file, because I
need it for the autoconnect policy.

[stevenjb, 2013/09/16 16:29:49]

OK, but let's make sure we use it soon or remove it; we can always restore it if
we need it later.

-                               base::DictionaryValue* dest);
-
 }  // namespace shill_property_util
 
 class CHROMEOS_EXPORT NetworkTypePattern {
  public:
   // Matches any network.
   static NetworkTypePattern Default();
 
   // Matches wireless networks
   static NetworkTypePattern Wireless();
 
@@ skipping 34 matching lines @@
 
   // The bit array of the matching network types.
   int pattern_;
 
   DISALLOW_ASSIGN(NetworkTypePattern);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_NETWORK_SHILL_PROPERTY_UTIL_H_