cashew: don't invoke D-Bus methods from within a D-Bus handler

src/device.h

 // Copyright (c) 2010 The Chromium OS 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 SRC_DEVICE_H_
 #define SRC_DEVICE_H_
 
+#include <glib.h>
+
 #include <string>
 
 #include <base/basictypes.h>  // NOLINT
 #include <dbus-c++/dbus.h>  // NOLINT
 
 #include "src/flimflam_device_client_glue.h"
 
 namespace cashew {
 
 class Service;
@@ skipping 29 matching lines @@
     static const char *kCarrierUnknown;
 
     // get cellular carrier for this device
     // returns kCarrierUnknown if we don't know
     virtual const std::string& GetCarrier() const;
 
     // Flimflam Device D-Bus Proxy methods
     virtual void PropertyChanged(const std::string& property_name,
                                  const DBus::Variant& new_value);
 
+    // glib integration interface
+
+    // get idle/timer source id
+    virtual guint GetGetPropertiesSourceId() const;
+
+    // set idle/timer source id
+    virtual void SetGetPropertiesSourceId(guint source_id);
+
+    // are we in the process of retrying our GetProperties call?
+    virtual bool RetryingGetProperties() const;
+
+    // set the retrying flag
+    virtual void OnRetryingGetProperties(bool retrying);
+
   private:
     // back pointer to our parent Service
     Service * const parent_;
 
     // D-Bus path for Flimflam device that we represent
     // this is our unique identifier
     const DBus::Path path_;
 
     // device type
     Type type_;
 
     // cellular carrier
     std::string carrier_;
 
+    // GetProperties timer glib source id
+    // 0 means no source
+    guint get_properties_source_id_;
+
+    // are we in the process of retrying our GetProperties call?
+    // this flag exists to distinguish between our initial g_idle_add call
+    // and our subsequent timer calls
+    bool retrying_get_properties_;
+
     // convert type string to Type enum value
     Type TypeFromString(const std::string& type) const;
 
     // we've received updated Cellular.Carrier info from Flimflam
     void OnCarrierUpdate(const std::string& carrier);
 
     // we've received updated Type info from Flimflam
     void OnTypeUpdate(const std::string& type);
 
+    // glib integration: static wrapper for GetDeviceProperties
+    // takes object ptr as data and invokes object->GetDeviceProperties
+    static gboolean StaticGetDevicePropertiesCallback(gpointer data);
+
     // get device properties from Flimflam
-    void GetDeviceProperties();
+    // returns true on success and false on failure
+    bool GetDeviceProperties();
 
     DISALLOW_COPY_AND_ASSIGN(Device);
 };
 
 }  // namespace cashew
 
 #endif  // SRC_DEVICE_H_