Call Device.SetCarrier when the ONC Carrier property is specified.

chromeos/dbus/fake_shill_device_client.h

 // Copyright 2013 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_DBUS_FAKE_SHILL_DEVICE_CLIENT_H_
 #define CHROMEOS_DBUS_FAKE_SHILL_DEVICE_CLIENT_H_
 
 #include <map>
 #include <set>
 #include <string>
@@ skipping 105 matching lines @@
   typedef ObserverList<ShillPropertyChangedObserver> PropertyObserverList;
 
   void SetDefaultProperties();
   void PassStubDeviceProperties(const dbus::ObjectPath& device_path,
                                 const DictionaryValueCallback& callback) const;
 
   // Posts a task to run a void callback with status code |status|.
   void PostVoidCallback(const VoidDBusMethodCallback& callback,
                         DBusMethodCallStatus status);
 
+  // SetProperty helper method.

[pneubeck (no reviews), 2015/03/27 18:26:46]

The function name doesn't really explain what's this for nor is the comment.
What means 'Always'?

Looking at the implementation, maybe call it ...Internal? or Unconditionally...

[stevenjb, 2015/03/27 23:18:04]

In english, 'always' and 'unconditionally' are pretty much synonymous :P
Renamed to SetPropertyInternal.

+  void AlwaysSetProperty(const dbus::ObjectPath& device_path,
+                         const std::string& name,
+                         const base::Value& value,
+                         const base::Closure& callback,
+                         const ErrorCallback& error_callback);
+
   void NotifyObserversPropertyChanged(const dbus::ObjectPath& device_path,
                                       const std::string& property);
   base::DictionaryValue* GetDeviceProperties(const std::string& device_path);
   PropertyObserverList& GetObserverList(const dbus::ObjectPath& device_path);
 
   // Dictionary of <device_name, Dictionary>.
   base::DictionaryValue stub_devices_;
   // Observer list for each device.
   std::map<dbus::ObjectPath, PropertyObserverList*> observer_list_;
 
@@ skipping 14 matching lines @@
   // Note: This should remain the last member so it'll be destroyed and
   // invalidate its weak pointers before any other members are destroyed.
   base::WeakPtrFactory<FakeShillDeviceClient> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(FakeShillDeviceClient);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_DBUS_FAKE_SHILL_DEVICE_CLIENT_H_