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

chromeos/dbus/experimental_bluetooth_device_client.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 "chromeos/dbus/experimental_bluetooth_device_client.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/stl_util.h"
 #include "chromeos/dbus/fake_bluetooth_device_client.h"
@@ skipping 68 matching lines @@
   virtual void RemoveObserver(
       ExperimentalBluetoothDeviceClient::Observer* observer) OVERRIDE {
     DCHECK(observer);
     observers_.RemoveObserver(observer);
   }
 
   // dbus::ObjectManager::Interface override.
   virtual dbus::PropertySet* CreateProperties(
       dbus::ObjectProxy* object_proxy,
       const dbus::ObjectPath& object_path,
-      const std::string& interface_name) {
+      const std::string& interface_name) OVERRIDE {
     Properties* properties = new Properties(
-        object_proxy, interface_name,
+        object_proxy,
+        interface_name,
         base::Bind(&ExperimentalBluetoothDeviceClientImpl::OnPropertyChanged,
                    weak_ptr_factory_.GetWeakPtr(),
                    object_path));
     return static_cast<dbus::PropertySet*>(properties);
   }
 
   // ExperimentalBluetoothDeviceClient override.
   virtual std::vector<dbus::ObjectPath> GetDevicesForAdapter(
       const dbus::ObjectPath& adapter_path) OVERRIDE {
     std::vector<dbus::ObjectPath> object_paths, device_paths;
@@ skipping 169 matching lines @@
         dbus::ObjectProxy::TIMEOUT_USE_DEFAULT,
         base::Bind(&ExperimentalBluetoothDeviceClientImpl::OnSuccess,
                    weak_ptr_factory_.GetWeakPtr(), callback),
         base::Bind(&ExperimentalBluetoothDeviceClientImpl::OnError,
                    weak_ptr_factory_.GetWeakPtr(), error_callback));
   }
 
  private:
   // Called by dbus::ObjectManager when an object with the device interface
   // is created. Informs observers.
-  void ObjectAdded(const dbus::ObjectPath& object_path,
-                   const std::string& interface_name) OVERRIDE {
-    FOR_EACH_OBSERVER(ExperimentalBluetoothDeviceClient::Observer, observers_,
+  virtual void ObjectAdded(const dbus::ObjectPath& object_path,
+                           const std::string& interface_name) OVERRIDE {
+    FOR_EACH_OBSERVER(ExperimentalBluetoothDeviceClient::Observer,
+                      observers_,
                       DeviceAdded(object_path));
   }
 
   // Called by dbus::ObjectManager when an object with the device interface
   // is removed. Informs observers.
-  void ObjectRemoved(const dbus::ObjectPath& object_path,
-                     const std::string& interface_name) OVERRIDE {
-    FOR_EACH_OBSERVER(ExperimentalBluetoothDeviceClient::Observer, observers_,
+  virtual void ObjectRemoved(const dbus::ObjectPath& object_path,
+                             const std::string& interface_name) OVERRIDE {
+    FOR_EACH_OBSERVER(ExperimentalBluetoothDeviceClient::Observer,
+                      observers_,
                       DeviceRemoved(object_path));
   }
 
   // Called by BluetoothPropertySet when a property value is changed,
   // either by result of a signal or response to a GetAll() or Get()
   // call. Informs observers.
   void OnPropertyChanged(const dbus::ObjectPath& object_path,
                          const std::string& property_name) {
     FOR_EACH_OBSERVER(ExperimentalBluetoothDeviceClient::Observer, observers_,
                       DevicePropertyChanged(object_path, property_name));
@@ skipping 47 matching lines @@
 ExperimentalBluetoothDeviceClient* ExperimentalBluetoothDeviceClient::Create(
     DBusClientImplementationType type,
     dbus::Bus* bus) {
   if (type == REAL_DBUS_CLIENT_IMPLEMENTATION)
     return new ExperimentalBluetoothDeviceClientImpl(bus);
   DCHECK_EQ(STUB_DBUS_CLIENT_IMPLEMENTATION, type);
   return new FakeBluetoothDeviceClient();
 }
 
 }  // namespace chromeos