Remove bad use of CancelableCallback from BrowingDataAppCacheHelper

chrome/browser/browsing_data/browsing_data_appcache_helper.cc

Index: chrome/browser/browsing_data/browsing_data_appcache_helper.cc
diff --git a/chrome/browser/browsing_data/browsing_data_appcache_helper.cc b/chrome/browser/browsing_data/browsing_data_appcache_helper.cc
index b11e486353a4390f7066b90d78c7df38337888a6..a8e7241cb3ff15737c9e6d13e3ce9f2547a8acfe 100644
--- a/chrome/browser/browsing_data/browsing_data_appcache_helper.cc
+++ b/chrome/browser/browsing_data/browsing_data_appcache_helper.cc
@@ -10,6 +10,7 @@
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/storage_partition.h"
+#include "net/base/completion_callback.h"
 
 using content::BrowserContext;
 using content::BrowserThread;
@@ -35,11 +36,10 @@ void BrowsingDataAppCacheHelper::StartFetching(const base::Closure& callback) {
   }
 
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
-  appcache_info_callback_.Reset(
-      base::Bind(&BrowsingDataAppCacheHelper::OnFetchComplete,
-                 base::Unretained(this)));
+  net::CompletionCallback appcache_info_callback =
+      base::Bind(&BrowsingDataAppCacheHelper::OnFetchComplete, this);

[michaeln, 2015/01/13 21:46:27]

nit: its real common to put the base::Bind call inline in the param list (as on
line 34)

   appcache_service_->GetAllAppCacheInfo(info_collection_.get(),
-                                        appcache_info_callback_.callback());
+                                        appcache_info_callback);
 }
 
 void BrowsingDataAppCacheHelper::DeleteAppCacheGroup(