Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1882)

Unified Diff: Tools/Scripts/webkitpy/style/checkers/cpp_unittest.py

Issue 648223003: Remove a few residual references to the old OVERRIDE and FINAL macros. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Tools/Scripts/webkitpy/style/checkers/cpp.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Tools/Scripts/webkitpy/style/checkers/cpp_unittest.py
diff --git a/Tools/Scripts/webkitpy/style/checkers/cpp_unittest.py b/Tools/Scripts/webkitpy/style/checkers/cpp_unittest.py
index b1fe057cd2324b0a4605f3da18d888d8a3ea8611..42ffe18747624d20b1b70174cf1e3432a9e2c209 100644
--- a/Tools/Scripts/webkitpy/style/checkers/cpp_unittest.py
+++ b/Tools/Scripts/webkitpy/style/checkers/cpp_unittest.py
@@ -1602,18 +1602,9 @@ class CppStyleTest(CppStyleTestBase):
'int foo() const {',
'Place brace on its own line for function definitions. [whitespace/braces] [4]')
self.assert_multi_line_lint(
- 'int foo() const OVERRIDE {',
- 'Place brace on its own line for function definitions. [whitespace/braces] [4]')
- self.assert_multi_line_lint(
- 'int foo() OVERRIDE {',
- 'Place brace on its own line for function definitions. [whitespace/braces] [4]')
- self.assert_multi_line_lint(
'int foo() override {',
'Place brace on its own line for function definitions. [whitespace/braces] [4]')
self.assert_multi_line_lint(
- 'int foo() FINAL {',
- 'Place brace on its own line for function definitions. [whitespace/braces] [4]')
- self.assert_multi_line_lint(
'int foo() final {',
'Place brace on its own line for function definitions. [whitespace/braces] [4]')
self.assert_multi_line_lint(
@@ -1622,21 +1613,11 @@ class CppStyleTest(CppStyleTestBase):
'}\n',
'')
self.assert_multi_line_lint(
- 'int foo() OVERRIDE\n'
- '{\n'
- '}\n',
- '')
- self.assert_multi_line_lint(
'int foo() override\n'
'{\n'
'}\n',
'')
self.assert_multi_line_lint(
- 'int foo() FINAL\n'
- '{\n'
- '}\n',
- '')
- self.assert_multi_line_lint(
'int foo() final\n'
'{\n'
'}\n',
« no previous file with comments | « Tools/Scripts/webkitpy/style/checkers/cpp.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698