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

Unified Diff: tools/clang/rewrite_to_chrome_style/tests/methods-expected.cc

Issue 2627003003: Add --idl-methods <filepath> parameter for skipping renaming of IDL methods. (Closed)
Patch Set: Covering static methods multiple method arities. Created 3 years, 11 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
Index: tools/clang/rewrite_to_chrome_style/tests/methods-expected.cc
diff --git a/tools/clang/rewrite_to_chrome_style/tests/methods-expected.cc b/tools/clang/rewrite_to_chrome_style/tests/methods-expected.cc
index af33c2b94281f75c9b1a9786bc8b7a1ba6d5db93..e3576ef405cd74d0ba0aeedc9a7f9991e2b57c35 100644
--- a/tools/clang/rewrite_to_chrome_style/tests/methods-expected.cc
+++ b/tools/clang/rewrite_to_chrome_style/tests/methods-expected.cc
@@ -302,6 +302,31 @@ class InternalClass {
} // namespace internal
+// Tests for --idl-methods. See also
+// MethodsNamedAfterIdlMatcherState::SetupBlacklistForTesting method.
+class IdlTestClass {
+ public:
+ static int IdlTestMethodNoParams(char x) { return 123; }
+ static int idlTestMethodOneParam(char x) { return 123; }
+
+ int idlTestMethodNoParams() { return 123; }
+ int IdlTestMethodNoParams(int x) { return 123; }
+
+ int IdlTestMethodOneParam() { return 123; }
+ int idlTestMethodOneParam(int x) { return 123; }
+
+ int IdlTestMethodTwoOrThreeParams() { return 123; }
+ int IdlTestMethodTwoOrThreeParams(int x) { return 123; }
+ int idlTestMethodTwoOrThreeParams(int x, int y) { return 123; }
+ int idlTestMethodTwoOrThreeParams(int x, int y, int z) { return 123; }
+
+ int idlTestMethodAnyNumberOfParams() { return 123; }
+ int idlTestMethodAnyNumberOfParams(int x) { return 123; }
+
+ int path() { return 123; }
+ int GetPath(int x) { return 123; }
+};
+
} // namespace blink
// https://crbug.com/640688 - need to rewrite method name below.

Powered by Google App Engine
This is Rietveld 408576698