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

Side by Side Diff: third_party/WebKit/Tools/Scripts/webkitpy/style/checkers/cpp.py

Issue 2353003004: Move ViewHostMsg_CreateWindow to mojom (Closed)
Patch Set: . Created 4 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 unified diff | Download patch
« no previous file with comments | « third_party/WebKit/PRESUBMIT.py ('k') | third_party/WebKit/public/BUILD.gn » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # -*- coding: utf-8 -*- 1 # -*- coding: utf-8 -*-
2 # 2 #
3 # Copyright (C) 2009, 2010, 2012 Google Inc. All rights reserved. 3 # Copyright (C) 2009, 2010, 2012 Google Inc. All rights reserved.
4 # Copyright (C) 2009 Torch Mobile Inc. 4 # Copyright (C) 2009 Torch Mobile Inc.
5 # Copyright (C) 2009 Apple Inc. All rights reserved. 5 # Copyright (C) 2009 Apple Inc. All rights reserved.
6 # Copyright (C) 2010 Chris Jerdonek (cjerdonek@webkit.org) 6 # Copyright (C) 2010 Chris Jerdonek (cjerdonek@webkit.org)
7 # 7 #
8 # Redistribution and use in source and binary forms, with or without 8 # Redistribution and use in source and binary forms, with or without
9 # modification, are permitted provided that the following conditions are 9 # modification, are permitted provided that the following conditions are
10 # met: 10 # met:
(...skipping 3492 matching lines...) Expand 10 before | Expand all | Expand 10 after
3503 return 3503 return
3504 3504
3505 is_function_arguments = is_function_arguments or character_after_identif ier == '(' 3505 is_function_arguments = is_function_arguments or character_after_identif ier == '('
3506 3506
3507 # Remove "m_" and "s_" to allow them. 3507 # Remove "m_" and "s_" to allow them.
3508 modified_identifier = sub(r'(^|(?<=::))[ms]_', '', identifier) 3508 modified_identifier = sub(r'(^|(?<=::))[ms]_', '', identifier)
3509 if not file_state.is_objective_c() and modified_identifier.find('_') >= 0: 3509 if not file_state.is_objective_c() and modified_identifier.find('_') >= 0:
3510 # Various exceptions to the rule: JavaScript op codes functions, con st_iterator. 3510 # Various exceptions to the rule: JavaScript op codes functions, con st_iterator.
3511 if (not (filename.find('JavaScriptCore') >= 0 and modified_identifie r.find('op_') >= 0) 3511 if (not (filename.find('JavaScriptCore') >= 0 and modified_identifie r.find('op_') >= 0)
3512 and not (filename.find('gtk') >= 0 and modified_identifier.s tartswith('webkit_') >= 0) 3512 and not (filename.find('gtk') >= 0 and modified_identifier.s tartswith('webkit_') >= 0)
3513 and not (filename.find('StructTraits.h') >= 0)
3513 and not modified_identifier.startswith('tst_') 3514 and not modified_identifier.startswith('tst_')
3514 and not modified_identifier.startswith('webkit_dom_object_') 3515 and not modified_identifier.startswith('webkit_dom_object_')
3515 and not modified_identifier.startswith('webkit_soup') 3516 and not modified_identifier.startswith('webkit_soup')
3516 and not modified_identifier.startswith('NPN_') 3517 and not modified_identifier.startswith('NPN_')
3517 and not modified_identifier.startswith('NPP_') 3518 and not modified_identifier.startswith('NPP_')
3518 and not modified_identifier.startswith('NP_') 3519 and not modified_identifier.startswith('NP_')
3519 and not modified_identifier.startswith('qt_') 3520 and not modified_identifier.startswith('qt_')
3520 and not modified_identifier.startswith('_q_') 3521 and not modified_identifier.startswith('_q_')
3521 and not modified_identifier.startswith('cairo_') 3522 and not modified_identifier.startswith('cairo_')
3522 and not modified_identifier.startswith('Ecore_') 3523 and not modified_identifier.startswith('Ecore_')
(...skipping 658 matching lines...) Expand 10 before | Expand all | Expand 10 after
4181 4182
4182 def check(self, lines): 4183 def check(self, lines):
4183 _process_lines(self.file_path, self.file_extension, lines, 4184 _process_lines(self.file_path, self.file_extension, lines,
4184 self.handle_style_error, self.min_confidence) 4185 self.handle_style_error, self.min_confidence)
4185 4186
4186 4187
4187 # FIXME: Remove this function (requires refactoring unit tests). 4188 # FIXME: Remove this function (requires refactoring unit tests).
4188 def process_file_data(filename, file_extension, lines, error, min_confidence, fs =None): 4189 def process_file_data(filename, file_extension, lines, error, min_confidence, fs =None):
4189 checker = CppChecker(filename, file_extension, error, min_confidence, fs) 4190 checker = CppChecker(filename, file_extension, error, min_confidence, fs)
4190 checker.check(lines) 4191 checker.check(lines)
OLDNEW
« no previous file with comments | « third_party/WebKit/PRESUBMIT.py ('k') | third_party/WebKit/public/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698