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

Unified Diff: tools/clang/value_cleanup/ListValueRewriter.cpp

Issue 2476493003: Remove FundamentalValue
Patch Set: Fix Created 4 years, 1 month 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 | « skia/ext/benchmarking_canvas.cc ('k') | tools/clang/value_cleanup/tests/list-value-append-original.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/clang/value_cleanup/ListValueRewriter.cpp
diff --git a/tools/clang/value_cleanup/ListValueRewriter.cpp b/tools/clang/value_cleanup/ListValueRewriter.cpp
index f32f818d85c969f050d73edc5bc1a48505d60f78..1a22868571459759da4f10811c735bf82b2e0364 100644
--- a/tools/clang/value_cleanup/ListValueRewriter.cpp
+++ b/tools/clang/value_cleanup/ListValueRewriter.cpp
@@ -351,7 +351,7 @@ void ListValueRewriter::RegisterMatchers(MatchFinder* match_finder) {
callee(cxxMethodDecl(hasName("::base::ListValue::Append"))),
argumentCountIs(1));
- // base::ListValue::Append(new base::FundamentalValue(bool))
+ // base::ListValue::Append(new base::Value(bool))
// => base::ListValue::AppendBoolean()
match_finder->addMatcher(
id("callExpr",
@@ -362,14 +362,14 @@ void ListValueRewriter::RegisterMatchers(MatchFinder* match_finder) {
"newExpr",
cxxNewExpr(has(cxxConstructExpr(
hasDeclaration(cxxMethodDecl(hasName(
- "::base::FundamentalValue::FundamentalValue"))),
+ "::base::Value::Value"))),
argumentCountIs(1),
hasArgument(
0, id("argExpr",
expr(hasType(booleanType())))))))))))),
&append_boolean_callback_);
- // base::ListValue::Append(new base::FundamentalValue(int))
+ // base::ListValue::Append(new base::Value(int))
// => base::ListValue::AppendInteger()
match_finder->addMatcher(
id("callExpr",
@@ -381,7 +381,7 @@ void ListValueRewriter::RegisterMatchers(MatchFinder* match_finder) {
"newExpr",
cxxNewExpr(has(cxxConstructExpr(
hasDeclaration(cxxMethodDecl(hasName(
- "::base::FundamentalValue::FundamentalValue"))),
+ "::base::Value::Value"))),
argumentCountIs(1),
hasArgument(0, id("argExpr",
expr(hasType(isInteger()),
@@ -389,7 +389,7 @@ void ListValueRewriter::RegisterMatchers(MatchFinder* match_finder) {
booleanType()))))))))))))),
&append_integer_callback_);
- // base::ListValue::Append(new base::FundamentalValue(double))
+ // base::ListValue::Append(new base::Value(double))
// => base::ListValue::AppendDouble()
match_finder->addMatcher(
id("callExpr",
@@ -400,7 +400,7 @@ void ListValueRewriter::RegisterMatchers(MatchFinder* match_finder) {
"newExpr",
cxxNewExpr(has(cxxConstructExpr(
hasDeclaration(cxxMethodDecl(hasName(
- "::base::FundamentalValue::FundamentalValue"))),
+ "::base::Value::Value"))),
argumentCountIs(1),
hasArgument(
0, id("argExpr",
« no previous file with comments | « skia/ext/benchmarking_canvas.cc ('k') | tools/clang/value_cleanup/tests/list-value-append-original.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698