base::Bind fixes

chrome/browser/net/chrome_url_request_context.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/net/chrome_url_request_context.h"
 
+#include "base/bind.h"
 #include "base/compiler_specific.h"
 #include "base/message_loop.h"
 #include "base/message_loop_proxy.h"
 #include "base/synchronization/waitable_event.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/io_thread.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_io_data.h"
 #include "chrome/common/chrome_notification_types.h"
@@ skipping 137 matching lines @@
 
   // If we aren't running on the IO thread, we cannot call
   // GetURLRequestContext(). Instead we will post a task to the IO loop
   // and wait for it to complete.
 
   base::WaitableEvent completion(false, false);
   net::CookieStore* result = NULL;
 
   BrowserThread::PostTask(
       BrowserThread::IO, FROM_HERE,
-      NewRunnableMethod(this,
-          &ChromeURLRequestContextGetter::GetCookieStoreAsyncHelper,
+      base::Bind(&ChromeURLRequestContextGetter::GetCookieStoreAsyncHelper,
+          this,
           &completion,
           &result));

[wtc, 2011/11/17 08:16:21]

Nit: align all the function arguments of base::Bind().

 
   completion.Wait();
   DCHECK(result);
   return result;
 }
 
 scoped_refptr<base::MessageLoopProxy>
 ChromeURLRequestContextGetter::GetIOMessageLoopProxy() const {
   return BrowserThread::GetMessageLoopProxyForThread(BrowserThread::IO);
 }
@@ skipping 90 matching lines @@
 
   if (chrome::NOTIFICATION_PREF_CHANGED == type) {
     std::string* pref_name_in = content::Details<std::string>(details).ptr();
     PrefService* prefs = content::Source<PrefService>(source).ptr();
     DCHECK(pref_name_in && prefs);
     if (*pref_name_in == prefs::kAcceptLanguages) {
       std::string accept_language =
           prefs->GetString(prefs::kAcceptLanguages);
       BrowserThread::PostTask(
           BrowserThread::IO, FROM_HERE,
-          NewRunnableMethod(
+          base::Bind(
+              &ChromeURLRequestContextGetter::OnAcceptLanguageChange,
               this,
-              &ChromeURLRequestContextGetter::OnAcceptLanguageChange,
               accept_language));
     } else if (*pref_name_in == prefs::kDefaultCharset) {
       std::string default_charset =
           prefs->GetString(prefs::kDefaultCharset);
       BrowserThread::PostTask(
           BrowserThread::IO, FROM_HERE,
-          NewRunnableMethod(
+          base::Bind(
+              &ChromeURLRequestContextGetter::OnDefaultCharsetChange,
               this,
-              &ChromeURLRequestContextGetter::OnDefaultCharsetChange,
               default_charset));
     } else if (*pref_name_in == prefs::kClearSiteDataOnExit) {
       bool clear_site_data =
           prefs->GetBoolean(prefs::kClearSiteDataOnExit);
       BrowserThread::PostTask(
           BrowserThread::IO, FROM_HERE,
-          NewRunnableMethod(
+          base::Bind(
+              &ChromeURLRequestContextGetter::OnClearSiteDataOnExitChange,
               this,
-              &ChromeURLRequestContextGetter::OnClearSiteDataOnExitChange,
               clear_site_data));
     }
   } else {
     NOTREACHED();
   }
 }
 
 void ChromeURLRequestContextGetter::RegisterPrefsObserver(Profile* profile) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 
@@ skipping 78 matching lines @@
       net::HttpUtil::GenerateAcceptLanguageHeader(accept_language));
 }
 
 void ChromeURLRequestContext::OnDefaultCharsetChange(
     const std::string& default_charset) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
   set_referrer_charset(default_charset);
   set_accept_charset(
       net::HttpUtil::GenerateAcceptCharsetHeader(default_charset));
 }