Add missing (and remove superfluous) 'explicit' from constructors.

base/bind_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/callback.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
@@ skipping 88 matching lines @@
       : copies_(copies), assigns_(assigns) {
   }
 
   CopyCounter(const CopyCounter& other)
       : copies_(other.copies_),
         assigns_(other.assigns_) {
     (*copies_)++;
   }
 
   // Probing for copies from coercion.
-  CopyCounter(const DerivedCopyCounter& other)
+  explicit CopyCounter(const DerivedCopyCounter& other)
       : copies_(other.copies_),
         assigns_(other.assigns_) {
     (*copies_)++;
   }
 
   const CopyCounter& operator=(const CopyCounter& rhs) {
     copies_ = rhs.copies_;
     assigns_ = rhs.assigns_;
 
     if (assigns_) {
@@ skipping 639 matching lines @@
       Bind(&VoidPolymorphic1<CopyCounter>);
   forward_cb.Run(counter);
   EXPECT_GE(1, copies);
   EXPECT_EQ(0, assigns);
 
   copies = 0;
   assigns = 0;
   DerivedCopyCounter dervied(&copies, &assigns);
   Callback<void(CopyCounter)> coerce_cb =
       Bind(&VoidPolymorphic1<CopyCounter>);
-  coerce_cb.Run(dervied);
+  coerce_cb.Run(CopyCounter(dervied));
   EXPECT_GE(2, copies);
   EXPECT_EQ(0, assigns);
 }
 
 // Callback construction and assignment tests.
 //   - Construction from an InvokerStorageHolder should not cause ref/deref.
 //   - Assignment from other callback should only cause one ref
 //
 // TODO(ajwong): Is there actually a way to test this?
 
@@ skipping 25 matching lines @@
   base::Callback<void(int)> null_cb;
   ASSERT_TRUE(null_cb.is_null());
   EXPECT_DEATH(base::Bind(null_cb, 42), "");
 }
 
 #endif  // (!defined(NDEBUG) || defined(DCHECK_ALWAYS_ON)) &&
         //     GTEST_HAS_DEATH_TEST
 
 }  // namespace
 }  // namespace base