rearrange methods

Author: yaacovCR

src/execution/execute.ts

@@ -2640,14 +2640,23 @@ class DeferredFragmentRecord {
     }
   }
 
-  removePendingTree(path: Path<FieldGroup>) {
-    const children = this._results.get(path);
-    if (children !== undefined) {
-      for (const [childPath] of children) {
-        this.removePendingTree(childPath);
+  reportDeferredValue(path: Path<FieldGroup>, result: unknown) {
+    this._pending.delete(path);
+    const siblings = this._results.get(path.prev);
+    if (siblings !== undefined) {
+      const existingValue = siblings.get(path);
+      // if a null has already bubbled, do not overwrite
+      if (existingValue === undefined) {
+        siblings.set(path, result);
       }
     }
-    this.removePendingField(path);
+    this.completeIfReady();
+  }
+
+  completeIfReady() {
+    if (this._pending.size === 0) {
+      this.complete();
+    }
   }
 
   complete(): void {
@@ -2661,18 +2670,6 @@ class DeferredFragmentRecord {
     this._resolve?.(this.data);
   }
 
-  addValue(path: Path<FieldGroup>, value: unknown): void {
-    this._pending.delete(path);
-    const siblings = this._results.get(path.prev);
-    if (siblings !== undefined) {
-      const existingValue = siblings.get(path);
-      // if a null has already bubbled, do not overwrite
-      if (existingValue === undefined) {
-        siblings.set(path, value);
-      }
-    }
-  }
-
   addError(path: Path<FieldGroup>, error: GraphQLError): void {
     this.errors.push(error);
     this.removePendingTree(path);
@@ -2683,15 +2680,14 @@ class DeferredFragmentRecord {
     }
   }
 
-  completeIfReady() {
-    if (this._pending.size === 0) {
-      this.complete();
+  removePendingTree(path: Path<FieldGroup>) {
+    const children = this._results.get(path);
+    if (children !== undefined) {
+      for (const [childPath] of children) {
+        this.removePendingTree(childPath);
+      }
     }
-  }
-
-  reportDeferredValue(path: Path<FieldGroup>, result: unknown) {
-    this.addValue(path, result);
-    this.completeIfReady();
+    this.removePendingField(path);
   }
 
   _buildData(