Update HeaderFilterRegex once and for all. (#48145)

Includes a test (`header_filter_regex_test.dart`) that hopefully keeps
us on rails going forward.

Author: matanlurey

.clang-tidy

@@ -45,10 +45,19 @@ CheckOptions:
   - key: readability-identifier-naming.PrivateMemberSuffix
     value: "_"
 
-# Lint headers within paths that contain "/flutter/" but not:
-# - gen         (generated code, the fact it compiles is good enough™)
-# - third_party (we didn't author most of the code, and can't fix the lints)
+# Intended to include (lint) all headers except:
+# ... those in ../../gen (generated files don't need to follow clang tidy)
+# ... those in flutter/third_party (we didn't author this code, it's a dep)
 #
-# Note this is because of our buildroot setup, so the full path of a lint is:
-# "../../flutter/impeller/core/runtime_types.h:1:1" as reported.
-HeaderFilterRegex: "(!third_party/)(!gen/).*/flutter/.*"
+# Unfortunately Clang Tidy uses an ancient version of regular expressions
+# without lookaheads, so the next best thing is to write out every directory
+# except third_party.
+#
+# If we ever add a new root directory, we'll have to update this expression.
+#
+# See the test in ./tools/clang_tidy/test/header_filter_regex_test.dart which
+# should theoretically catch if new directories are added but this regex is not
+# updated.
+#
+# tl;dr: I'm sorry.
+HeaderFilterRegex: "[..\/]+\/flutter\/(assets|benchmarking|build|ci|common|display_list|docs|examples|flow|flutter_frontend_server|flutter_vma|fml|impeller|lib|runtime|shell|skia|sky|testing|tools|vulkan|wasm|web_sdk)\/.*"

impeller/renderer/vertex_buffer_builder.h

@@ -42,7 +42,7 @@ class VertexBufferBuilder {
     return impeller::IndexType::kUnknown;
   }
 
-  void SetLabel(std::string label) { label_ = std::move(label); }
+  void SetLabel(const std::string& label) { label_ = label; }
 
   void Reserve(size_t count) { return vertices_.reserve(count); }
 

shell/common/shell_test_platform_view_metal.mm

@@ -31,12 +31,13 @@
 // non-Objective-C TUs.
 class DarwinContextMetal {
  public:
-  explicit DarwinContextMetal(bool impeller,
-                              std::shared_ptr<const fml::SyncSwitch> is_gpu_disabled_sync_switch)
+  explicit DarwinContextMetal(
+      bool impeller,
+      const std::shared_ptr<const fml::SyncSwitch>& is_gpu_disabled_sync_switch)
       : context_(impeller ? nil : [[FlutterDarwinContextMetalSkia alloc] initWithDefaultMTLDevice]),
-        impeller_context_(impeller ? [[FlutterDarwinContextMetalImpeller alloc]
-                                         init:std::move(is_gpu_disabled_sync_switch)]
-                                   : nil),
+        impeller_context_(
+            impeller ? [[FlutterDarwinContextMetalImpeller alloc] init:is_gpu_disabled_sync_switch]
+                     : nil),
         offscreen_texture_(CreateOffscreenTexture(
             impeller ? [impeller_context_ context]->GetMTLDevice() : [context_ device])) {}
 

shell/platform/darwin/common/framework/Source/FlutterStandardCodecHelper.h

@@ -54,6 +54,8 @@ typedef enum {
   // NOLINTEND(readability-identifier-naming)
 } FlutterStandardCodecObjcType;
 
+// NOLINTBEGIN(google-objc-function-naming)
+
 ///////////////////////////////////////////////////////////////////////////////
 ///\name Reader Helpers
 ///@{
@@ -111,6 +113,7 @@ bool FlutterStandardCodecHelperWriteNumber(CFMutableDataRef data,
 
 ///@}
 
+// NOLINTEND(google-objc-function-naming)
 // NOLINTEND(google-runtime-int)
 
 #if defined(__cplusplus)

shell/platform/darwin/graphics/FlutterDarwinContextMetalImpeller.h

@@ -25,7 +25,7 @@ NS_ASSUME_NONNULL_BEGIN
 /**
  * Initializes a FlutterDarwinContextMetalImpeller.
  */
-- (instancetype)init:(std::shared_ptr<const fml::SyncSwitch>)is_gpu_disabled_sync_switch;
+- (instancetype)init:(const std::shared_ptr<const fml::SyncSwitch>&)is_gpu_disabled_sync_switch;
 
 /**
  * Creates an external texture with the specified ID and contents.

shell/platform/darwin/graphics/FlutterDarwinContextMetalImpeller.mm

@@ -20,21 +20,21 @@
 FLUTTER_ASSERT_ARC
 
 static std::shared_ptr<impeller::ContextMTL> CreateImpellerContext(
-    std::shared_ptr<const fml::SyncSwitch> is_gpu_disabled_sync_switch) {
+    const std::shared_ptr<const fml::SyncSwitch>& is_gpu_disabled_sync_switch) {
   std::vector<std::shared_ptr<fml::Mapping>> shader_mappings = {
-    std::make_shared<fml::NonOwnedMapping>(impeller_entity_shaders_data,
-                                           impeller_entity_shaders_length),
+      std::make_shared<fml::NonOwnedMapping>(impeller_entity_shaders_data,
+                                             impeller_entity_shaders_length),
 #if IMPELLER_ENABLE_3D
-    std::make_shared<fml::NonOwnedMapping>(impeller_scene_shaders_data,
-                                           impeller_scene_shaders_length),
+      std::make_shared<fml::NonOwnedMapping>(impeller_scene_shaders_data,
+                                             impeller_scene_shaders_length),
 #endif  // IMPELLER_ENABLE_3D
-    std::make_shared<fml::NonOwnedMapping>(impeller_modern_shaders_data,
-                                           impeller_modern_shaders_length),
-    std::make_shared<fml::NonOwnedMapping>(impeller_framebuffer_blend_shaders_data,
-                                           impeller_framebuffer_blend_shaders_length),
+      std::make_shared<fml::NonOwnedMapping>(impeller_modern_shaders_data,
+                                             impeller_modern_shaders_length),
+      std::make_shared<fml::NonOwnedMapping>(impeller_framebuffer_blend_shaders_data,
+                                             impeller_framebuffer_blend_shaders_length),
   };
-  auto context = impeller::ContextMTL::Create(
-      shader_mappings, std::move(is_gpu_disabled_sync_switch), "Impeller Library");
+  auto context = impeller::ContextMTL::Create(shader_mappings, is_gpu_disabled_sync_switch,
+                                              "Impeller Library");
   if (!context) {
     FML_LOG(ERROR) << "Could not create Metal Impeller Context.";
     return nullptr;
@@ -46,10 +46,10 @@
 
 @implementation FlutterDarwinContextMetalImpeller
 
-- (instancetype)init:(std::shared_ptr<const fml::SyncSwitch>)is_gpu_disabled_sync_switch {
+- (instancetype)init:(const std::shared_ptr<const fml::SyncSwitch>&)is_gpu_disabled_sync_switch {
   self = [super init];
   if (self != nil) {
-    _context = CreateImpellerContext(std::move(is_gpu_disabled_sync_switch));
+    _context = CreateImpellerContext(is_gpu_disabled_sync_switch);
     id<MTLDevice> device = _context->GetMTLDevice();
     if (!device) {
       FML_DLOG(ERROR) << "Could not acquire Metal device.";

shell/platform/darwin/ios/framework/Headers/FlutterEngine.h

@@ -22,11 +22,13 @@ NS_ASSUME_NONNULL_BEGIN
  * The dart entrypoint that is associated with `main()`.  This is to be used as an argument to the
  * `runWithEntrypoint*` methods.
  */
+// NOLINTNEXTLINE(readability-identifier-naming)
 extern NSString* const FlutterDefaultDartEntrypoint;
 
 /**
  * The default Flutter initial route ("/").
  */
+// NOLINTNEXTLINE(readability-identifier-naming)
 extern NSString* const FlutterDefaultInitialRoute;
 
 /**

shell/platform/darwin/ios/framework/Headers/FlutterPlugin.h

@@ -250,6 +250,7 @@ typedef void (*FlutterPluginRegistrantCallback)(NSObject<FlutterPluginRegistry>*
  * Flutter Framework (e.g. When an interact-able widget is covering the platform view).
  */
 typedef enum {
+  // NOLINTBEGIN(readability-identifier-naming)
   /**
    * Flutter blocks all the UIGestureRecognizers on the platform view as soon as it
    * decides they should be blocked.
@@ -266,6 +267,7 @@ typedef enum {
    * but never recognizing the gesture (and never invoking actions).
    */
   FlutterPlatformViewGestureRecognizersBlockingPolicyWaitUntilTouchesEnded,
+  // NOLINTEND(readability-identifier-naming)
 } FlutterPlatformViewGestureRecognizersBlockingPolicy;
 
 #pragma mark -

shell/platform/darwin/ios/framework/Headers/FlutterViewController.h

@@ -26,6 +26,7 @@ NS_ASSUME_NONNULL_BEGIN
  * with the update.
  */
 FLUTTER_DARWIN_EXPORT
+// NOLINTNEXTLINE(readability-identifier-naming)
 extern NSNotificationName const FlutterSemanticsUpdateNotification;
 
 /**

shell/platform/darwin/ios/framework/Source/FlutterEmbedderKeyResponder.h

@@ -7,19 +7,16 @@
 
 #import <Foundation/NSObject.h>
 #import <UIKit/UIKit.h>
+
 #include "fml/memory/weak_ptr.h"
 
 #import "flutter/shell/platform/darwin/ios/framework/Headers/FlutterViewController.h"
 #import "flutter/shell/platform/darwin/ios/framework/Source/FlutterKeyPrimaryResponder.h"
 #import "flutter/shell/platform/embedder/embedder.h"
 
-namespace {
-typedef void* _VoidPtr;
-}
-
 typedef void (^FlutterSendKeyEvent)(const FlutterKeyEvent& /* event */,
                                     _Nullable FlutterKeyEventCallback /* callback */,
-                                    _Nullable _VoidPtr /* user_data */);
+                                    void* _Nullable /* user_data */);
 
 /**
  * A primary responder of |FlutterKeyboardManager| that handles events by