Add 'setProperties' to InternetOptionsHandler

chromeos/network/network_state.h

 // 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.
 
 #ifndef CHROMEOS_NETWORK_NETWORK_STATE_H_
 #define CHROMEOS_NETWORK_NETWORK_STATE_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 26 matching lines @@
 
   // ManagedState overrides
   // If you change this method, update GetProperties too.
   virtual bool PropertyChanged(const std::string& key,
                                const base::Value& value) override;
   virtual bool InitialPropertiesReceived(
       const base::DictionaryValue& properties) override;
   virtual void GetStateProperties(
       base::DictionaryValue* dictionary) const override;
 
+  // Like GetStateProperties, but only sets properties needed for configuration
+  // i.e. no visible state properties.

[pneubeck (no reviews), 2014/11/11 16:09:31]

what are 'visible state properties' ?
maybe drop the 'visible', or if 'visible' an example then maybe change to
'i.e. does not set state properties like 'visible'.

[stevenjb, 2014/11/12 01:23:29]

Clarified.

+  void GetConfigurationStateProperties(base::DictionaryValue* dictionary) const;
+
   void IPConfigPropertiesChanged(const base::DictionaryValue& properties);
 
   // Returns true, if the network requires a service activation.
   bool RequiresActivation() const;
 
   // Accessors
   bool visible() const { return visible_; }
   const std::string& security() const { return security_; }
   const std::string& device_path() const { return device_path_; }
   const std::string& guid() const { return guid_; }
@@ skipping 123 matching lines @@
   // TODO(pneubeck): Remove this once (Managed)NetworkConfigurationHandler
   // provides proxy configuration. crbug.com/241775
   base::DictionaryValue proxy_config_;
 
   DISALLOW_COPY_AND_ASSIGN(NetworkState);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_NETWORK_NETWORK_STATE_H_