base::Bind: Low-hanging fruit conversions of NewRunnableFunction.

chrome_frame/crash_reporting/nt_loader_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 "chrome_frame/crash_reporting/nt_loader.h"
 
 #include <tlhelp32.h>
 #include <winnt.h>
 
 #include "base/at_exit.h"
+#include "base/bind.h"
+#include "base/bind_helpers.h"
 #include "base/environment.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop.h"
 #include "base/string_util.h"
 #include "base/sys_info.h"
 #include "base/threading/thread.h"
 #include "base/utf_string_conversions.h"
 #include "base/win/scoped_handle.h"
 #include "chrome_frame/crash_reporting/crash_dll.h"
 #include "gtest/gtest.h"
@@ skipping 53 matching lines @@
     EXPECT_TRUE(OwnsCriticalSection(&cs));
   }
 
   // Should no longer own it.
   EXPECT_FALSE(OwnsCriticalSection(&cs));
 
   // Make another thread grab it.
   base::Thread other("Other threads");
   ASSERT_TRUE(other.Start());
   other.message_loop()->PostTask(
-      FROM_HERE, NewRunnableFunction(::EnterCriticalSection, &cs));
+      FROM_HERE, base::Bind(::EnterCriticalSection, &cs));
 
   base::win::ScopedHandle event(::CreateEvent(NULL, FALSE, FALSE, NULL));
   other.message_loop()->PostTask(
-      FROM_HERE, NewRunnableFunction(::SetEvent, event.Get()));
+      FROM_HERE, base::IgnoreReturn<BOOL>(base::Bind(::SetEvent, event.Get())));
 
   ASSERT_EQ(WAIT_OBJECT_0, ::WaitForSingleObject(event.Get(), INFINITE));
 
   // We still shouldn't own it - the other thread does.
   EXPECT_FALSE(OwnsCriticalSection(&cs));
   // And we shouldn't be able to enter it.
   EXPECT_EQ(0, ::TryEnterCriticalSection(&cs));
 
   // Make the other thread release it.
   other.message_loop()->PostTask(
-      FROM_HERE, NewRunnableFunction(::LeaveCriticalSection, &cs));
+      FROM_HERE, base::Bind(::LeaveCriticalSection, &cs));
 
   other.Stop();
 
   ::DeleteCriticalSection(&cs);
 }
 
 TEST(NtLoader, GetLoaderLock) {
   CRITICAL_SECTION* loader_lock = GetLoaderLock();
 
   AssertIsCriticalSection(loader_lock);
@@ skipping 201 matching lines @@
 
   exceptions_handled = 0;
   set_exception_function(OnCrashDuringUnloadLibrary);
 
   // We should crash during unload.
   if (module != NULL)
     ::FreeLibrary(module);
 
   EXPECT_EQ(1, exceptions_handled);
 }