Revert "unittests: Provide Context in TestWithHeapInternals"

This reverts commit f300a01a.

Reason for revert: Makes TSAN unhappy: https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux64%20TSAN%20-%20stress-incremental-marking/5299/overview

Original change's description:
> unittests: Provide Context in TestWithHeapInternals
>
> Change-Id: I54e658325dfbfb425c41cab2fd7b32253b380e37
> Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3247038
> Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
> Commit-Queue: Omer Katz <omerkatz@chromium.org>
> Auto-Submit: Michael Lippautz <mlippautz@chromium.org>
> Reviewed-by: Omer Katz <omerkatz@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#77577}

Change-Id: I3806a40847d327cc86e2816e00a74c80ba7b512e
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3247633
Auto-Submit: Leszek Swirski <leszeks@chromium.org>
Owners-Override: Leszek Swirski <leszeks@chromium.org>
Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#77578}

test/unittests/heap/heap-utils.h

@@ -37,13 +37,12 @@ class WithHeapInternals : public TMixin, HeapInternalsBase {
   }
 };
 
-using TestWithHeapInternals =           //
-    WithHeapInternals<                  //
-        WithContextMixin<               //
-            WithInternalIsolateMixin<   //
-                WithIsolateScopeMixin<  //
-                    WithIsolateMixin<   //
-                        ::testing::Test>>>>>;
+using TestWithHeapInternals =       //
+    WithHeapInternals<              //
+        WithInternalIsolateMixin<   //
+            WithIsolateScopeMixin<  //
+                WithIsolateMixin<   //
+                    ::testing::Test>>>>;
 
 }  // namespace internal
 }  // namespace v8

test/unittests/heap/unified-heap-unittest.cc

@@ -60,9 +60,11 @@ TEST_F(UnifiedHeapTest, OnlyGC) { CollectGarbageWithEmbedderStack(); }
 
 TEST_F(UnifiedHeapTest, FindingV8ToBlinkReference) {
   v8::HandleScope scope(v8_isolate());
+  v8::Local<v8::Context> context = v8::Context::New(v8_isolate());
+  v8::Context::Scope context_scope(context);
   uint16_t wrappable_type = WrapperHelper::kTracedEmbedderId;
   v8::Local<v8::Object> api_object = WrapperHelper::CreateWrapper(
-      context(), &wrappable_type,
+      context, &wrappable_type,
       cppgc::MakeGarbageCollected<Wrappable>(allocation_handle()));
   Wrappable::destructor_callcount = 0;
   EXPECT_FALSE(api_object.IsEmpty());
@@ -76,11 +78,12 @@ TEST_F(UnifiedHeapTest, FindingV8ToBlinkReference) {
 
 TEST_F(UnifiedHeapTest, WriteBarrierV8ToCppReference) {
   if (!FLAG_incremental_marking) return;
-
   v8::HandleScope scope(v8_isolate());
+  v8::Local<v8::Context> context = v8::Context::New(v8_isolate());
+  v8::Context::Scope context_scope(context);
   void* wrappable = cppgc::MakeGarbageCollected<Wrappable>(allocation_handle());
   v8::Local<v8::Object> api_object =
-      WrapperHelper::CreateWrapper(context(), nullptr, nullptr);
+      WrapperHelper::CreateWrapper(context, nullptr, nullptr);
   Wrappable::destructor_callcount = 0;
   WrapperHelper::ResetWrappableConnection(api_object);
   SimulateIncrementalMarking();
@@ -102,8 +105,9 @@ TEST_F(UnifiedHeapTest, WriteBarrierV8ToCppReference) {
 
 TEST_F(UnifiedHeapTest, WriteBarrierCppToV8Reference) {
   if (!FLAG_incremental_marking) return;
-
   v8::HandleScope scope(v8_isolate());
+  v8::Local<v8::Context> context = v8::Context::New(v8_isolate());
+  v8::Context::Scope context_scope(context);
   cppgc::Persistent<Wrappable> wrappable =
       cppgc::MakeGarbageCollected<Wrappable>(allocation_handle());
   Wrappable::destructor_callcount = 0;
@@ -115,7 +119,7 @@ TEST_F(UnifiedHeapTest, WriteBarrierCppToV8Reference) {
     // setter for C++ to JS references.
     v8::HandleScope nested_scope(v8_isolate());
     v8::Local<v8::Object> api_object =
-        WrapperHelper::CreateWrapper(context(), nullptr, nullptr);
+        WrapperHelper::CreateWrapper(context, nullptr, nullptr);
     // Setting only one field to avoid treating this as wrappable backref, see
     // `LocalEmbedderHeapTracer::ExtractWrapperInfo`.
     api_object->SetAlignedPointerInInternalField(1, kMagicAddress);
@@ -144,6 +148,8 @@ class Unreferenced : public cppgc::GarbageCollected<Unreferenced> {
 
 TEST_F(UnifiedHeapTest, FreeUnreferencedDuringNoGcScope) {
   v8::HandleScope handle_scope(v8_isolate());
+  v8::Local<v8::Context> context = v8::Context::New(v8_isolate());
+  v8::Context::Scope context_scope(context);
   auto* unreferenced = cppgc::MakeGarbageCollected<Unreferenced>(
       allocation_handle(),
       cppgc::AdditionalBytes(cppgc::internal::api_constants::kMB));
@@ -172,6 +178,8 @@ TEST_F(UnifiedHeapTest, FreeUnreferencedDuringNoGcScope) {
 #if !V8_OS_FUCHSIA
 TEST_F(UnifiedHeapTest, TracedReferenceRetainsFromStack) {
   v8::HandleScope handle_scope(v8_isolate());
+  v8::Local<v8::Context> context = v8::Context::New(v8_isolate());
+  v8::Context::Scope context_scope(context);
   TracedReference<v8::Object> holder;
   {
     v8::HandleScope inner_handle_scope(v8_isolate());