Finish landing missing/incorrect header guards across flutter/engine (flutter#50069)

Generated by flutter/engine#48903 (`dart ./tools/header_guard_check/bin/main.dart --fix`).

As discussed with @cbracken and @jmagman, the guards are not technically needed on the Mac/iOS code, but they (a) do not hurt and (b) still provide value if for some reason `#include` is used instead of `#import` (though I suspect we could try to add that to the tool in the future as well).

Author: matanlurey

impeller/aiks/aiks_playground_inspector.h

@@ -2,6 +2,9 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#ifndef FLUTTER_IMPELLER_AIKS_AIKS_PLAYGROUND_INSPECTOR_H_
+#define FLUTTER_IMPELLER_AIKS_AIKS_PLAYGROUND_INSPECTOR_H_
+
 #include <functional>
 #include <optional>
 
@@ -48,3 +51,5 @@ class AiksInspector {
 };
 
 };  // namespace impeller
+
+#endif  // FLUTTER_IMPELLER_AIKS_AIKS_PLAYGROUND_INSPECTOR_H_

impeller/compiler/shader_bundle.h

@@ -2,6 +2,9 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#ifndef FLUTTER_IMPELLER_COMPILER_SHADER_BUNDLE_H_
+#define FLUTTER_IMPELLER_COMPILER_SHADER_BUNDLE_H_
+
 #include "impeller/compiler/source_options.h"
 #include "impeller/compiler/switches.h"
 #include "impeller/shader_bundle/shader_bundle_flatbuffers.h"
@@ -33,3 +36,5 @@ bool GenerateShaderBundle(Switches& switches);
 
 }  // namespace compiler
 }  // namespace impeller
+
+#endif  // FLUTTER_IMPELLER_COMPILER_SHADER_BUNDLE_H_

impeller/renderer/backend/gles/test/mock_gles.h

@@ -2,6 +2,9 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#ifndef FLUTTER_IMPELLER_RENDERER_BACKEND_GLES_TEST_MOCK_GLES_H_
+#define FLUTTER_IMPELLER_RENDERER_BACKEND_GLES_TEST_MOCK_GLES_H_
+
 #include <memory>
 #include <optional>
 #include "fml/macros.h"
@@ -60,3 +63,5 @@ class MockGLES final {
 
 }  // namespace testing
 }  // namespace impeller
+
+#endif  // FLUTTER_IMPELLER_RENDERER_BACKEND_GLES_TEST_MOCK_GLES_H_

impeller/renderer/backend/vulkan/gpu_tracer_vk.h

@@ -2,6 +2,9 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#ifndef FLUTTER_IMPELLER_RENDERER_BACKEND_VULKAN_GPU_TRACER_VK_H_
+#define FLUTTER_IMPELLER_RENDERER_BACKEND_VULKAN_GPU_TRACER_VK_H_
+
 #include <memory>
 #include <thread>
 
@@ -115,3 +118,5 @@ class GPUProbe {
 };
 
 }  // namespace impeller
+
+#endif  // FLUTTER_IMPELLER_RENDERER_BACKEND_VULKAN_GPU_TRACER_VK_H_

impeller/renderer/texture_mipmap.h

@@ -2,8 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef FLUTTER_IMPELLER_TEXTURE_MIPMAP_H_
-#define FLUTTER_IMPELLER_TEXTURE_MIPMAP_H_
+#ifndef FLUTTER_IMPELLER_RENDERER_TEXTURE_MIPMAP_H_
+#define FLUTTER_IMPELLER_RENDERER_TEXTURE_MIPMAP_H_
 
 #include "flutter/fml/status.h"
 #include "impeller/core/texture.h"
@@ -20,4 +20,4 @@ namespace impeller {
 
 }  // namespace impeller
 
-#endif  // FLUTTER_IMPELLER_TEXTURE_MIPMAP_H_
+#endif  // FLUTTER_IMPELLER_RENDERER_TEXTURE_MIPMAP_H_

