Cleanup: Remove an unneeded variable from ScopedMtpDeviceMapEntry.

Cleanup: Remove an unneeded variable from ScopedMtpDeviceMapEntry.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=166366

[kmadhusu, 2012-11-02 03:47:47]

lgtm

[commit-bot: I haz the power, 2012-11-02 22:44:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/thestig@chromium.org/11275121/1

[commit-bot: I haz the power, 2012-11-02 22:44:31]

Failed to apply patch for
chrome/browser/media_gallery/media_file_system_registry.cc:
While running patch -p0 --forward --force --no-backup-if-mismatch;
  patching file chrome/browser/media_gallery/media_file_system_registry.cc
  Hunk #1 FAILED at 138.
  Hunk #2 FAILED at 165.
  2 out of 2 hunks FAILED -- saving rejects to file
chrome/browser/media_gallery/media_file_system_registry.cc.rej

Patch:       chrome/browser/media_gallery/media_file_system_registry.cc
Index: chrome/browser/media_gallery/media_file_system_registry.cc
===================================================================
--- chrome/browser/media_gallery/media_file_system_registry.cc	(revision 165016)
+++ chrome/browser/media_gallery/media_file_system_registry.cc	(working copy)
@@ -138,14 +138,13 @@
   ScopedMtpDeviceMapEntry(const FilePath::StringType& device_location,
                           const base::Closure& no_references_callback)
       : device_location_(device_location),
-        delegate_(new MtpDeviceDelegateImpl(device_location)),
         no_references_callback_(no_references_callback) {
-    DCHECK(delegate_);
     BrowserThread::PostTask(
         BrowserThread::IO, FROM_HERE,
         Bind(&MtpDeviceMapService::AddDelegate,
              base::Unretained(MtpDeviceMapService::GetInstance()),
-             device_location_, make_scoped_refptr(delegate_)));
+             device_location_,
+             make_scoped_refptr(new MtpDeviceDelegateImpl(device_location))));
   }
 
  private:
@@ -165,12 +164,6 @@
   // Store the mtp or ptp device location.
   const FilePath::StringType device_location_;
 
-  // Store a raw pointer of MtpDeviceDelegateImpl object.
-  // MtpDeviceDelegateImpl is ref-counted and owned by MtpDeviceMapService.
-  // This class tells MtpDeviceMapService to dispose of it when the last
-  // reference to |this| goes away.
-  MtpDeviceDelegateImpl* delegate_;
-
   // A callback to call when the last reference of this object goes away.
   base::Closure no_references_callback_;

[commit-bot: I haz the power, 2012-11-07 01:08:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/thestig@chromium.org/11275121/4

[commit-bot: I haz the power, 2012-11-07 06:58:05]

Change committed as 166366