Committing clang-format changes

Author: openwebf-bot

bridge/bindings/qjs/cppgc/member.h

@@ -25,7 +25,6 @@ class ScriptWrappable;
 template <typename T, typename = std::is_base_of<ScriptWrappable, T>>
 class Member {
  public:
-
   struct KeyHasher {
     std::size_t operator()(const Member& k) const { return k.raw_; }
   };

bridge/core/dom/child_list_mutation_scope.cc

@@ -138,10 +138,10 @@ bool ChildListMutationAccumulator::IsEmpty() {
 void ChildListMutationAccumulator::Trace(GCVisitor* visitor) const {
   visitor->TraceMember(target_);
 
-  for(auto& entry : removed_nodes_) {
+  for (auto& entry : removed_nodes_) {
     visitor->TraceMember(entry);
   }
-  for(auto& entry : added_nodes_) {
+  for (auto& entry : added_nodes_) {
     visitor->TraceMember(entry);
   }
 

bridge/core/dom/child_list_mutation_scope.h

@@ -1,13 +1,13 @@
 /*
-* Copyright (C) 2022-present The WebF authors. All rights reserved.
-*/
+ * Copyright (C) 2022-present The WebF authors. All rights reserved.
+ */
 
 #ifndef WEBF_CORE_DOM_CHILD_LIST_MUTATION_SCOPE_H_
 #define WEBF_CORE_DOM_CHILD_LIST_MUTATION_SCOPE_H_
 
+#include "document.h"
 #include "foundation/macros.h"
 #include "mutation_observer_interest_group.h"
-#include "document.h"
 #include "static_node_list.h"
 
 namespace webf {
@@ -61,8 +61,7 @@ class ChildListMutationScope final {
 
  public:
   explicit ChildListMutationScope(Node& target) {
-    if (target.ownerDocument()->HasMutationObserversOfType(
-            kMutationTypeChildList)) {
+    if (target.ownerDocument()->HasMutationObserversOfType(kMutationTypeChildList)) {
       accumulator_ = ChildListMutationAccumulator::GetOrCreate(target);
       // Register another user of the accumulator.
       accumulator_->EnterMutationScope();
@@ -93,7 +92,6 @@ class ChildListMutationScope final {
   std::shared_ptr<ChildListMutationAccumulator> accumulator_ = nullptr;
 };
 
-
-}
+}  // namespace webf
 
 #endif  // WEBF_CORE_DOM_CHILD_LIST_MUTATION_SCOPE_H_

bridge/core/dom/container_node.cc

@@ -29,12 +29,12 @@
 #include "container_node.h"
 #include "bindings/qjs/cppgc/garbage_collected.h"
 #include "bindings/qjs/cppgc/gc_visitor.h"
+#include "child_list_mutation_scope.h"
 #include "child_node_list.h"
 #include "core/html/html_all_collection.h"
 #include "document.h"
 #include "document_fragment.h"
 #include "node_traversal.h"
-#include "child_list_mutation_scope.h"
 
 namespace webf {
 

bridge/core/dom/document.h

@@ -100,13 +100,9 @@ class Document : public ContainerNode, public TreeScope {
 
   ScriptValue location() const;
 
-  bool HasMutationObserversOfType(MutationType type) const {
-    return mutation_observer_types_ & type;
-  }
+  bool HasMutationObserversOfType(MutationType type) const { return mutation_observer_types_ & type; }
   bool HasMutationObservers() const { return mutation_observer_types_; }
-  void AddMutationObserverTypes(MutationType types) {
-    mutation_observer_types_ |= types;
-  }
+  void AddMutationObserverTypes(MutationType types) { mutation_observer_types_ |= types; }
 
   void IncrementNodeCount() { node_count_++; }
   void DecrementNodeCount() {

bridge/core/dom/mutation_observer.cc

@@ -34,8 +34,8 @@
 
 #include "mutation_observer.h"
 #include "mutation_observer_options.h"
-#include "node.h"
 #include "mutation_observer_registration.h"
+#include "node.h"
 
 namespace webf {
 
@@ -54,14 +54,14 @@ MutationObserver::MutationObserver(ExecutingContext* context, const std::shared_
 
 MutationObserver::~MutationObserver() = default;
 
-void MutationObserver::observe(Node* node, const std::shared_ptr<MutationObserverInit>& observer_init, ExceptionState& exception_state) {
-
+void MutationObserver::observe(Node* node,
+                               const std::shared_ptr<MutationObserverInit>& observer_init,
+                               ExceptionState& exception_state) {
   assert(node != nullptr);
 
   MutationObserverOptions options = 0;
 
-  if (observer_init->hasAttributeOldValue() &&
-      observer_init->attributeOldValue())
+  if (observer_init->hasAttributeOldValue() && observer_init->attributeOldValue())
     options |= kAttributeOldValue;
 
   std::set<AtomicString> attribute_filter;
@@ -71,21 +71,16 @@ void MutationObserver::observe(Node* node, const std::shared_ptr<MutationObserve
     options |= kAttributeFilter;
   }
 
-  bool attributes =
-      observer_init->hasAttributes() && observer_init->attributes();
+  bool attributes = observer_init->hasAttributes() && observer_init->attributes();
   if (attributes || (!observer_init->hasAttributes() &&
-                     (observer_init->hasAttributeOldValue() ||
-                      observer_init->hasAttributeFilter())))
+                     (observer_init->hasAttributeOldValue() || observer_init->hasAttributeFilter())))
     options |= kMutationTypeAttributes;
 
-  if (observer_init->hasCharacterDataOldValue() &&
-      observer_init->characterDataOldValue())
+  if (observer_init->hasCharacterDataOldValue() && observer_init->characterDataOldValue())
     options |= kCharacterDataOldValue;
 
-  bool character_data =
-      observer_init->hasCharacterData() && observer_init->characterData();
-  if (character_data || (!observer_init->hasCharacterData() &&
-                         observer_init->hasCharacterDataOldValue()))
+  bool character_data = observer_init->hasCharacterData() && observer_init->characterData();
+  if (character_data || (!observer_init->hasCharacterData() && observer_init->hasCharacterDataOldValue()))
     options |= kMutationTypeCharacterData;
 
   if (observer_init->childList())
@@ -103,23 +98,22 @@ void MutationObserver::observe(Node* node, const std::shared_ptr<MutationObserve
     }
     if (options & kAttributeFilter) {
       exception_state.ThrowException(ctx(), ErrorType::TypeError,
-          "The options object may only set 'attributeFilter' when 'attributes' "
-          "is true or not present.");
+                                     "The options object may only set 'attributeFilter' when 'attributes' "
+                                     "is true or not present.");
       return;
     }
   }
-  if (!((options & kMutationTypeCharacterData) ||
-        !(options & kCharacterDataOldValue))) {
+  if (!((options & kMutationTypeCharacterData) || !(options & kCharacterDataOldValue))) {
     exception_state.ThrowException(ctx(), ErrorType::TypeError,
-        "The options object may only set 'characterDataOldValue' to true when "
-        "'characterData' is true or not present.");
+                                   "The options object may only set 'characterDataOldValue' to true when "
+                                   "'characterData' is true or not present.");
     return;
   }
 
   if (!(options & kMutationTypeAll)) {
     exception_state.ThrowException(ctx(), ErrorType::TypeError,
-        "The options object must set at least one of 'attributes', "
-        "'characterData', or 'childList' to true.");
+                                   "The options object must set at least one of 'attributes', "
+                                   "'characterData', or 'childList' to true.");
     return;
   }
 
@@ -169,14 +163,13 @@ std::set<Member<Node>> MutationObserver::GetObservedNodes() const {
 }
 
 void MutationObserver::Trace(GCVisitor* visitor) const {
-  for(auto& record : records_) {
+  for (auto& record : records_) {
     visitor->TraceMember(record);
   }
 
-  for(auto& re : registrations_) {
+  for (auto& re : registrations_) {
     re->Trace(visitor);
   }
 }
 
-
 }  // namespace webf

bridge/core/dom/mutation_observer.h

@@ -35,11 +35,11 @@
 #ifndef WEBF_MUTATION_OBSERVER_H
 #define WEBF_MUTATION_OBSERVER_H
 
+#include "bindings/qjs/cppgc/member.h"
 #include "bindings/qjs/exception_state.h"
 #include "bindings/qjs/script_wrappable.h"
-#include "bindings/qjs/cppgc/member.h"
-#include "qjs_mutation_observer_init.h"
 #include "mutation_record.h"
+#include "qjs_mutation_observer_init.h"
 
 namespace webf {
 
@@ -49,13 +49,13 @@ class MutationObserverInit;
 class MutationObserverRegistration;
 
 using MutationObserverSet = std::set<Member<MutationObserver>>;
-using MutationObserverRegistrationSet =
-    std::set<std::shared_ptr<MutationObserverRegistration>>;
+using MutationObserverRegistrationSet = std::set<std::shared_ptr<MutationObserverRegistration>>;
 using MutationObserverVector = std::vector<Member<MutationObserver>>;
 using MutationRecordVector = std::vector<MutationRecord*>;
 
 class MutationObserver final : public ScriptWrappable {
   DEFINE_WRAPPERTYPEINFO();
+
  public:
   enum ObservationFlags { kSubtree = 1 << 3, kAttributeFilter = 1 << 4 };
   enum DeliveryFlags {
@@ -67,8 +67,7 @@ class MutationObserver final : public ScriptWrappable {
    public:
     ~Delegate() = default;
     virtual ExecutingContext* GetExecutingContext() const = 0;
-    virtual void Deliver(const MutationRecordVector& records,
-                         MutationObserver&) = 0;
+    virtual void Deliver(const MutationRecordVector& records, MutationObserver&) = 0;
     virtual void Trace(GCVisitor* visitor) const {}
   };
 
@@ -92,7 +91,7 @@ class MutationObserver final : public ScriptWrappable {
 
   bool HasPendingActivity() const { return !records_.empty(); }
 
-  void Trace(webf::GCVisitor *visitor) const override;
+  void Trace(webf::GCVisitor* visitor) const override;
 
  private:
   std::vector<Member<MutationRecord>> records_;

bridge/core/dom/mutation_observer_interest_group.cc

@@ -53,7 +53,8 @@ std::shared_ptr<MutationObserverInterestGroup> MutationObserverInterestGroup::Cr
 
 MutationObserverInterestGroup::MutationObserverInterestGroup(
     std::unordered_map<MutationObserver*, MutationRecordDeliveryOptions>& observers,
-    webf::MutationRecordDeliveryOptions old_value_flag): old_value_flag_(old_value_flag) {
+    webf::MutationRecordDeliveryOptions old_value_flag)
+    : old_value_flag_(old_value_flag) {
   assert(!observers.empty());
   observers_.swap(observers);
 }
@@ -66,8 +67,7 @@ bool MutationObserverInterestGroup::IsOldValueRequested() {
   return false;
 }
 
-void MutationObserverInterestGroup::EnqueueMutationRecord(
-    MutationRecord* mutation) {
+void MutationObserverInterestGroup::EnqueueMutationRecord(MutationRecord* mutation) {
   MutationRecord* mutation_with_null_old_value = nullptr;
 
   for (auto& iter : observers_) {
@@ -80,8 +80,7 @@ void MutationObserverInterestGroup::EnqueueMutationRecord(
       if (mutation->oldValue().IsNull())
         mutation_with_null_old_value = mutation;
       else
-        mutation_with_null_old_value =
-            MutationRecord::CreateWithNullOldValue(mutation);
+        mutation_with_null_old_value = MutationRecord::CreateWithNullOldValue(mutation);
     }
     observer->EnqueueMutationRecord(mutation_with_null_old_value);
   }