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

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: Addressed remaining CR feedback from dcheng@. 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..956be9e330f787ef61efde73bc739918a7dac45e 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,42 @@ 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, int y) { return 123; }
+ int idlTestMethodTwoOrThreeParams(int x, int y, int z) { return 123; }
+
+ int idlOptionalArgsPass(int x = 0) { return x; }
+ int IdlOptionalArgsStillTooMany(int x, int y = 0) { return x + y; }
+ int IdlOptionalArgsTooLittle(int x = 0) { return x; }
+
+ template <typename T>
+ int idlTemplateMethod(T x) {
+ return 123;
+ }
+
+ int path() { return 123; }
+ int GetPath(int x) { return 123; }
+};
+
+template <typename T>
+class IdlTemplateClass {
+ public:
+ int idlTestMethod(T x) { return 123; }
+};
+
} // namespace blink
// https://crbug.com/640688 - need to rewrite method name below.

Powered by Google App Engine
This is Rietveld 408576698