Fix tidy issues (#42457)

Alternative to #42456

@itsjustkevin @zanderso

---------

Co-authored-by: Zachary Anderson <[email protected]>

Author: dnfield

lib/ui/painting/image_decoder_impeller.cc

@@ -338,6 +338,7 @@ ImageDecoderImpeller::UploadTextureToPrivate(
     const std::shared_ptr<impeller::Context>& context,
     const std::shared_ptr<impeller::DeviceBuffer>& buffer,
     const SkImageInfo& image_info,
+    // NOLINTNEXTLINE(performance-unnecessary-value-param)
     std::shared_ptr<SkBitmap> bitmap,
     const std::shared_ptr<fml::SyncSwitch>& gpu_disabled_switch) {
   TRACE_EVENT0("impeller", __FUNCTION__);
@@ -365,6 +366,7 @@ ImageDecoderImpeller::UploadTextureToPrivate(
 std::pair<sk_sp<DlImage>, std::string>
 ImageDecoderImpeller::UploadTextureToShared(
     const std::shared_ptr<impeller::Context>& context,
+    // NOLINTNEXTLINE(performance-unnecessary-value-param)
     std::shared_ptr<SkBitmap> bitmap,
     const std::shared_ptr<fml::SyncSwitch>& gpu_disabled_switch,
     bool create_mips) {

lib/ui/painting/image_decoder_impeller.h

@@ -80,6 +80,7 @@ class ImageDecoderImpeller final : public ImageDecoder {
       const std::shared_ptr<impeller::Context>& context,
       const std::shared_ptr<impeller::DeviceBuffer>& buffer,
       const SkImageInfo& image_info,
+      // NOLINTNEXTLINE(performance-unnecessary-value-param)
       std::shared_ptr<SkBitmap> bitmap,
       const std::shared_ptr<fml::SyncSwitch>& gpu_disabled_switch);
 
@@ -93,6 +94,7 @@ class ImageDecoderImpeller final : public ImageDecoder {
   /// @return            A DlImage.
   static std::pair<sk_sp<DlImage>, std::string> UploadTextureToShared(
       const std::shared_ptr<impeller::Context>& context,
+      // NOLINTNEXTLINE(performance-unnecessary-value-param)
       std::shared_ptr<SkBitmap> bitmap,
       const std::shared_ptr<fml::SyncSwitch>& gpu_disabled_switch,
       bool create_mips = true);