Reduce repetitive EventTarget subclassing

Source/modules/indexeddb/IDBDatabase.cpp

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 375 matching lines @@
             return it->key;
         }
     }
     return IDBObjectStoreMetadata::InvalidId;
 }
 
 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()
 {
     // Stop fires at a deterministic time, so we need to call close in it.
     close();
 
     m_contextStopped = true;
 }
 
 const AtomicString& IDBDatabase::interfaceName() const
 {
     return eventNames().interfaceForIDBDatabase;
 }
 
 ScriptExecutionContext* IDBDatabase::scriptExecutionContext() const
 {
     return ActiveDOMObject::scriptExecutionContext();
 }
 
-EventTargetData* IDBDatabase::eventTargetData()
-{
-    return &m_eventTargetData;
-}
-
-EventTargetData* IDBDatabase::ensureEventTargetData()
-{
-    return &m_eventTargetData;
-}
-
 } // namespace WebCore