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

chromeos/display/output_configurator_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 "chromeos/display/output_configurator.h"
 
 #include <cstdarg>
 #include <map>
 #include <string>
 #include <vector>
@@ skipping 137 matching lines @@
       *interlaced = it->second.interlaced;
     return true;
   }
   virtual void ConfigureCrtc(OutputConfigurator::CrtcConfig* config) OVERRIDE {
     AppendAction(GetCrtcAction(config->crtc, config->x, config->y, config->mode,
                                config->output));
   }
   virtual void CreateFrameBuffer(
       int width,
       int height,
-      const std::vector<OutputConfigurator::CrtcConfig>& configs) {
-    AppendAction(GetFramebufferAction(width, height,
-        configs.size() >= 1 ? configs[0].crtc : 0,
-        configs.size() >= 2 ? configs[1].crtc : 0));
+      const std::vector<OutputConfigurator::CrtcConfig>& configs) OVERRIDE {
+    AppendAction(
+        GetFramebufferAction(width,
+                             height,
+                             configs.size() >= 1 ? configs[0].crtc : 0,
+                             configs.size() >= 2 ? configs[1].crtc : 0));
   }
   virtual void ConfigureCTM(
       int touch_device_id,
       const OutputConfigurator::CoordinateTransformation& ctm) OVERRIDE {
     AppendAction(GetCTMAction(touch_device_id, ctm));
   }
   virtual void SendProjectingStateToPowerManager(bool projecting) OVERRIDE {
     AppendAction(projecting ? kProjectingOn : kProjectingOff);
   }
 
@@ skipping 22 matching lines @@
   std::vector<OutputConfigurator::OutputSnapshot> outputs_;
 
   std::string actions_;
 
   DISALLOW_COPY_AND_ASSIGN(TestDelegate);
 };
 
 class TestStateController : public OutputConfigurator::StateController {
  public:
   TestStateController() : state_(STATE_DUAL_EXTENDED) {}
-  ~TestStateController() {}
+  virtual ~TestStateController() {}
 
   void set_state(OutputState state) { state_ = state; }
 
   // OutputConfigurator::StateController overrides:
   virtual OutputState GetStateForOutputs(
       const std::vector<OutputInfo>& outputs) const OVERRIDE { return state_; }
 
  private:
   OutputState state_;
 
@@ skipping 371 matching lines @@
   UpdateOutputs(2);
   state_controller_.set_state(STATE_DUAL_EXTENDED);
   EXPECT_TRUE(test_api_.SendOutputChangeEvents(true));
   EXPECT_FALSE(configurator_.SetDisplayMode(STATE_HEADLESS));
   EXPECT_FALSE(configurator_.SetDisplayMode(STATE_SINGLE));
   EXPECT_TRUE(configurator_.SetDisplayMode(STATE_DUAL_MIRROR));
   EXPECT_TRUE(configurator_.SetDisplayMode(STATE_DUAL_EXTENDED));
 }
 
 }  // namespace chromeos