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

chromeos/dbus/fake_shill_device_client.h

Index: chromeos/dbus/fake_shill_device_client.h
diff --git a/chromeos/dbus/fake_shill_device_client.h b/chromeos/dbus/fake_shill_device_client.h
index 44bc15d647f37497dae4b6cdb1b63f0767fdef75..aabb01e2f4a81a2879e2ebda4e6aac7e0d736ca9 100644
--- a/chromeos/dbus/fake_shill_device_client.h
+++ b/chromeos/dbus/fake_shill_device_client.h
@@ -123,6 +123,13 @@ class CHROMEOS_EXPORT FakeShillDeviceClient
   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);