Oilpan: Replace EAGERLY_FINALIZE in EventTarget's hierarchy with pre-finalizers

third_party/WebKit/Source/core/events/EventTarget.h

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
  * Copyright (C) 2006 Alexey Proskuryakov (ap@webkit.org)
  *           (C) 2007, 2008 Nikolas Zimmermann <zimmermann@kde.org>
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ skipping 205 matching lines @@
 // EventTargetWithInlineData above.
 //
 // This class template exists to circumvent Oilpan's "leftmost class rule", where the Oilpan classes must come first in
 // the base class list to avoid memory offset adjustment. In non-Oilpan world, RefCountedGarbageCollected<T> must come
 // first, but in Oilpan world EventTargetWithInlineData needs to come first. This class templates does the required
 // #if-switch here, in order to avoid a lot of "#if ENABLE(OILPAN)"-s sprinkled in the derived classes.
 #if ENABLE(OILPAN)
 template <typename T>
 class RefCountedGarbageCollectedEventTargetWithInlineData : public EventTargetWithInlineData {
 public:
-    GC_PLUGIN_IGNORE("491488")
-    void* operator new(size_t size)
-    {
-        // If T is eagerly finalized, it needs to be allocated accordingly.
-        // Redefinition of the operator is needed to accomplish that, as otherwise
-        // it would be allocated using GarbageCollected<EventTarget>'s operator new.
-        // EventTarget is not eagerly finalized.
-        return allocateObject(size, IsEagerlyFinalizedType<T>::value);
-    }
-
     DEFINE_INLINE_VIRTUAL_TRACE()
     {
+        static_assert(!IsEagerlyFinalizedType<T>::value, "EventTargets do not support eager finalization.");
         EventTargetWithInlineData::trace(visitor);
     }
 };
 #else
 template <typename T>
 class RefCountedGarbageCollectedEventTargetWithInlineData : public RefCountedGarbageCollected<T>, public EventTargetWithInlineData {
 public:
     DEFINE_INLINE_VIRTUAL_TRACE() { EventTargetWithInlineData::trace(visitor); }
 };
 #endif
@@ skipping 73 matching lines @@
     using baseClass::deref; \
 private: \
     void refEventTarget() final { ref(); } \
     void derefEventTarget() final { deref(); } \
     typedef int thisIsHereToForceASemiColonAfterThisEventTargetMacro
 #define REFCOUNTED_EVENT_TARGET(baseClass) DEFINE_EVENT_TARGET_REFCOUNTING(RefCounted<baseClass>)
 #define REFCOUNTED_GARBAGE_COLLECTED_EVENT_TARGET(baseClass) DEFINE_EVENT_TARGET_REFCOUNTING(RefCountedGarbageCollected<baseClass>)
 #endif // ENABLE(OILPAN)
 
 #endif // EventTarget_h