Tidy

Author: chinmaygarde

impeller/renderer/backend/gles/test/surface_gles_unittests.cc

@@ -25,6 +25,7 @@ TEST_P(SurfaceGLESTest, CanWrapNonZeroFBO) {
       *(surface->GetRenderTarget().GetColorAttachments().at(0).texture));
   auto wrapped = texture.GetFBO();
   ASSERT_TRUE(wrapped.has_value());
+  // NOLINTNEXTLINE(bugprone-unchecked-optional-access)
   ASSERT_EQ(wrapped.value(), fbo);
 }
 

impeller/renderer/backend/gles/texture_gles.cc

@@ -163,8 +163,8 @@ std::shared_ptr<TextureGLES> TextureGLES::WrapTexture(
     VALIDATION_LOG << "Cannot wrap a non-texture handle.";
     return nullptr;
   }
-  auto texture = std::shared_ptr<TextureGLES>(new TextureGLES(
-      std::move(reactor), desc, std::nullopt, std::move(external_handle)));
+  auto texture = std::shared_ptr<TextureGLES>(
+      new TextureGLES(std::move(reactor), desc, std::nullopt, external_handle));
   if (!texture->IsValid()) {
     return nullptr;
   }