Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(513)

Unified Diff: Source/core/dom/CustomEvent.cpp

Issue 17063016: Remove leak of objects between isolated worlds on custom events. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Added GC test Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/dom/CustomEvent.cpp
diff --git a/Source/core/dom/CustomEvent.cpp b/Source/core/dom/CustomEvent.cpp
index c7e67b3eb97a9e9f44106767f0457f359c2fa9ab..b545d1b844835a1d12d0b776941497d91e2cf0dd 100644
--- a/Source/core/dom/CustomEvent.cpp
+++ b/Source/core/dom/CustomEvent.cpp
@@ -41,7 +41,7 @@ CustomEvent::CustomEvent()
CustomEvent::CustomEvent(const AtomicString& type, const CustomEventInit& initializer)
: Event(type, initializer)
- , m_detail(initializer.detail)
+ , m_detailIsSet(initializer.detailIsSet)
{
ScriptWrappable::init(this);
}
@@ -50,20 +50,8 @@ CustomEvent::~CustomEvent()
{
}
-void CustomEvent::initCustomEvent(const AtomicString& type, bool canBubble, bool cancelable, const ScriptValue& detail)
-{
- ASSERT(!m_serializedScriptValue.get());
- if (dispatched())
- return;
-
- initEvent(type, canBubble, cancelable);
-
- m_detail = detail;
-}
-
void CustomEvent::initCustomEvent(const AtomicString& type, bool canBubble, bool cancelable, PassRefPtr<SerializedScriptValue> serializedScriptValue)
{
- ASSERT(m_detail.hasNoValue());
if (dispatched())
return;

Powered by Google App Engine
This is Rietveld 408576698