Use GetManagedProperties in InternetOptionsHandler

chromeos/network/onc/onc_translator_shill_to_onc.cc

Index: chromeos/network/onc/onc_translator_shill_to_onc.cc
diff --git a/chromeos/network/onc/onc_translator_shill_to_onc.cc b/chromeos/network/onc/onc_translator_shill_to_onc.cc
index 9c6052f1e4dda0f6d8e15d7a183b4b3266732b8e..2bd1ebf43b4e788ac870789f8201e0116ed30f10 100644
--- a/chromeos/network/onc/onc_translator_shill_to_onc.cc
+++ b/chromeos/network/onc/onc_translator_shill_to_onc.cc
@@ -115,6 +115,12 @@ class ShillToONCTranslator {
   // entry from |shill_dictionary_| to |onc_object_| if it exists.
   void CopyProperty(const OncFieldSignature* field_signature);
 
+  // If |shill_property_name| exists in |shill_dictionary_|, copy it to
+  // |dictionary|.

[pneubeck (no reviews), 2014/09/03 15:14:29]

nit: mention that it copies to the key |onc_property_name| (called
onc_field_name in all other comments)

[stevenjb, 2014/09/03 21:28:06]

Done, and renamed -> onc_field_name for consistency.

+  void CopyPropertyToDictionary(const std::string& shill_property_name,
+                                const std::string& onc_property_name,
+                                base::DictionaryValue* dictionary);
+
   // If existent, translates the entry at |shill_property_name| in
   // |shill_dictionary_| using |table|. It is an error if no matching table
   // entry is found. Writes the result as entry at |onc_field_name| in
@@ -377,8 +383,7 @@ void ShillToONCTranslator::TranslateNetworkWithState() {
   // Since Name is a read only field in Shill unless it's a VPN, it is copied
   // here, but not when going the other direction (if it's not a VPN).
   std::string name;
-  shill_dictionary_->GetStringWithoutPathExpansion(shill::kNameProperty,
-                                                   &name);
+  shill_dictionary_->GetStringWithoutPathExpansion(shill::kNameProperty, &name);
   onc_object_->SetStringWithoutPathExpansion(::onc::network_config::kName,
                                              name);
 
@@ -418,6 +423,43 @@ void ShillToONCTranslator::TranslateNetworkWithState() {
     TranslateAndAddListOfObjects(::onc::network_config::kIPConfigs,
                                  *shill_ipconfigs);
   }
+
+  // Convert StaticIP and SavedIP properties to dictionary format.
+  scoped_ptr<base::DictionaryValue> static_ip(new base::DictionaryValue);

[pneubeck (no reviews), 2014/09/03 15:14:28]

I think if you can save most of this code by supporting both translation
directions by adding the right signature and entries to the translation tables:

Add two new signatures that point to the base IPConfig signature (these
signatures inherit all members from kIPConfigSignature):

const OncValueSignature kStaticIPConfigSignature = {
 base::Value::TYPE_DICTIONARY, NULL, NULL, &kIPConfigSignature };

const OncValueSignature kSavedIPConfigSignature = {
 base::Value::TYPE_DICTIONARY, NULL, NULL, &kIPConfigSignature };

... network_configuration_fields[] = {
   ...
   { ::onc::network_config::kSavedIPConfig, &kSavedIPConfigSignature},
   { ::onc::network_config::kStaticIPConfig, &kStaticIPConfigSignature},
   ...
};

And for each signature add a translation table:

const FieldTranslationEntry static_ipconfig_fields[] = {
  { ::onc::ipconfig::kIPAddress, shill::kStaticIPAddressProperty},
  { ::onc::ipconfig::kGateway, shill::kStaticIPGatewayProperty},
  { ::onc::ipconfig::kRoutingPrefix, shill::kStaticIPPrefixlenProperty},
  { ::onc::ipconfig::kNameServers, shill::kStaticIPNameServersProperty}
};

const FieldTranslationEntry saved_ipconfig_fields[] = {
  ...
};

const OncValueTranslationEntry onc_value_translation_table[] = {
  ...
  { &kSavedIPConfigSignature, saved_ipconfig_fields },
  { &kStaticIPConfigSignature, static_ipconfig_fields },
  ...
};


To trigger the translation of these nested objects, add here the calls
 TranslateAndAddNestedObject(::onc::network_config::kSavedIPConfig)
 TranslateAndAddNestedObject(::onc::network_config::kStaticIPConfig)

The new function CopyPropertyToDictionary will not be necessary anymore and you
get the other direction of translation at the same time.

[stevenjb, 2014/09/03 21:28:06]

I'll... try doing this in a separate CL. I think I agree that sounds better, but
it all still kind of makes my head hurt.

[pneubeck (no reviews), 2014/09/04 14:54:13]

Thanks

+  CopyPropertyToDictionary(shill::kStaticIPAddressProperty,
+                           ::onc::ipconfig::kIPAddress,
+                           static_ip.get());
+  CopyPropertyToDictionary(shill::kStaticIPGatewayProperty,
+                           ::onc::ipconfig::kGateway,
+                           static_ip.get());
+  CopyPropertyToDictionary(shill::kStaticIPNameServersProperty,
+                           ::onc::ipconfig::kNameServers,
+                           static_ip.get());
+  CopyPropertyToDictionary(shill::kStaticIPPrefixlenProperty,
+                           ::onc::ipconfig::kRoutingPrefix,
+                           static_ip.get());
+  if (!static_ip->empty()) {
+    onc_object_->SetWithoutPathExpansion(::onc::network_config::kStaticIPConfig,
+                                         static_ip.release());
+  }
+
+  scoped_ptr<base::DictionaryValue> saved_ip(new base::DictionaryValue);
+  CopyPropertyToDictionary(shill::kSavedIPAddressProperty,
+                           ::onc::ipconfig::kIPAddress,
+                           saved_ip.get());
+  CopyPropertyToDictionary(shill::kSavedIPGatewayProperty,
+                           ::onc::ipconfig::kGateway,
+                           saved_ip.get());
+  CopyPropertyToDictionary(shill::kSavedIPNameServersProperty,
+                           ::onc::ipconfig::kNameServers,
+                           saved_ip.get());
+  CopyPropertyToDictionary(shill::kSavedIPPrefixlenProperty,
+                           ::onc::ipconfig::kRoutingPrefix,
+                           saved_ip.get());
+  if (!saved_ip->empty()) {
+    onc_object_->SetWithoutPathExpansion(::onc::network_config::kSavedIPConfig,
+                                         saved_ip.release());
+  }
 }
 
 void ShillToONCTranslator::TranslateIPConfig() {
@@ -551,6 +593,19 @@ void ShillToONCTranslator::CopyProperty(
                                       shill_value->DeepCopy());
 }
 
+void ShillToONCTranslator::CopyPropertyToDictionary(
+    const std::string& shill_property_name,
+    const std::string& onc_property_name,
+    base::DictionaryValue* dictionary) {
+  const base::Value* shill_value;
+  if (!shill_dictionary_->GetWithoutPathExpansion(shill_property_name,
+                                                  &shill_value)) {
+    return;
+  }
+  dictionary->SetWithoutPathExpansion(onc_property_name,
+                                      shill_value->DeepCopy());
+}
+
 void ShillToONCTranslator::TranslateWithTableAndSet(
     const std::string& shill_property_name,
     const StringTranslationEntry table[],