MD Settings: Fix networking config errors

extensions/common/api/networking_private.idl

Index: extensions/common/api/networking_private.idl
diff --git a/extensions/common/api/networking_private.idl b/extensions/common/api/networking_private.idl
index 5b9d9bd5f1fb8e1ba699a0d6c843f7fad02356da..176cb2eacc30a1e98a4f79b85cf320d684b08694 100644
--- a/extensions/common/api/networking_private.idl
+++ b/extensions/common/api/networking_private.idl
@@ -233,9 +233,12 @@ namespace networkingPrivate {
     DOMString? ClientCertType;
     DOMString? Identity;
     DOMString? Inner;
-    DOMString Outer;
+    // Outer is required by ONC, but may not be provided when translating
+    // fron Shill. TODO(stevenjb): Fix that in ShillToONCTranslator.

[Devlin, 2017/04/25 16:58:50]

Typo: fron -> from

Note: These become public documents at
developer.chrome.com/extensions/networkingPrivate.  Now, this isn't the end of
the world, because it's a private API, but I'd love to try and minimize the
TODO-style comments we add to these.  Could we put this somewhere else?

[stevenjb, 2017/04/25 18:05:15]

Thanks for pointing that out. The TODO isn't really necessary, I'll just leave
the rest of the comment. The fix is tricky and not really necessary so may not
happen. 

+    DOMString? Outer;

[Devlin, 2017/04/25 16:58:50]

Does networking_onc.idl need to be updated?

[stevenjb, 2017/04/25 18:05:15]

Kind of. It's actually missing the Managed version of these, and has some other
differences with this dictionary, I'm not exactly sure why, and I plan to follow
up on that, but I'll go ahead and at least change its EAPProperties to be
consistent with these changes.

     DOMString? Password;
     boolean? SaveCredentials;
+    DOMString[]? ServerCAPEMs;
     DOMString[]? ServerCARefs;
     boolean? UseProactiveKeyCaching;
     boolean? UseSystemCAs;
@@ -248,9 +251,12 @@ namespace networkingPrivate {
     ManagedDOMString? ClientCertType;
     ManagedDOMString? Identity;
     ManagedDOMString? Inner;
-    ManagedDOMString Outer;
+    // Outer is required by ONC, but may not be provided when translating
+    // fron Shill. TODO(stevenjb): Fix that in ShillToONCTranslator.
+    ManagedDOMString? Outer;
     ManagedDOMString? Password;
     ManagedBoolean? SaveCredentials;
+    ManagedDOMStringList? ServerCAPEMs;
     ManagedDOMStringList? ServerCARefs;
     ManagedBoolean? UseProactiveKeyCaching;
     ManagedBoolean? UseSystemCAs;