Cleanup: Remove ScopedGenericObj usage in chrome/.

Review URL: https://chromiumcodereview.appspot.com/16041004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@202754 0039d316-1c4b-4281-b951-d872f2087c98

3 files changed, 30 insertions, 26 deletions

diff --git a/chrome/browser/storage_monitor/storage_monitor_linux.cc b/chrome/browser/storage_monitor/storage_monitor_linux.cc
index 50848c2..1626f8e 100644
--- a/chrome/browser/storage_monitor/storage_monitor_linux.cc
+++ b/chrome/browser/storage_monitor/storage_monitor_linux.cc
@@ -160,27 +160,30 @@ scoped_ptr<StorageInfo> GetDeviceInfo(const base::FilePath& device_path,
     return storage_info.Pass();  // Not a supported type.
 
   ScopedUdevDeviceObject device(
-      udev_device_new_from_devnum(udev_obj, device_type, device_stat.st_rdev));
+      udev_device_new_from_devnum(udev_obj.get(), device_type,
+                                  device_stat.st_rdev));
   if (!device.get())
     return storage_info.Pass();
 
-  string16 volume_label = UTF8ToUTF16(GetUdevDevicePropertyValue(device,
+  string16 volume_label = UTF8ToUTF16(GetUdevDevicePropertyValue(device.get(),
                                                                  kLabel));
-  string16 vendor_name = UTF8ToUTF16(GetUdevDevicePropertyValue(device,
+  string16 vendor_name = UTF8ToUTF16(GetUdevDevicePropertyValue(device.get(),
                                                                 kVendor));
-  string16 model_name = UTF8ToUTF16(GetUdevDevicePropertyValue(device, kModel));
+  string16 model_name = UTF8ToUTF16(GetUdevDevicePropertyValue(device.get(),
+                                                               kModel));
 
-  std::string unique_id = MakeDeviceUniqueId(device);
+  std::string unique_id = MakeDeviceUniqueId(device.get());
 
   // Keep track of device info details to see how often we get invalid values.
   MediaStorageUtil::RecordDeviceInfoHistogram(true, unique_id, volume_label);
 
-  const char* value = udev_device_get_sysattr_value(device, kRemovableSysAttr);
+  const char* value =
+      udev_device_get_sysattr_value(device.get(), kRemovableSysAttr);
   if (!value) {
     // |parent_device| is owned by |device| and does not need to be cleaned
     // up.
     struct udev_device* parent_device =
-        udev_device_get_parent_with_subsystem_devtype(device,
+        udev_device_get_parent_with_subsystem_devtype(device.get(),
                                                       kBlockSubsystemKey,
                                                       kDiskDeviceTypeKey);
     value = udev_device_get_sysattr_value(parent_device, kRemovableSysAttr);
@@ -204,7 +207,7 @@ scoped_ptr<StorageInfo> GetDeviceInfo(const base::FilePath& device_path,
       volume_label,
       vendor_name,
       model_name,
-      GetDeviceStorageSize(device_path, device)));
+      GetDeviceStorageSize(device_path, device.get())));
   return storage_info.Pass();
 }
 
diff --git a/chrome/browser/storage_monitor/udev_util_linux.cc b/chrome/browser/storage_monitor/udev_util_linux.cc
index e6c62ba..c88260c 100644
--- a/chrome/browser/storage_monitor/udev_util_linux.cc
+++ b/chrome/browser/storage_monitor/udev_util_linux.cc
@@ -8,6 +8,14 @@
 
 namespace chrome {
 
+void UdevDeleter::operator()(struct udev* udev) {
+  udev_unref(udev);
+}
+
+void UdevDeviceDeleter::operator()(struct udev_device* device) {
+  udev_device_unref(device);
+}
+
 std::string GetUdevDevicePropertyValue(struct udev_device* udev_device,
                                        const char* key) {
   const char* value = udev_device_get_property_value(udev_device, key);
diff --git a/chrome/browser/storage_monitor/udev_util_linux.h b/chrome/browser/storage_monitor/udev_util_linux.h
index fe20f19..308af96 100644
--- a/chrome/browser/storage_monitor/udev_util_linux.h
+++ b/chrome/browser/storage_monitor/udev_util_linux.h
@@ -9,7 +9,7 @@
 
 #include <string>
 
-#include "base/memory/scoped_generic_obj.h"
+#include "base/memory/scoped_ptr.h"
 
 namespace base {
 class FilePath;
@@ -17,25 +17,18 @@ class FilePath;
 
 namespace chrome {
 
-// ScopedGenericObj functor for UdevObjectRelease().
-class ScopedReleaseUdevObject {
- public:
-  void operator()(struct udev* udev) const {
-    udev_unref(udev);
-  }
+// Deleter for ScopedUdevObject.
+struct UdevDeleter {
+  void operator()(struct udev* udev);
 };
-typedef ScopedGenericObj<struct udev*,
-                         ScopedReleaseUdevObject> ScopedUdevObject;
-
-// ScopedGenericObj functor for UdevDeviceObjectRelease().
-class ScopedReleaseUdevDeviceObject {
- public:
-  void operator()(struct udev_device* device) const {
-    udev_device_unref(device);
-  }
+typedef scoped_ptr<struct udev, UdevDeleter> ScopedUdevObject;
+
+// Deleter for ScopedUdevDeviceObject().
+struct UdevDeviceDeleter {
+  void operator()(struct udev_device* device);
 };
-typedef ScopedGenericObj<struct udev_device*,
-                         ScopedReleaseUdevDeviceObject> ScopedUdevDeviceObject;
+typedef scoped_ptr<struct udev_device, UdevDeviceDeleter>
+    ScopedUdevDeviceObject;
 
 // Wrapper function for udev_device_get_property_value() that also checks for
 // valid but empty values.