Remove use of weakptr for RemovableDeviceNotificationsMac

chrome/browser/system_monitor/removable_device_notifications_mac.mm

Index: chrome/browser/system_monitor/removable_device_notifications_mac.mm
diff --git a/chrome/browser/system_monitor/removable_device_notifications_mac.mm b/chrome/browser/system_monitor/removable_device_notifications_mac.mm
index 185ae18ef727170cd51982aa4fd093ed6baea745..6793a80e09eedeeafa51fb3799e39424c19ae333 100644
--- a/chrome/browser/system_monitor/removable_device_notifications_mac.mm
+++ b/chrome/browser/system_monitor/removable_device_notifications_mac.mm
@@ -14,7 +14,7 @@ static RemovableDeviceNotificationsMac*
     g_removable_device_notifications_mac = NULL;
 
 void GetDiskInfoAndUpdateOnFileThread(
-    const base::WeakPtr<RemovableDeviceNotificationsMac>& notifications,
+    const scoped_refptr<RemovableDeviceNotificationsMac>& notifications,
     base::mac::ScopedCFTypeRef<CFDictionaryRef> dict,
     RemovableDeviceNotificationsMac::UpdateType update_type) {
   DiskInfoMac info = DiskInfoMac::BuildDiskInfoOnFileThread(dict);
@@ -31,7 +31,7 @@ void GetDiskInfoAndUpdateOnFileThread(
 }
 
 void GetDiskInfoAndUpdate(
-    const base::WeakPtr<RemovableDeviceNotificationsMac>& notifications,
+    const scoped_refptr<RemovableDeviceNotificationsMac>& notifications,
     DADiskRef disk,
     RemovableDeviceNotificationsMac::UpdateType update_type) {
   base::mac::ScopedCFTypeRef<CFDictionaryRef> dict(DADiskCopyDescription(disk));
@@ -147,7 +147,7 @@ void RemovableDeviceNotificationsMac::DiskAppearedCallback(
     void* context) {
   RemovableDeviceNotificationsMac* notifications =
       static_cast<RemovableDeviceNotificationsMac*>(context);
-  GetDiskInfoAndUpdate(notifications->AsWeakPtr(),
+  GetDiskInfoAndUpdate(notifications,

[vandebo (ex-Chrome), 2012/09/18 22:41:56]

Does this work? raw pointer passed to function that cast const ref to refptr? 
Maybe the above functions should just take the raw pointer?

[sail, 2012/09/18 22:52:49]

Yep! When the raw pointer is passed to GetDiskInfoAndUpdate() it creates a new
scoped_refptr<> object. This object will ensure that the
RemovableDeviceNotificationsMac instance will stay alive while the task is
pending.
I think using refcounted object is better than raw pointers since it guarantees
that the instance will be alive when the task complets.

We could use the global object and not pass a pointer at all. I think this is
not as great because then that force this object to be a singleton.

[vandebo (ex-Chrome), 2012/09/18 23:03:47]

What I meant is that maybe GetDiskInfoAndUpdate could take a raw pointer.  Bind
will take a reference when it stores the value.  callback.h isn't very clear on
this point, but the way I understand things, if the type of one of the arguments
is refcount'able, then it takes a reference when it stores it for currying. I
don't think it has to be a scoped_refptr for that to work. By creating a
scoped_refptr in the argument list you're doing an extra ref unref that isn't
strictly needed.  But I could be wrong about this.
Agreed.

[sail, 2012/09/18 23:28:51]

Ahh, I see what you mean.
I tried simply removing scoped_refptr<> from GetDiskInfoAndUpdate() but it
didn't work. It seems like base::bind() isn't smart enough to automatically do
this.

                        disk,
                        UPDATE_DEVICE_ADDED);
 }
@@ -158,7 +158,7 @@ void RemovableDeviceNotificationsMac::DiskDisappearedCallback(
     void* context) {
   RemovableDeviceNotificationsMac* notifications =
       static_cast<RemovableDeviceNotificationsMac*>(context);
-  GetDiskInfoAndUpdate(notifications->AsWeakPtr(),
+  GetDiskInfoAndUpdate(notifications,
                        disk,
                        UPDATE_DEVICE_REMOVED);
 }
@@ -170,7 +170,7 @@ void RemovableDeviceNotificationsMac::DiskDescriptionChangedCallback(
     void *context) {
   RemovableDeviceNotificationsMac* notifications =
       static_cast<RemovableDeviceNotificationsMac*>(context);
-  GetDiskInfoAndUpdate(notifications->AsWeakPtr(),
+  GetDiskInfoAndUpdate(notifications,
                        disk,
                        UPDATE_DEVICE_CHANGED);
 }