feat(trace-tree-node): Minor cleanup

Author: Abdullah Khan

static/app/views/performance/newTraceDetails/traceModels/traceTreeNode/spanNode.tsx

@@ -44,18 +44,10 @@ export class SpanNode extends BaseNode<TraceTree.Span> {
     );
   }
 
-  get description(): string | undefined {
-    return this.value.description;
-  }
-
   get endTimestamp(): number {
     return this.value.timestamp;
   }
 
-  get startTimestamp(): number {
-    return this.value.start_timestamp;
-  }
-
   get drawerTabsTitle(): string {
     return this.op + (this.value.description ? ' - ' + this.value.description : '');
   }

static/app/views/performance/newTraceDetails/traceModels/traceTreeNode/transactionNode.tsx

@@ -307,13 +307,11 @@ export class TransactionNode extends BaseNode<TraceTree.Transaction> {
 
       parent.children.push(transaction);
       transaction.parent = parent;
-
-      // Invalidating the node and its children to ensure the tree is rebuilt correctly,
-      // after we have potentially done some re-parenting.
-      this.invalidate();
-      this.forEachChild(child => child.invalidate());
     }
 
+    // Invalidating the node and its children to ensure the tree is rebuilt correctly,
+    // after we have potentially done some re-parenting.
+    this.invalidate();
     this.children.sort(traceChronologicalSort);
     this.forEachChild(c => {
       c.invalidate();