lib/gpu/fixtures.h

@@ -2,12 +2,15 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#ifndef FLUTTER_LIB_GPU_FIXTURES_H_
+#define FLUTTER_LIB_GPU_FIXTURES_H_
+
 #include "impeller/core/shader_types.h"
 
 struct FlutterGPUUnlitVertexShader {
   struct PerVertexData {
     impeller::Point position;  // (offset 0, size 8)
-  };                           // struct PerVertexData (size 8)
+  };  // struct PerVertexData (size 8)
 
   static constexpr auto kInputPosition = impeller::ShaderStageIOSlot{
       // position
@@ -46,7 +49,7 @@ struct FlutterGPUTextureVertexShader {
     impeller::Vector3 position;      // (offset 0, size 12)
     impeller::Point texture_coords;  // (offset 12, size 8)
     impeller::Vector4 color;         // (offset 20, size 16)
-  };                                 // struct PerVertexData (size 36)
+  };  // struct PerVertexData (size 36)
 
   static constexpr auto kInputTextureCoords = impeller::ShaderStageIOSlot{
       // texture_coords
@@ -107,3 +110,5 @@ extern unsigned char kFlutterGPUTextureVertIPLR[];
 
 constexpr unsigned int kFlutterGPUTextureFragIPLRLength = 800;
 extern unsigned char kFlutterGPUTextureFragIPLR[];
+
+#endif  // FLUTTER_LIB_GPU_FIXTURES_H_

lib/gpu/formats.h

@@ -2,6 +2,9 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#ifndef FLUTTER_LIB_GPU_FORMATS_H_
+#define FLUTTER_LIB_GPU_FORMATS_H_
+
 #include "fml/logging.h"
 #include "impeller/core/formats.h"
 #include "impeller/core/shader_types.h"
@@ -500,3 +503,5 @@ constexpr impeller::StencilOperation ToImpellerStencilOperation(int value) {
 
 }  // namespace gpu
 }  // namespace flutter
+
+#endif  // FLUTTER_LIB_GPU_FORMATS_H_

lib/ui/painting/image_decoder_no_gl_unittests.h

@@ -2,6 +2,9 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#ifndef FLUTTER_LIB_UI_PAINTING_IMAGE_DECODER_NO_GL_UNITTESTS_H_
+#define FLUTTER_LIB_UI_PAINTING_IMAGE_DECODER_NO_GL_UNITTESTS_H_
+
 #include <stdint.h>
 
 #include "flutter/impeller/core/allocator.h"
@@ -99,3 +102,5 @@ float DecodeBGR10(uint32_t x);
 
 }  // namespace testing
 }  // namespace flutter
+
+#endif  // FLUTTER_LIB_UI_PAINTING_IMAGE_DECODER_NO_GL_UNITTESTS_H_

lib/web_ui/skwasm/skwasm_support.h

@@ -2,6 +2,9 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#ifndef FLUTTER_LIB_WEB_UI_SKWASM_SKWASM_SUPPORT_H_
+#define FLUTTER_LIB_WEB_UI_SKWASM_SKWASM_SUPPORT_H_
+
 #include <emscripten/threading.h>
 #include <cinttypes>
 #include "third_party/skia/include/core/SkPicture.h"
@@ -36,3 +39,5 @@ extern unsigned int skwasm_createGlTextureFromTextureSource(
     int width,
     int height);
 }
+
+#endif  // FLUTTER_LIB_WEB_UI_SKWASM_SKWASM_SUPPORT_H_

shell/common/vsync_waiters_test.h

@@ -2,11 +2,11 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#define FML_USED_ON_EMBEDDER
-
 #ifndef FLUTTER_SHELL_COMMON_VSYNC_WAITERS_TEST_H_
 #define FLUTTER_SHELL_COMMON_VSYNC_WAITERS_TEST_H_
 
+#define FML_USED_ON_EMBEDDER
+
 #include <utility>
 
 #include "flutter/shell/common/shell.h"