Add RequestUpdate to NetworkSmsHandler

chromeos/network/network_sms_handler.cc

Index: chromeos/network/network_sms_handler.cc
diff --git a/chromeos/network/network_sms_handler.cc b/chromeos/network/network_sms_handler.cc
index e28e5d37f8d40aad3c0c7f11b1fd4a8ad504e8a3..58b181de5e43df37bca041c58306fb5721f4d90e 100644
--- a/chromeos/network/network_sms_handler.cc
+++ b/chromeos/network/network_sms_handler.cc
@@ -35,6 +35,8 @@ class NetworkSmsHandler::NetworkSmsDeviceHandler {
                           std::string dbus_connection,
                           dbus::ObjectPath object_path);
 
+  void RequestUpdate();
+
  private:
   void ListCallback(const base::ListValue& message_list);
   void SmsReceivedCallback(uint32 index, bool complete);
@@ -74,6 +76,11 @@ NetworkSmsHandler::NetworkSmsDeviceHandler::NetworkSmsDeviceHandler(
                  weak_ptr_factory_.GetWeakPtr()));
 }
 
+void NetworkSmsHandler::NetworkSmsDeviceHandler::RequestUpdate() {
+  DBusThreadManager::Get()->GetGsmSMSClient()->RequestUpdate(
+      dbus_connection_, object_path_);
+}
+
 void NetworkSmsHandler::NetworkSmsDeviceHandler::ListCallback(
     const base::ListValue& message_list) {
   // This receives all messages, so clear any pending deletes.
@@ -151,6 +158,12 @@ void NetworkSmsHandler::Init() {
                  weak_ptr_factory_.GetWeakPtr()));
 }
 
+void NetworkSmsHandler::RequestUpdate() {
+  for (ScopedVector<NetworkSmsDeviceHandler>::iterator iter =
+           device_handlers_.begin(); iter != device_handlers_.end(); ++iter)
+    (*iter)->RequestUpdate();

[satorux1, 2012/05/16 20:59:10]

nit: add {} ? If I understand right, we omit {} only if the conditional and the
body are both one line.

[stevenjb, 2012/05/16 21:12:13]

Done.

+}
+
 void NetworkSmsHandler::AddObserver(Observer* observer) {
   observers_.AddObserver(observer);
 }