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

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

Issue 15030003: Updated style checker for wtf/file.h includes. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Updated webkitpy tests Created 7 years, 7 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 20a0b79673623c51d2d03eed239eb28b0456045b..c0f2da3b0e18366d35d6c16358f7b878c145e19d 100644
--- a/Tools/Scripts/webkitpy/style/checkers/cpp_unittest.py
+++ b/Tools/Scripts/webkitpy/style/checkers/cpp_unittest.py
@@ -2831,14 +2831,14 @@ class OrderOfIncludesTest(CppStyleTestBase):
'#include "foo.h"\n'
'\n'
'#include <wtf/Assertions.h>\n',
- '')
+ 'wtf includes should be "wtf/file.h" instead of <wtf/file.h>.'
+ ' [build/include] [4]')
self.assert_language_rules_check('foo.cpp',
'#include "config.h"\n'
'#include "foo.h"\n'
'\n'
'#include "wtf/Assertions.h"\n',
- 'wtf includes should be <wtf/file.h> instead of "wtf/file.h".'
- ' [build/include] [4]')
+ '')
def test_check_cc_includes(self):
self.assert_language_rules_check('bar/chromium/foo.cpp',
« 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