Changeset 129776 in webkit

Timestamp:

Sep 27, 2012, 10:23:11 AM (13 years ago)

Author:

kling@webkit.org

Message:

332kB below DocumentEventQueue::create() on Membuster3.
<​http://webkit.org/b/97712>

Reviewed by Anders Carlsson.

Give DocumentEventQueue::m_queuedEvents an inline capacity of 16 (the default is 256.)
312kB progression on Membuster3.

• dom/DocumentEventQueue.h:
• dom/DocumentEventQueue.cpp:

(WebCore::DocumentEventQueue::pendingEventTimerFired):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• dom/DocumentEventQueue.cpp (modified) (1 diff)
• dom/DocumentEventQueue.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-09-27  Andreas Kling  <kling@webkit.org>
+
+        332kB below DocumentEventQueue::create() on Membuster3.
+        <http://webkit.org/b/97712>
+
+        Reviewed by Anders Carlsson.
+
+        Give DocumentEventQueue::m_queuedEvents an inline capacity of 16 (the default is 256.)
+        312kB progression on Membuster3.
+
+        * dom/DocumentEventQueue.h:
+        * dom/DocumentEventQueue.cpp:
+        (WebCore::DocumentEventQueue::pendingEventTimerFired):
+
 2012-09-27  Vsevolod Vlasov  <vsevik@chromium.org>
 

trunk/Source/WebCore/dom/DocumentEventQueue.cpp

@@ -149 +149 @@
 
     while (!m_queuedEvents.isEmpty()) {
-        ListHashSet<RefPtr<Event> >::iterator iter = m_queuedEvents.begin();
+        ListHashSet<RefPtr<Event>, 16>::iterator iter = m_queuedEvents.begin();
         RefPtr<Event> event = *iter;
         m_queuedEvents.remove(iter);

trunk/Source/WebCore/dom/DocumentEventQueue.h

@@ -69 +69 @@
 
     OwnPtr<DocumentEventQueueTimer> m_pendingEventTimer;
-    ListHashSet<RefPtr<Event> > m_queuedEvents;
+    ListHashSet<RefPtr<Event>, 16> m_queuedEvents;
     HashSet<Node*> m_nodesWithQueuedScrollEvents;
     bool m_isClosed;