libchrome: Several upstreamable fixes from libchrome

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 <memory>
 #include <utility>
 #include <vector>
 
@@ skipping 45 matching lines @@
 class HasRefPrivateDtor : public HasRef {
  private:
   ~HasRefPrivateDtor() {}
 };
 
 static const int kParentValue = 1;
 static const int kChildValue = 2;
 
 class Parent {
  public:
+  virtual ~Parent() = default;
   void AddRef() const {}
   void Release() const {}
   virtual void VirtualSet() { value = kParentValue; }
   void NonVirtualSet() { value = kParentValue; }
   int value;
 };
 
 class Child : public Parent {
  public:
+  ~Child() override = default;
   void VirtualSet() override { value = kChildValue; }
   void NonVirtualSet() { value = kChildValue; }
 };
 
 class NoRefParent {
  public:
+  virtual ~NoRefParent() = default;
   virtual void VirtualSet() { value = kParentValue; }
   void NonVirtualSet() { value = kParentValue; }
   int value;
 };
 
 class NoRefChild : public NoRefParent {
+ public:
+  ~NoRefChild() override = default;
   void VirtualSet() override { value = kChildValue; }
   void NonVirtualSet() { value = kChildValue; }
 };
 
 // Used for probing the number of copies and moves that occur if a type must be
 // coerced during argument forwarding in the Run() methods.
 struct DerivedCopyMoveCounter {
   DerivedCopyMoveCounter(int* copies,
                          int* assigns,
                          int* move_constructs,
@@ skipping 997 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