Remove suppressing [chromium-style] errors

Lots of little style nits needed to be fixed for this work.

BUG=tint:44

Change-Id: Ibb45d9e3f6795ee0c09f5eca994bb28e20979d97
Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/19221
Reviewed-by: dan sinclair <dsinclair@google.com>
diff --git a/src/ast/identifier_expression.h b/src/ast/identifier_expression.h
index 3e46129..e6eee06 100644
--- a/src/ast/identifier_expression.h
+++ b/src/ast/identifier_expression.h
@@ -42,7 +42,7 @@
   /// @param name the name
   IdentifierExpression(const Source& source, std::vector<std::string> name);
   /// Move constructor
-  IdentifierExpression(IdentifierExpression&&) = default;
+  IdentifierExpression(IdentifierExpression&&);
   ~IdentifierExpression() override;
 
   /// Sets the name
@@ -52,7 +52,7 @@
   std::vector<std::string> name() const { return name_; }
 
   /// @returns true if this is an identifier expression
-  bool IsIdentifier() const override { return true; }
+  bool IsIdentifier() const override;
 
   /// @returns true if the node is valid
   bool IsValid() const override;