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

chrome/browser/chrome_security_exploit_browsertest.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 "base/command_line.h"
 #include "base/stringprintf.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_commands.h"
 #include "chrome/browser/ui/singleton_tabs.h"
@@ skipping 12 matching lines @@
 // can send arbitrary IPC messages and confuse browser process internal state,
 // leading to security bugs. We are trying to verify that the browser doesn't
 // perform any dangerous operations in such cases.
 // This is similar to the security_exploit_browsertest.cc tests, but also
 // includes chrome/ layer concepts such as extensions.
 class ChromeSecurityExploitBrowserTest : public InProcessBrowserTest {
  public:
   ChromeSecurityExploitBrowserTest() {}
   virtual ~ChromeSecurityExploitBrowserTest() {}
 
-  virtual void SetUpCommandLine(CommandLine* command_line) {
+  virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
     ASSERT_TRUE(test_server()->Start());
     net::TestServer https_server(
         net::TestServer::TYPE_HTTPS,
         net::TestServer::kLocalhost,
         base::FilePath(FILE_PATH_LITERAL("chrome/test/data")));
     ASSERT_TRUE(https_server.Start());
 
     // Add a host resolver rule to map all outgoing requests to the test server.
     // This allows us to use "real" hostnames in URLs, which we can use to
     // create arbitrary SiteInstances.
@@ skipping 19 matching lines @@
 
   content::DOMMessageQueue msg_queue;
 
   ui_test_utils::NavigateToURL(browser(), foo);
 
   std::string status;
   std::string expected_status("0");
   EXPECT_TRUE(msg_queue.WaitForMessage(&status));
   EXPECT_STREQ(status.c_str(), expected_status.c_str());
 }