Oilpan: add transition types to track interface objects.

Source/core/html/track/TextTrackList.h

Index: Source/core/html/track/TextTrackList.h
diff --git a/Source/core/html/track/TextTrackList.h b/Source/core/html/track/TextTrackList.h
index bc70584e39e576c5a437b89ffbea9c9206d14bcd..ca97fb1e6942675a899175423e53b9b3f0d5d77a 100644
--- a/Source/core/html/track/TextTrackList.h
+++ b/Source/core/html/track/TextTrackList.h
@@ -31,6 +31,7 @@
 #include "core/events/EventTarget.h"
 #include "core/html/HTMLMediaElement.h"
 #include "platform/Timer.h"
+#include "platform/heap/Handle.h"
 #include "wtf/PassRefPtr.h"
 #include "wtf/RefCounted.h"
 #include "wtf/Vector.h"
@@ -40,12 +41,12 @@ namespace WebCore {
 class GenericEventQueue;
 class TextTrack;
 
-class TextTrackList FINAL : public RefCounted<TextTrackList>, public ScriptWrappable, public EventTargetWithInlineData {
-    REFCOUNTED_EVENT_TARGET(TextTrackList);
+class TextTrackList FINAL : public RefCountedWillBeRefCountedGarbageCollected<TextTrackList>, public ScriptWrappable, public EventTargetWithInlineData {
+    DEFINE_EVENT_TARGET_REFCOUNTING(RefCountedWillBeRefCountedGarbageCollected<TextTrackList>);
 public:
-    static PassRefPtr<TextTrackList> create(HTMLMediaElement* owner)
+    static PassRefPtrWillBeRawPtr<TextTrackList> create(HTMLMediaElement* owner)
     {
-        return adoptRef(new TextTrackList(owner));
+        return adoptRefWillBeRefCountedGarbageCollected(new TextTrackList(owner));
     }
     virtual ~TextTrackList();
 
@@ -56,7 +57,7 @@ public:
 
     TextTrack* item(unsigned index);
     TextTrack* getTrackById(const AtomicString& id);
-    void append(PassRefPtr<TextTrack>);
+    void append(PassRefPtrWillBeRawPtr<TextTrack>);
     void remove(TextTrack*);
 
     // EventTarget
@@ -67,29 +68,33 @@ public:
     DEFINE_ATTRIBUTE_EVENT_LISTENER(change);
     DEFINE_ATTRIBUTE_EVENT_LISTENER(removetrack);
 
+#if !ENABLE(OILPAN)
     void clearOwner();
+#endif
     HTMLMediaElement* owner() const;
 
     void scheduleChangeEvent();
     void removeAllInbandTracks();
 
+    void trace(Visitor*);
+
 private:
     explicit TextTrackList(HTMLMediaElement*);
 
-    void scheduleTrackEvent(const AtomicString& eventName, PassRefPtr<TextTrack>);
+    void scheduleTrackEvent(const AtomicString& eventName, PassRefPtrWillBeRawPtr<TextTrack>);
 
-    void scheduleAddTrackEvent(PassRefPtr<TextTrack>);
-    void scheduleRemoveTrackEvent(PassRefPtr<TextTrack>);
+    void scheduleAddTrackEvent(PassRefPtrWillBeRawPtr<TextTrack>);
+    void scheduleRemoveTrackEvent(PassRefPtrWillBeRawPtr<TextTrack>);
 
     void invalidateTrackIndexesAfterTrack(TextTrack*);
 
-    HTMLMediaElement* m_owner;
+    RawPtrWillBeMember<HTMLMediaElement> m_owner;

[haraken, 2014/04/23 01:51:45]

Shouldn't this be a WeakMember?

[sof, 2014/04/23 05:38:47]

No. 

See https://code.google.com/p/chromium/issues/detail?id=365260

[haraken, 2014/04/23 05:53:54]

Ah, makes sense. This is implemented as a weak member in the current non-oilpan
implementation, but that is wrong. So you're fixing the wrong behavior in the
oilpan implementation. Am I understanding correctly?

Then do you need to add some tests to OilpanTestExpectations or add a new test
for this behavior change?

[sof, 2014/04/23 07:36:48]

We need to address that crasher in some way for the non-Oilpan case. Perhaps we
can handle the extra tests via it?

[haraken, 2014/04/23 07:45:26]

Probably you can add the test to this CL, and then mark it as [CRASH] in
TestExpectation but mark nothing in OilpanTestExpectation?

It sounds nicer if you could fix the crash in the non-oilpan case, but it won't
be worth spending a lot of time on it.

 
     OwnPtr<GenericEventQueue> m_asyncEventQueue;
 
-    Vector<RefPtr<TextTrack> > m_addTrackTracks;
-    Vector<RefPtr<TextTrack> > m_elementTracks;
-    Vector<RefPtr<TextTrack> > m_inbandTracks;
+    WillBeHeapVector<RefPtrWillBeMember<TextTrack> > m_addTrackTracks;
+    WillBeHeapVector<RefPtrWillBeMember<TextTrack> > m_elementTracks;
+    WillBeHeapVector<RefPtrWillBeMember<TextTrack> > m_inbandTracks;
 };
 
 } // namespace WebCore