Reduce repetitive EventTarget subclassing

Source/modules/indexeddb/IDBDatabase.cpp

Index: Source/modules/indexeddb/IDBDatabase.cpp
diff --git a/Source/modules/indexeddb/IDBDatabase.cpp b/Source/modules/indexeddb/IDBDatabase.cpp
index fb9989817d0871cdade91e5254caaadc77731bdc..c5c82f33b5336aabf0283b285417be4e8d3984de 100644
--- a/Source/modules/indexeddb/IDBDatabase.cpp
+++ b/Source/modules/indexeddb/IDBDatabase.cpp
@@ -393,7 +393,7 @@ bool IDBDatabase::hasPendingActivity() const
 {
     // The script wrapper must not be collected before the object is closed or
     // we can't fire a "versionchange" event to let script manually close the connection.
-    return !m_closePending && !m_eventTargetData.eventListenerMap.isEmpty() && !m_contextStopped;
+    return !m_closePending && !const_cast<IDBDatabase*>(this)->ensureEventTargetData().eventListenerMap.isEmpty() && !m_contextStopped;

[tkent, 2013/10/10 21:16:25]

!...eventListernMap.isEmpty() looks like equivalent to hasEventListeners().

[jsbell, 2013/10/10 21:49:16]

I was probably ignorant of hasEventListeners when I implemented this, so no
objection to changing it.

 }
 
 void IDBDatabase::stop()
@@ -414,14 +414,4 @@ ScriptExecutionContext* IDBDatabase::scriptExecutionContext() const
     return ActiveDOMObject::scriptExecutionContext();
 }
 
-EventTargetData* IDBDatabase::eventTargetData()
-{
-    return &m_eventTargetData;
-}
-
-EventTargetData* IDBDatabase::ensureEventTargetData()
-{
-    return &m_eventTargetData;
-}
-
 } // namespace WebCore