base::Bind: Cleanup almost-unused OldCompletionCallbacks.

chrome/browser/browsing_data_remover.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/browsing_data_remover.h"
 
 #include <map>
 #include <set>
 
 #include "base/bind.h"
@@ skipping 450 matching lines @@
     switch (next_cache_state_) {
       case STATE_CREATE_MAIN:
       case STATE_CREATE_MEDIA: {
         // Get a pointer to the cache.
         net::URLRequestContextGetter* getter =
             (next_cache_state_ == STATE_CREATE_MAIN) ?
                 main_context_getter_ : media_context_getter_;
         net::HttpTransactionFactory* factory =
             getter->GetURLRequestContext()->http_transaction_factory();
 
         rv = factory->GetCache()->GetBackend(&cache_, &cache_callback_);

[dpapad, 2011/12/15 17:27:41]

Could this also be replaced?

[James Hawkins, 2011/12/15 17:54:42]

Not in this CL.

         next_cache_state_ = (next_cache_state_ == STATE_CREATE_MAIN) ?
                                 STATE_DELETE_MAIN : STATE_DELETE_MEDIA;
         break;
       }
       case STATE_DELETE_MAIN:
       case STATE_DELETE_MEDIA: {
         // |cache_| can be null if it cannot be initialized.
         if (cache_) {
           if (delete_begin_.is_null()) {
-            rv = cache_->DoomAllEntries(&cache_callback_);
+            rv = cache_->DoomAllEntries(
+                base::Bind(&BrowsingDataRemover::DoClearCache,
+                           base::Unretained(this)));
           } else {
-            rv = cache_->DoomEntriesBetween(delete_begin_, delete_end_,
-                                            &cache_callback_);
+            rv = cache_->DoomEntriesBetween(
+                delete_begin_, delete_end_,
+                base::Bind(&BrowsingDataRemover::DoClearCache,
+                           base::Unretained(this)));
           }
           cache_ = NULL;
         }
         next_cache_state_ = (next_cache_state_ == STATE_DELETE_MAIN) ?
                                 STATE_CREATE_MEDIA : STATE_DONE;
         break;
       }
       case STATE_DONE: {
         cache_ = NULL;
 
@@ skipping 123 matching lines @@
       GetURLRequestContext()->cookie_store()->GetCookieMonster();
   if (cookie_monster) {
       cookie_monster->DeleteAllCreatedBetweenAsync(
           delete_begin_, delete_end_,
           base::Bind(&BrowsingDataRemover::OnClearedCookies,
                      base::Unretained(this)));
   } else {
     OnClearedCookies(0);
   }
 }