Chromium Code Reviews| Index: chromeos/network/managed_state.h |
| diff --git a/chromeos/network/managed_state.h b/chromeos/network/managed_state.h |
| index 5dcaff2818a76cf50b38c2240e1dfa72cd3c8fe4..a3c63d8ecf6cdb6d0f67661d50bdda460b262867 100644 |
| --- a/chromeos/network/managed_state.h |
| +++ b/chromeos/network/managed_state.h |
| @@ -40,9 +40,14 @@ class ManagedState { |
| NetworkState* AsNetworkState(); |
| DeviceState* AsDeviceState(); |
| - // Called by NetworkStateHandler when a property changes. Returns false if |
| - // the property was not recognized, was not parsed successfully, or is |
| - // unchanged (complex properties may be assumed to have changed). |
| + // Called by NetworkStateHandler when a property was received. The return |
| + // value indicates if the state changed and is used to reduce the number of |
| + // notifications. The only guarantee however is: If the return value is false |
| + // then the state wasn't modified. This might happen because of |
| + // * |key| was not recognized. |
| + // * |value| was not parsed successfully. |
| + // * |value| is equal to the cached property value. |
| + // If the return value is true, the state might or might not be notified. |
|
stevenjb
2013/05/21 15:40:00
s/notified/modified
pneubeck (no reviews)
2013/05/22 07:35:14
Done.
|
| virtual bool PropertyChanged(const std::string& key, |
| const base::Value& value) = 0; |