webkit: Use base::MessageLoop.

webkit/database/database_connections_unittest.cc

 // Copyright (c) 2012 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 "base/bind.h"
 #include "base/message_loop.h"
 #include "base/threading/thread.h"
 #include "base/utf_string_conversions.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "webkit/database/database_connections.h"
@@ skipping 82 matching lines @@
   EXPECT_FALSE(connections.AddConnection(kOriginId, kName));
   EXPECT_FALSE(connections.RemoveConnection(kOriginId, kName));
   EXPECT_FALSE(connections.RemoveConnection(kOriginId, kName));
   EXPECT_TRUE(connections.RemoveConnection(kOriginId, kName));
 }
 
 TEST(DatabaseConnectionsTest, DatabaseConnectionsWrapperTest) {
   const base::string16 kOriginId(ASCIIToUTF16("origin_id"));
   const base::string16 kName(ASCIIToUTF16("database_name"));
 
-  MessageLoop message_loop;
-  scoped_refptr<DatabaseConnectionsWrapper> obj(
-      new DatabaseConnectionsWrapper);
+  base::MessageLoop message_loop;
+  scoped_refptr<DatabaseConnectionsWrapper> obj(new DatabaseConnectionsWrapper);
   EXPECT_FALSE(obj->HasOpenConnections());
   obj->AddOpenConnection(kOriginId, kName);
   EXPECT_TRUE(obj->HasOpenConnections());
   obj->AddOpenConnection(kOriginId, kName);
   EXPECT_TRUE(obj->HasOpenConnections());
   obj->RemoveOpenConnection(kOriginId, kName);
   EXPECT_TRUE(obj->HasOpenConnections());
   obj->RemoveOpenConnection(kOriginId, kName);
   EXPECT_FALSE(obj->HasOpenConnections());
   obj->WaitForAllDatabasesToClose();  // should return immediately
 
   // Test WaitForAllDatabasesToClose with the last connection
   // being removed on the current thread.
   obj->AddOpenConnection(kOriginId, kName);
   bool did_task_execute = false;
-  MessageLoop::current()->PostTask(
+  base::MessageLoop::current()->PostTask(
       FROM_HERE,
-      base::Bind(&RemoveConnectionTask, kOriginId, kName, obj,
-                 &did_task_execute));
+      base::Bind(
+          &RemoveConnectionTask, kOriginId, kName, obj, &did_task_execute));
   obj->WaitForAllDatabasesToClose();  // should return after the task executes
   EXPECT_TRUE(did_task_execute);
   EXPECT_FALSE(obj->HasOpenConnections());
 
   // Test WaitForAllDatabasesToClose with the last connection
   // being removed on another thread.
   obj->AddOpenConnection(kOriginId, kName);
   base::Thread thread("WrapperTestThread");
   thread.Start();
   did_task_execute = false;
-  MessageLoop::current()->PostTask(
+  base::MessageLoop::current()->PostTask(
       FROM_HERE,
-      base::Bind(&ScheduleRemoveConnectionTask, &thread, kOriginId, kName, obj,
+      base::Bind(&ScheduleRemoveConnectionTask,

[brettw, 2013/05/06 17:43:33]

I think the old style was fine.

[xhwang, 2013/05/07 00:11:07]

Done.

+                 &thread,
+                 kOriginId,
+                 kName,
+                 obj,
                  &did_task_execute));
   obj->WaitForAllDatabasesToClose();  // should return after the task executes
   EXPECT_TRUE(did_task_execute);
   EXPECT_FALSE(obj->HasOpenConnections());
 }
 
 }  // namespace webkit_database