Warn on missing OVERRIDE/virtual everywhere, not just in header files.

sync/syncable/parent_child_index_unittest.cc

 // Copyright (c) 2013 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 "sync/syncable/parent_child_index.h"
 
 #include <list>
 
 #include "base/stl_util.h"
 #include "base/strings/string_number_conversions.h"
 #include "sync/syncable/entry_kernel.h"
 #include "sync/syncable/syncable_util.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace syncer {
 namespace syncable {
 
 namespace {
 
 static const std::string kCacheGuid = "8HhNIHlEOCGQbIAALr9QEg==";
 
 class ParentChildIndexTest : public testing::Test {
  public:
-  void TearDown() {
+  virtual void TearDown() {
     // To make memory management easier, we take ownership of all EntryKernels
     // returned by our factory methods and delete them here.
     STLDeleteElements(&owned_entry_kernels_);
   }
 
   // Unfortunately, we can't use the regular Entry factory methods, because the
   // ParentChildIndex deals in EntryKernels.
 
   static syncable::Id GetBookmarkRootId() {
     return syncable::Id::CreateFromServerId("bookmark_folder");
@@ skipping 300 matching lines @@
   it++;
   EXPECT_EQ(*it, u1);
   it++;
   EXPECT_TRUE(it == children->end());
 }
 
 }  // namespace
 }  // namespace syncable
 }  // namespace syncer