Turned on performance-unnecessary-value-param everywhere.

Author: gaaclarke

.clang-tidy

@@ -28,7 +28,8 @@ clang-analyzer-*,\
 readability-identifier-naming,\
 clang-diagnostic-*,\
 google-objc-*,\
-google-explicit-constructor"
+google-explicit-constructor,\
+performance-unnecessary-value-param"
 
 CheckOptions:
   - key:    modernize-use-default-member-init.UseAssignment

ci/lint.sh

@@ -34,8 +34,9 @@ DART_BIN="${SRC_DIR}/third_party/dart/tools/sdks/dart-sdk/bin"
 DART="${DART_BIN}/dart"
 # TODO(https://github.com/flutter/flutter/issues/113848): Migrate all platforms
 # to have this as an error.
-MAC_HOST_WARNINGS_AS_ERRORS="performance-move-const-arg,performance-unnecessary-value-param"
+MAC_HOST_WARNINGS_AS_ERRORS="performance-move-const-arg"
 
+FLUTTER_LINT_PRINT_FIX=1
 # FLUTTER_LINT_PRINT_FIX will make it so that fix is executed and the generated
 # diff is printed to stdout if clang-tidy fails. This is helpful for enabling
 # new lints.