Log Message
Remove EventListenerMap destructor. <http://webkit.org/b/74096> Reviewed by Darin Adler.
Let the compiler generate ~EventListenerMap(). We only needed it when the hash map was managing raw pointers. We're losing the no-iterators assertion from clear() by doing this, but that was superfluous to begin with - we were just using it to avoid duplicating code. * dom/EventListenerMap.cpp: * dom/EventListenerMap.h:
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (102348 => 102349)
--- trunk/Source/WebCore/ChangeLog 2011-12-08 18:38:53 UTC (rev 102348)
+++ trunk/Source/WebCore/ChangeLog 2011-12-08 18:50:41 UTC (rev 102349)
@@ -1,3 +1,19 @@
+2011-12-08 Andreas Kling <kl...@webkit.org>
+
+ Remove EventListenerMap destructor.
+ <http://webkit.org/b/74096>
+
+ Reviewed by Darin Adler.
+
+ Let the compiler generate ~EventListenerMap(). We only needed it when the hash map
+ was managing raw pointers.
+
+ We're losing the no-iterators assertion from clear() by doing this, but that was
+ superfluous to begin with - we were just using it to avoid duplicating code.
+
+ * dom/EventListenerMap.cpp:
+ * dom/EventListenerMap.h:
+
2011-12-08 Takashi Toyoshima <toyos...@chromium.org>
Provide more specific error description for SocketStreamError.
Modified: trunk/Source/WebCore/dom/EventListenerMap.cpp (102348 => 102349)
--- trunk/Source/WebCore/dom/EventListenerMap.cpp 2011-12-08 18:38:53 UTC (rev 102348)
+++ trunk/Source/WebCore/dom/EventListenerMap.cpp 2011-12-08 18:50:41 UTC (rev 102349)
@@ -50,11 +50,6 @@
{
}
-EventListenerMap::~EventListenerMap()
-{
- clear();
-}
-
bool EventListenerMap::isEmpty() const
{
if (m_hashMap)
Modified: trunk/Source/WebCore/dom/EventListenerMap.h (102348 => 102349)
--- trunk/Source/WebCore/dom/EventListenerMap.h 2011-12-08 18:38:53 UTC (rev 102348)
+++ trunk/Source/WebCore/dom/EventListenerMap.h 2011-12-08 18:50:41 UTC (rev 102349)
@@ -47,7 +47,6 @@
class EventListenerMap {
public:
EventListenerMap();
- ~EventListenerMap();
bool isEmpty() const;
bool contains(const AtomicString& eventType) const;
_______________________________________________ webkit-changes mailing list webkit-changes@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
