Chromium Code Reviews| Index: Source/core/workers/WorkerEventQueue.h |
| diff --git a/Source/core/workers/WorkerEventQueue.h b/Source/core/workers/WorkerEventQueue.h |
| index a738f95ec976d2b3d0c7a93205ff238cadb3671b..9fe0ad57cd6e8541811a37156ee4ea86a95a1364 100644 |
| --- a/Source/core/workers/WorkerEventQueue.h |
| +++ b/Source/core/workers/WorkerEventQueue.h |
| @@ -39,12 +39,13 @@ class Event; |
| class Node; |
| class ExecutionContext; |
| -class WorkerEventQueue FINAL : public EventQueue { |
| - WTF_MAKE_FAST_ALLOCATED; |
| +class WorkerEventQueue FINAL : public NoBaseWillBeGarbageCollectedFinalized<WorkerEventQueue>, public EventQueue { |
|
sof
2014/05/30 10:23:29
Does WorkerEventQueue need finalization?
tkent
2014/05/30 13:47:58
I don't think so. I'll update the patch.
tkent
2014/05/30 14:01:23
I found finalization was required.
~EventQueue is
sof
2014/05/30 15:05:41
thanks, it might become possible at some point lat
|
| + WTF_MAKE_FAST_ALLOCATED_WILL_BE_REMOVED; |
| public: |
| - static PassOwnPtr<WorkerEventQueue> create(ExecutionContext*); |
| + static PassOwnPtrWillBeRawPtr<WorkerEventQueue> create(ExecutionContext*); |
| virtual ~WorkerEventQueue(); |
| + void trace(Visitor*); |
| // EventQueue |
| virtual bool enqueueEvent(PassRefPtrWillBeRawPtr<Event>) OVERRIDE; |
| @@ -59,7 +60,7 @@ private: |
| bool m_isClosed; |
| class EventDispatcherTask; |
| - typedef WillBePersistentHeapHashMap<RefPtrWillBeMember<Event>, EventDispatcherTask*> EventTaskMap; |
| + typedef WillBeHeapHashMap<RefPtrWillBeMember<Event>, EventDispatcherTask*> EventTaskMap; |
| EventTaskMap m_eventTaskMap; |
| }; |