Reimplement RequestHiddenWifiNetworkProperties and RequestVirtualNetworkProperties without Libcros

chrome/browser/chromeos/cros/cros_network_functions.cc

 // Copyright (c) 2012 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/chromeos/cros/cros_network_functions.h"
 
 #include "base/bind.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/values.h"
 #include "chrome/browser/chromeos/cros/gvalue_util.h"
@@ skipping 193 matching lines @@
 }
 
 // A callback used to implement CrosRequest*Properties functions.
 void RunCallbackWithDictionaryValue(const NetworkPropertiesCallback& callback,
                                     const std::string& path,
                                     DBusMethodCallStatus call_status,
                                     const base::DictionaryValue& value) {
   callback.Run(path, call_status == DBUS_METHOD_CALL_SUCCESS ? &value : NULL);
 }
 
+// Used as a callback for FlimflamManagerClient::GetService
+void OnGetService(const NetworkPropertiesCallback& callback,
+                  DBusMethodCallStatus call_status,
+                  const dbus::ObjectPath& service_path) {
+  if (call_status == DBUS_METHOD_CALL_SUCCESS) {
+    DBusThreadManager::Get()->GetFlimflamServiceClient()->GetProperties(
+        service_path, base::Bind(&RunCallbackWithDictionaryValue,
+                                 callback,
+                                 service_path.value()));
+  }
+}
+
 // Used as a callback for chromeos::ConfigureService.
 void OnConfigureService(void* object,
                         const char* service_path,
                         NetworkMethodErrorType error,
                         const char* error_message) {
   if (error != NETWORK_METHOD_ERROR_NONE) {
     LOG(WARNING) << "Error from ConfigureService callback: "
                  << " Error: " << error << " Message: " << error_message;
   }
 }
@@ skipping 234 matching lines @@
         base::Bind(&RunCallbackWithDictionaryValue,
                    callback,
                    profile_entry_path));
   }
 }
 
 void CrosRequestHiddenWifiNetworkProperties(
     const std::string& ssid,
     const std::string& security,
     const NetworkPropertiesCallback& callback) {
-  // The newly allocated callback will be deleted in OnRequestNetworkProperties.
-  chromeos::RequestHiddenWifiNetworkProperties(
-      ssid.c_str(),
-      security.c_str(),
-      &OnRequestNetworkProperties,
-      new OnRequestNetworkPropertiesCallback(callback));
+  if (g_libcros_network_functions_enabled) {
+    // The newly allocated callback will be deleted in
+    // OnRequestNetworkProperties.
+    chromeos::RequestHiddenWifiNetworkProperties(
+        ssid.c_str(),
+        security.c_str(),
+        &OnRequestNetworkProperties,
+        new OnRequestNetworkPropertiesCallback(callback));
+  } else {
+    base::DictionaryValue properties;
+    properties.SetWithoutPathExpansion(
+        flimflam::kModeProperty,
+        base::Value::CreateStringValue(flimflam::kModeManaged));
+    properties.SetWithoutPathExpansion(
+        flimflam::kTypeProperty,
+        base::Value::CreateStringValue(flimflam::kTypeWifi));
+    properties.SetWithoutPathExpansion(
+        flimflam::kSSIDProperty,
+        base::Value::CreateStringValue(ssid));
+    properties.SetWithoutPathExpansion(
+        flimflam::kSecurityProperty,
+        base::Value::CreateStringValue(security));
+    // flimflam.Manger.GetService() will apply the property changes in
+    // |properties| and return a new or existing service to OnGetService().
+    // OnGetService will then call GetProperties which will then call callback.
+    DBusThreadManager::Get()->GetFlimflamManagerClient()->GetService(
+        properties, base::Bind(&OnGetService, callback));
+  }
 }
 
 void CrosRequestVirtualNetworkProperties(
     const std::string& service_name,
     const std::string& server_hostname,
     const std::string& provider_type,
     const NetworkPropertiesCallback& callback) {
-  // The newly allocated callback will be deleted in OnRequestNetworkProperties.
-  chromeos::RequestVirtualNetworkProperties(
-      service_name.c_str(),
-      server_hostname.c_str(),
-      provider_type.c_str(),
-      &OnRequestNetworkProperties,
-      new OnRequestNetworkPropertiesCallback(callback));
+  if (g_libcros_network_functions_enabled) {
+    // The newly allocated callback will be deleted in
+    // OnRequestNetworkProperties.
+    chromeos::RequestVirtualNetworkProperties(
+        service_name.c_str(),
+        server_hostname.c_str(),
+        provider_type.c_str(),
+        &OnRequestNetworkProperties,
+        new OnRequestNetworkPropertiesCallback(callback));
+  } else {
+    base::DictionaryValue properties;
+    properties.SetWithoutPathExpansion(
+        flimflam::kTypeProperty, base::Value::CreateStringValue("vpn"));

[stevenjb, 2012/04/23 21:37:26]

Use kTypeVPN

[hashimoto, 2012/04/24 03:53:11]

Done.

+    properties.SetWithoutPathExpansion(
+        flimflam::kProviderNameProperty,
+        base::Value::CreateStringValue(service_name));
+    properties.SetWithoutPathExpansion(
+        flimflam::kProviderHostProperty,
+        base::Value::CreateStringValue(server_hostname));
+    properties.SetWithoutPathExpansion(
+        flimflam::kProviderTypeProperty,
+        base::Value::CreateStringValue(provider_type));
+    // The actual value of Domain does not matter, so just use service_name.
+    properties.SetWithoutPathExpansion(
+        flimflam::kVPNDomainProperty,
+        base::Value::CreateStringValue(service_name));
+
+    // flimflam.Manger.GetService() will apply the property changes in
+    // |properties| and pass a new or existing service to OnGetService().
+    // OnGetService will then call GetProperties which will then call callback.
+    DBusThreadManager::Get()->GetFlimflamManagerClient()->GetService(
+        properties, base::Bind(&OnGetService, callback));
+  }
 }
 
 void CrosRequestNetworkServiceDisconnect(const std::string& service_path) {
   if (g_libcros_network_functions_enabled) {
     chromeos::RequestNetworkServiceDisconnect(service_path.c_str());
   } else {
     DBusThreadManager::Get()->GetFlimflamServiceClient()->Disconnect(
         dbus::ObjectPath(service_path), base::Bind(&DoNothing));
   }
 }
@@ skipping 167 matching lines @@
     ScopedGHashTable ghash(
         ConvertDictionaryValueToStringValueGHashTable(properties));
     chromeos::ConfigureService("", ghash.get(), OnConfigureService, NULL);
   } else {
     DBusThreadManager::Get()->GetFlimflamManagerClient()->ConfigureService(
         properties, base::Bind(&DoNothing));
   }
 }
 
 }  // namespace chromeos