[oilpan] Move CSSFontFace and CSSSegmentedFontFace to the managed heap

Source/core/css/CSSFontFace.cpp

Index: Source/core/css/CSSFontFace.cpp
diff --git a/Source/core/css/CSSFontFace.cpp b/Source/core/css/CSSFontFace.cpp
index f0e611432ee4eb38e6ec2029bc361538a9edd467..074ac120eceb5922e085e211e5c25d4b89f0b196 100644
--- a/Source/core/css/CSSFontFace.cpp
+++ b/Source/core/css/CSSFontFace.cpp
@@ -37,6 +37,8 @@
 
 namespace WebCore {
 
+DEFINE_GC_TYPE_MARKER(CSSFontFace);
+
 bool CSSFontFace::isLoaded() const
 {
     size_t size = m_sources.size();
@@ -57,19 +59,19 @@ bool CSSFontFace::isValid() const
     return false;
 }
 
-void CSSFontFace::addedToSegmentedFontFace(CSSSegmentedFontFace* segmentedFontFace)
+void CSSFontFace::addedToSegmentedFontFace(Handle<CSSSegmentedFontFace> segmentedFontFace)
 {
     m_segmentedFontFaces.add(segmentedFontFace);
 }
 
-void CSSFontFace::removedFromSegmentedFontFace(CSSSegmentedFontFace* segmentedFontFace)
+void CSSFontFace::removedFromSegmentedFontFace(Handle<CSSSegmentedFontFace> segmentedFontFace)
 {
     m_segmentedFontFaces.remove(segmentedFontFace);
 }
 
 void CSSFontFace::addSource(PassOwnPtr<CSSFontFaceSource> source)
 {
-    source->setFontFace(this);
+    source->setFontFace(Handle<CSSFontFace>(this));
     m_sources.append(source);
 }
 
@@ -96,9 +98,9 @@ void CSSFontFace::fontLoaded(CSSFontFaceSource* source)
             notifyFontLoader(Error);
     }
 
-    HashSet<CSSSegmentedFontFace*>::iterator end = m_segmentedFontFaces.end();
-    for (HashSet<CSSSegmentedFontFace*>::iterator it = m_segmentedFontFaces.begin(); it != end; ++it)
-        (*it)->fontLoaded(this);
+    HashSet<Member<CSSSegmentedFontFace> >::iterator end = m_segmentedFontFaces.end();
+    for (HashSet<Member<CSSSegmentedFontFace> >::iterator it = m_segmentedFontFaces.begin(); it != end; ++it)
+        (*it)->fontLoaded(Handle<CSSFontFace>(this));
 }
 
 PassRefPtr<SimpleFontData> CSSFontFace::getFontData(const FontDescription& fontDescription, bool syntheticBold, bool syntheticItalic)
@@ -163,4 +165,11 @@ bool CSSFontFace::hasSVGFontFaceSource() const
 }
 #endif
 
+void CSSFontFace::accept(Visitor* visitor) const
+{
+    visitor->visit(m_rule);
+    for (HashSet<Member<CSSSegmentedFontFace> >::const_iterator it = m_segmentedFontFaces.begin(); it != m_segmentedFontFaces.end(); ++it)

[Mads Ager (chromium), 2013/07/02 14:59:23]

Let's add this one in visitor next to visit(Vector<Member<T> >) so that we do
not have to repeat this loop for other hash sets and can just write
visitor->visit(m_segmentedFontFaces) here.

[haraken, 2013/07/03 04:39:20]

Done.

+        visitor->visit(*it);
+}
+
 }