Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(70)

Side by Side Diff: Source/core/workers/WorkerEventQueue.h

Issue 303133005: Oilpan: Prpare to move WorkerEventQueue to Oilpan heap. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2010 Google Inc. All Rights Reserved. 2 * Copyright (C) 2010 Google Inc. All Rights Reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 1. Redistributions of source code must retain the above copyright 7 * 1. Redistributions of source code must retain the above copyright
8 * notice, this list of conditions and the following disclaimer. 8 * notice, this list of conditions and the following disclaimer.
9 * 2. Redistributions in binary form must reproduce the above copyright 9 * 2. Redistributions in binary form must reproduce the above copyright
10 * notice, this list of conditions and the following disclaimer in the 10 * notice, this list of conditions and the following disclaimer in the
(...skipping 21 matching lines...) Expand all
32 #include "wtf/HashSet.h" 32 #include "wtf/HashSet.h"
33 #include "wtf/PassOwnPtr.h" 33 #include "wtf/PassOwnPtr.h"
34 #include "wtf/RefCounted.h" 34 #include "wtf/RefCounted.h"
35 35
36 namespace WebCore { 36 namespace WebCore {
37 37
38 class Event; 38 class Event;
39 class Node; 39 class Node;
40 class ExecutionContext; 40 class ExecutionContext;
41 41
42 class WorkerEventQueue FINAL : public EventQueue { 42 class WorkerEventQueue FINAL : public NoBaseWillBeGarbageCollectedFinalized<Work erEventQueue>, 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
43 WTF_MAKE_FAST_ALLOCATED; 43 WTF_MAKE_FAST_ALLOCATED_WILL_BE_REMOVED;
44 public: 44 public:
45 45
46 static PassOwnPtr<WorkerEventQueue> create(ExecutionContext*); 46 static PassOwnPtrWillBeRawPtr<WorkerEventQueue> create(ExecutionContext*);
47 virtual ~WorkerEventQueue(); 47 virtual ~WorkerEventQueue();
48 void trace(Visitor*);
48 49
49 // EventQueue 50 // EventQueue
50 virtual bool enqueueEvent(PassRefPtrWillBeRawPtr<Event>) OVERRIDE; 51 virtual bool enqueueEvent(PassRefPtrWillBeRawPtr<Event>) OVERRIDE;
51 virtual bool cancelEvent(Event*) OVERRIDE; 52 virtual bool cancelEvent(Event*) OVERRIDE;
52 virtual void close() OVERRIDE; 53 virtual void close() OVERRIDE;
53 54
54 private: 55 private:
55 explicit WorkerEventQueue(ExecutionContext*); 56 explicit WorkerEventQueue(ExecutionContext*);
56 void removeEvent(Event*); 57 void removeEvent(Event*);
57 58
58 ExecutionContext* m_executionContext; 59 ExecutionContext* m_executionContext;
59 bool m_isClosed; 60 bool m_isClosed;
60 61
61 class EventDispatcherTask; 62 class EventDispatcherTask;
62 typedef WillBePersistentHeapHashMap<RefPtrWillBeMember<Event>, EventDispatch erTask*> EventTaskMap; 63 typedef WillBeHeapHashMap<RefPtrWillBeMember<Event>, EventDispatcherTask*> E ventTaskMap;
63 EventTaskMap m_eventTaskMap; 64 EventTaskMap m_eventTaskMap;
64 }; 65 };
65 66
66 } 67 }
67 68
68 #endif // WorkerEventQueue_h 69 #endif // WorkerEventQueue_h
OLDNEW
« no previous file with comments | « no previous file | Source/core/workers/WorkerEventQueue.cpp » ('j') | Source/core/workers/WorkerEventQueue.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698