base::Bind usage

webkit/appcache/appcache_service_unittest.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 <string>
 
+#include "base/bind.h"
 #include "base/message_loop.h"
 #include "base/pickle.h"
 #include "net/base/completion_callback.h"
 #include "net/base/io_buffer.h"
 #include "net/http/http_response_headers.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "webkit/appcache/appcache_response.h"
 #include "webkit/appcache/appcache_service.h"
 #include "webkit/appcache/mock_appcache_storage.h"
 
@@ skipping 42 matching lines @@
     }
     DCHECK(buf_len >= data_size_);
     memcpy(buf->data(), data_, data_size_);
     ScheduleUserCallback(data_size_);
     data_size_ = 0;
   }
 
  private:
   void ScheduleUserCallback(int result) {
     MessageLoop::current()->PostTask(FROM_HERE,
-        method_factory_.NewRunnableMethod(
-            &MockResponseReader::InvokeUserOldCompletionCallback, result));
+        base::Bind(&MockResponseReader::InvokeUserOldCompletionCallback,
+            weak_factory_.GetWeakPtr(), result));

[James Hawkins, 2011/10/27 03:30:32]

Nit: For both function calls, line up parameters on the same column (per-call).

[michaeln, 2011/10/27 19:20:02]

Done.

   }
 
   scoped_ptr<net::HttpResponseInfo> info_;
   int info_size_;
   const char* data_;
   int data_size_;
 };
 
 }  // namespace
 
@@ skipping 223 matching lines @@
   MessageLoop::current()->RunAllPending();
   EXPECT_EQ(0, CountPendingHelpers());
   EXPECT_FALSE(IsGroupStored(kManifestUrl));
   ResetStorage();
 
   service_.reset();  // Clean up.
   MessageLoop::current()->RunAllPending();
 }
 
 }  // namespace appcache