Revert "cppgc: Add tests for in-construction during ctor"

This reverts commit 62ff82e4.

Reason for revert: compilation failure: https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Win64%20-%20msvc/b8850484852067442832/overview

Original change's description:
> cppgc: Add tests for in-construction during ctor
>
> Adds explicit tests that check that an object is marked as in
> construction while running the constructor.
>
> Bug: chromium:1056170
> Change-Id: I7f7340832e1bc31cec98784c261ed86deb402e72
> Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2811238
> Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
> Reviewed-by: Omer Katz <omerkatz@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#73869}

Bug: chromium:1056170
Change-Id: If9e75ab6da3762553e8e34e851cbf1120cd49925
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2816100
Auto-Submit: Francis McCabe <fgm@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/master@{#73870}

test/unittests/heap/cppgc/garbage-collected-unittest.cc

@@ -156,86 +156,5 @@ TEST_F(GarbageCollectedTestWithHeap, PostConstructionCallbackForMixin) {
   EXPECT_EQ(1u, MixinWithPostConstructionCallback::cb_callcount);
 }
 
-namespace {
-
-int GetDummyValue() {
-  static std::mutex mutex;
-  static int ret = 43;
-  // Global lock access to avoid reordering.
-  std::lock_guard<std::mutex> guard(mutex);
-  return ret;
-}
-
-class CheckObjectInConstructionBeforeInitializerList final
-    : public GarbageCollected<CheckObjectInConstructionBeforeInitializerList> {
- public:
-  CheckObjectInConstructionBeforeInitializerList()
-      : in_construction_before_initializer_list_(
-            HeapObjectHeader::FromPayload(this).IsInConstruction()),
-        unused_int_(GetDummyValue()) {
-    EXPECT_TRUE(in_construction_before_initializer_list_);
-    EXPECT_TRUE(HeapObjectHeader::FromPayload(this).IsInConstruction());
-  }
-
-  void Trace(Visitor*) const {}
-
- private:
-  bool in_construction_before_initializer_list_;
-  int unused_int_;
-};
-
-class CheckMixinInConstructionBeforeInitializerList
-    : public GarbageCollectedMixin {
- public:
-  explicit CheckMixinInConstructionBeforeInitializerList(void* payload_start)
-      : in_construction_before_initializer_list_(
-            HeapObjectHeader::FromPayload(payload_start).IsInConstruction()),
-        unused_int_(GetDummyValue()) {
-    EXPECT_TRUE(in_construction_before_initializer_list_);
-    EXPECT_TRUE(
-        HeapObjectHeader::FromPayload(payload_start).IsInConstruction());
-  }
-
-  void Trace(Visitor*) const override {}
-
- private:
-  bool in_construction_before_initializer_list_;
-  int unused_int_;
-};
-
-class UnmanagedMixinForcingVTable {
- protected:
-  virtual void ForceVTable() {}
-};
-
-class CheckGCedWithMixinInConstructionBeforeInitializerList
-    : public GarbageCollected<
-          CheckGCedWithMixinInConstructionBeforeInitializerList>,
-      public UnmanagedMixinForcingVTable,
-      public CheckMixinInConstructionBeforeInitializerList {
- public:
-  CheckGCedWithMixinInConstructionBeforeInitializerList()
-      : CheckMixinInConstructionBeforeInitializerList(this) {
-    // Ensure that compiler indeed generated an inner object.
-    CHECK_NE(
-        this,
-        static_cast<void*>(
-            static_cast<CheckMixinInConstructionBeforeInitializerList*>(this)));
-  }
-};
-
-}  // namespace
-
-TEST_F(GarbageCollectedTestWithHeap, GarbageCollectedInConstructionDuringCtor) {
-  MakeGarbageCollected<CheckObjectInConstructionBeforeInitializerList>(
-      GetAllocationHandle());
-}
-
-TEST_F(GarbageCollectedTestWithHeap,
-       GarbageCollectedMixinInConstructionDuringCtor) {
-  MakeGarbageCollected<CheckGCedWithMixinInConstructionBeforeInitializerList>(
-      GetAllocationHandle());
-}
-
 }  // namespace internal
 }  // namespace cppgc