Convert implicit scoped_refptr constructor calls to explicit ones, part 1

base/message_loop_unittest.cc

 // Copyright (c) 2010 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 <vector>
 
 #include "base/eintr_wrapper.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "base/platform_thread.h"
@@ skipping 77 matching lines @@
  private:
   friend class base::RefCounted<QuitMsgLoop>;
 
   ~QuitMsgLoop() {}
 };
 
 void RunTest_PostTask(MessageLoop::Type message_loop_type) {
   MessageLoop loop(message_loop_type);
 
   // Add tests to message loop
-  scoped_refptr<Foo> foo = new Foo();
+  scoped_refptr<Foo> foo(new Foo());
   std::string a("a"), b("b"), c("c"), d("d");
   MessageLoop::current()->PostTask(FROM_HERE, NewRunnableMethod(
       foo.get(), &Foo::Test0));
   MessageLoop::current()->PostTask(FROM_HERE, NewRunnableMethod(
     foo.get(), &Foo::Test1ConstRef, a));
   MessageLoop::current()->PostTask(FROM_HERE, NewRunnableMethod(
       foo.get(), &Foo::Test1Ptr, &b));
   MessageLoop::current()->PostTask(FROM_HERE, NewRunnableMethod(
       foo.get(), &Foo::Test1Int, 100));
   MessageLoop::current()->PostTask(FROM_HERE, NewRunnableMethod(
       foo.get(), &Foo::Test2Ptr, &a, &c));
   MessageLoop::current()->PostTask(FROM_HERE, NewRunnableMethod(
     foo.get(), &Foo::Test2Mixed, a, &d));
 
   // After all tests, post a message that will shut down the message loop
-  scoped_refptr<QuitMsgLoop> quit = new QuitMsgLoop();
+  scoped_refptr<QuitMsgLoop> quit(new QuitMsgLoop());
   MessageLoop::current()->PostTask(FROM_HERE, NewRunnableMethod(
       quit.get(), &QuitMsgLoop::QuitNow));
 
   // Now kick things off
   MessageLoop::current()->Run();
 
   EXPECT_EQ(foo->test_count(), 105);
   EXPECT_EQ(foo->result(), "abacad");
 }
 
 void RunTest_PostTask_SEH(MessageLoop::Type message_loop_type) {
   MessageLoop loop(message_loop_type);
 
   // Add tests to message loop
-  scoped_refptr<Foo> foo = new Foo();
+  scoped_refptr<Foo> foo(new Foo());
   std::string a("a"), b("b"), c("c"), d("d");
   MessageLoop::current()->PostTask(FROM_HERE, NewRunnableMethod(
       foo.get(), &Foo::Test0));
   MessageLoop::current()->PostTask(FROM_HERE, NewRunnableMethod(
       foo.get(), &Foo::Test1ConstRef, a));
   MessageLoop::current()->PostTask(FROM_HERE, NewRunnableMethod(
       foo.get(), &Foo::Test1Ptr, &b));
   MessageLoop::current()->PostTask(FROM_HERE, NewRunnableMethod(
       foo.get(), &Foo::Test1Int, 100));
   MessageLoop::current()->PostTask(FROM_HERE, NewRunnableMethod(
       foo.get(), &Foo::Test2Ptr, &a, &c));
   MessageLoop::current()->PostTask(FROM_HERE, NewRunnableMethod(
       foo.get(), &Foo::Test2Mixed, a, &d));
 
   // After all tests, post a message that will shut down the message loop
-  scoped_refptr<QuitMsgLoop> quit = new QuitMsgLoop();
+  scoped_refptr<QuitMsgLoop> quit(new QuitMsgLoop());
   MessageLoop::current()->PostTask(FROM_HERE, NewRunnableMethod(
       quit.get(), &QuitMsgLoop::QuitNow));
 
   // Now kick things off with the SEH block active.
   MessageLoop::current()->set_exception_restoration(true);
   MessageLoop::current()->Run();
   MessageLoop::current()->set_exception_restoration(false);
 
   EXPECT_EQ(foo->test_count(), 105);
   EXPECT_EQ(foo->result(), "abacad");
@@ skipping 1484 matching lines @@
   }
   if (HANDLE_EINTR(close(pipefds[0])) < 0)
     PLOG(ERROR) << "close";
   if (HANDLE_EINTR(close(pipefds[1])) < 0)
     PLOG(ERROR) << "close";
 }
 
 }  // namespace
 
 #endif  // defined(OS_POSIX)