Adding Presubmit error when OVERRIDE and FINAL is not used as C++11 standard

PRESUBMIT_test.py

 #!/usr/bin/env python
 # 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.
 
 import glob
 import json
 import os
 import re
 import subprocess
@@ skipping 395 matching lines @@
     self.assertTrue(':1 OS_WINDOWS' in errors[0])
     self.assertTrue('(did you mean OS_WIN?)' in errors[0])
 
   def testValidOSMacroNames(self):
     lines = ['#if defined(%s)' % m for m in PRESUBMIT._VALID_OS_MACROS]
     errors = PRESUBMIT._CheckForInvalidOSMacrosInFile(
         MockInputApi(), MockFile('some/path/foo_platform.cc', lines))
     self.assertEqual(0, len(errors))
 
 
+class InvalidOverideAndFinalTest(unittest.TestCase):
+  def testValidOverrideConstructs(self):
+    mock_input_api = MockInputApi()
+    lines = ['foo1() override;',
+             'foo2() final;',
+             '#DEFINE OVERRIDE_METHOD_OVERLOAD',
+             '#DEFINE FINAL_METHOD',

[Mike West, 2014/10/15 13:10:03]

Nit: For completeness, please add test cases for "SOMETHINGSOMETHING_OVERRIDE"
and "SOMETHING_OVERRIDE_SOMETHING".

[MRV, 2014/10/15 13:34:46]

Done.

+             '#endif  // FINAL_METHOD',
+             '#endif  // OVERRIDE_METHOD_OVERLOAD']
+    mock_file_h = MockFile('something.h', lines)
+    mock_input_api.files = [mock_file_h]
+    errors = PRESUBMIT._CheckForOverrideAndFinalRules(mock_input_api,
+                                                      MockOutputApi())
+    self.assertEqual(0, len(errors))
+
+  def testInvalidOverrideConstructsInHeaders(self):
+    mock_input_api = MockInputApi()
+    lines = ['foo1() OVERRIDE;']
+    mock_file_h = MockFile('something.h', lines)
+    mock_input_api.files = [mock_file_h]
+    errors = PRESUBMIT._CheckForOverrideAndFinalRules(mock_input_api,
+                                                      MockOutputApi())
+    self.assertEqual(1, len(errors))
+
+  def testInvalidOverrideConstructsInCpp(self):
+    mock_input_api = MockInputApi()
+    lines = ['foo2() FINAL;']
+    mock_file_cpp= MockFile('something.cpp', lines)
+    mock_input_api.files = [mock_file_cpp]
+    errors = PRESUBMIT._CheckForOverrideAndFinalRules(mock_input_api,
+                                                      MockOutputApi())
+    self.assertEqual(1, len(errors))
+
+  def testInvalidOverrideConstructsInCc(self):
+    mock_input_api = MockInputApi()
+    lines = ['foo3() override FINAL;']
+    mock_file_cc = MockFile('something.cc', lines)
+    mock_input_api.files = [mock_file_cc]
+    errors = PRESUBMIT._CheckForOverrideAndFinalRules(mock_input_api,
+                                                      MockOutputApi())
+    self.assertEqual(1, len(errors))
+
+  def testInvalidOverrideConstructsInMm(self):
+    mock_input_api = MockInputApi()
+    lines = ['foo4() OVERRIDE final;']
+    mock_file_mm = MockFile('something.mm', lines)
+    mock_input_api.files = [mock_file_mm]
+    errors = PRESUBMIT._CheckForOverrideAndFinalRules(mock_input_api,
+                                                      MockOutputApi())
+    self.assertEqual(1, len(errors))
+
+
 class InvalidIfDefinedMacroNamesTest(unittest.TestCase):
   def testInvalidIfDefinedMacroNames(self):
     lines = ['#if defined(TARGET_IPHONE_SIMULATOR)',
              '#if !defined(TARGET_IPHONE_SIMULATOR)',
              '#elif defined(TARGET_IPHONE_SIMULATOR)',
              '#ifdef TARGET_IPHONE_SIMULATOR',
              ' # ifdef TARGET_IPHONE_SIMULATOR',
              '# if defined(VALID) || defined(TARGET_IPHONE_SIMULATOR)',
              '# else  // defined(TARGET_IPHONE_SIMULATOR)',
              '#endif  // defined(TARGET_IPHONE_SIMULATOR)',]
@@ skipping 352 matching lines @@
     }
     for master, bots in bots.iteritems():
       for bot in bots:
         self.assertEqual(master, PRESUBMIT.GetTryServerMasterForBot(bot),
                          'bot=%s: expected %s, computed %s' % (
             bot, master, PRESUBMIT.GetTryServerMasterForBot(bot)))
 
 
 if __name__ == '__main__':
   unittest.main()