Add ConnectToBestServices call when Certs are loaded

chromeos/network/shill_property_handler.h

Index: chromeos/network/shill_property_handler.h
diff --git a/chromeos/network/shill_property_handler.h b/chromeos/network/shill_property_handler.h
index 66678acab34c7e73e6119c8bf1ec00b6ed8e7d96..154a65650651839789fbc5e756ae8a62128bf5fc 100644
--- a/chromeos/network/shill_property_handler.h
+++ b/chromeos/network/shill_property_handler.h
@@ -38,7 +38,8 @@ class ShillPropertyObserver;
 // It also observes Shill.Service for all services in Manager.ServiceWatchList.
 // This class must not outlive the ShillManagerClient instance.
 class CHROMEOS_EXPORT ShillPropertyHandler
-    : public ShillPropertyChangedObserver {
+    : public ShillPropertyChangedObserver,
+      public base::SupportsWeakPtr<ShillPropertyHandler> {

[Greg Spencer (Chromium), 2013/04/01 19:37:18]

Ooh, is that the new way to do weak pointer support?  Nice.

[stevenjb, 2013/04/01 20:34:48]

It's not exactly new, I'm not sure why I usually use a member instead of
inheriting SupportsWeakPtr. This is definitely better though if another class
needs to access this as a weak ptr though.

  public:
   typedef std::map<std::string, ShillPropertyObserver*>
       ShillPropertyObserverMap;
@@ -107,6 +108,9 @@ class CHROMEOS_EXPORT ShillPropertyHandler
   // Requests an immediate network scan.
   void RequestScan() const;
 
+  // Calls Manager.ConnectToBestServices().
+  void ConnectToBestServices() const;
+
   // Requests all properties for the service or device (called for new items).
   void RequestProperties(ManagedState::ManagedType type,
                          const std::string& path);
@@ -184,9 +188,6 @@ class CHROMEOS_EXPORT ShillPropertyHandler
   std::set<std::string> enabled_technologies_;
   std::set<std::string> uninitialized_technologies_;
 
-  // For Shill client callbacks
-  base::WeakPtrFactory<ShillPropertyHandler> weak_ptr_factory_;
-
   DISALLOW_COPY_AND_ASSIGN(ShillPropertyHandler);
 };