This adds the setProperties and getState functions to the networking API

chrome/browser/chromeos/extensions/networking_private_api.cc

Index: chrome/browser/chromeos/extensions/networking_private_api.cc
diff --git a/chrome/browser/chromeos/extensions/networking_private_api.cc b/chrome/browser/chromeos/extensions/networking_private_api.cc
index dbafaf8a7379f8913974fdb37ada39ae22eb7189..3b8acd623bde61e9444b413c605ad5f9e8346418 100644
--- a/chrome/browser/chromeos/extensions/networking_private_api.cc
+++ b/chrome/browser/chromeos/extensions/networking_private_api.cc
@@ -63,6 +63,71 @@ void NetworkingPrivateGetPropertiesFunction::GetPropertiesFailed(
 }
 
 ////////////////////////////////////////////////////////////////////////////////
+// NetworkingPrivateGetStateFunction
+
+NetworkingPrivateGetStateFunction::
+  ~NetworkingPrivateGetStateFunction() {
+}
+
+bool NetworkingPrivateGetStateFunction::RunImpl() {
+  scoped_ptr<api::GetState::Params> params =
+      api::GetState::Params::Create(*args_);
+  EXTENSION_FUNCTION_VALIDATE(params);
+  // The |network_guid| parameter is storing the service path.
+  std::string service_path = params->network_guid;
+
+  const NetworkState* state =
+      NetworkStateHandler::Get()->GetNetworkState(service_path);
+  if (!state) {
+    error_ = "Error.InvalidParameter";

[stevenjb, 2013/03/07 00:43:22]

Should this be "InvalidParameter"? A network might have disconnected and be no
longer available.

[Greg Spencer (Chromium), 2013/03/07 22:01:23]

Well, my thought was that the service path was no longer valid, but if it can be
still valid and still get back NULL, then what should I return (i.e. how can I
distinguish between an invalid service path and a disconnected network)?

[stevenjb, 2013/03/08 19:04:16]

I guess I was thinking that the service could be "no longer available", which
isn't exactly the same as "invalid", but I guess it is impractical to
differentiate that (we'd have to track services that were available) and no real
practical difference between the two anyway.

+    SendResponse(false);
+  }
+
+  scoped_ptr<base::DictionaryValue> result_dict(new base::DictionaryValue);
+  state->FillDictionary(result_dict.get());

[pneubeck (no reviews), 2013/03/07 13:06:18]

|result_dict| has to be translated to ONC, since "FillDictionary" doesn't do
that anymore.
See line 155 on the right.

[Greg Spencer (Chromium), 2013/03/07 22:01:23]

Ahh, no wonder my results were off...

+  SetResult(result_dict.release());
+  SendResponse(true);
+
+  return true;
+}
+
+////////////////////////////////////////////////////////////////////////////////
+// NetworkingPrivateSetPropertiesFunction
+
+NetworkingPrivateSetPropertiesFunction::
+  ~NetworkingPrivateSetPropertiesFunction() {

[stevenjb, 2013/03/07 00:43:22]

No WS at beginning of line here I think

[Greg Spencer (Chromium), 2013/03/07 22:01:23]

OK. :-)  I don't think there's a lot of precedent for this, so I was kind of
winging it.  The whitespace felt to me like it was a continuation more than not
having it.

[stevenjb, 2013/03/08 19:04:16]

There's not a ton of precedent, but it's what I have seen and used before, and
is consistent with the Google Emacs formatting plugin :)

+}
+
+bool NetworkingPrivateSetPropertiesFunction::RunImpl() {
+  scoped_ptr<api::SetProperties::Params> params =
+      api::SetProperties::Params::Create(*args_);
+  EXTENSION_FUNCTION_VALIDATE(params);
+
+  scoped_ptr<base::DictionaryValue> properties_dict(
+      params->properties.ToValue());
+
+  ManagedNetworkConfigurationHandler::Get()->SetProperties(
+      params->network_guid,
+      *properties_dict,
+      base::Bind(&NetworkingPrivateSetPropertiesFunction::ResultCallback,
+                 this),
+      base::Bind(&NetworkingPrivateSetPropertiesFunction::ErrorCallback,
+                 this));
+  return true;
+}
+
+void NetworkingPrivateSetPropertiesFunction::ErrorCallback(
+    const std::string& error_name,
+    const scoped_ptr<base::DictionaryValue> error_data) {
+  error_ = error_name;
+  SendResponse(false);
+}
+
+void NetworkingPrivateSetPropertiesFunction::ResultCallback() {
+  SendResponse(true);
+}
+
+////////////////////////////////////////////////////////////////////////////////
 // NetworkingPrivateGetVisibleNetworksFunction
 
 NetworkingPrivateGetVisibleNetworksFunction::
@@ -304,3 +369,4 @@ void NetworkingPrivateVerifyAndEncryptDataFunction::ErrorCallback(
   error_ = error_name;
   SendResponse(false);
 }
+