[handles] Fix benign data race

Upon destroying a v8::TracedReference while concurrent marking is
running, we merely reset the object pointer but do not get rid of the
global handle.

We were also restting the parameter which would read the internal
state for a DCHECK. Remove this clearing as the parameter field is not
used for v8::TracedReference.

Bug: chromium:1324074
Change-Id: Ic21bad78deba0925e12c3fc1215b087d0ef5dd7a
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3637796Reviewed-by: Dominik Inführ <dinfuehr@chromium.org>
Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
Auto-Submit: Michael Lippautz <mlippautz@chromium.org>
Cr-Commit-Position: refs/heads/main@{#80459}

src/handles/global-handles.cc

@@ -1125,7 +1125,6 @@ void GlobalHandles::DestroyTracedReference(Address* location) {
     // still be spuriously marked as live and is then only reclaimed on the
     // next cycle.
     node->clear_object();
-    node->set_parameter(nullptr);
   }
 }