[oilpan] Move CSSFontFace to the managed heap

Source/core/css/CSSSegmentedFontFace.cpp

 /*
  * Copyright (C) 2008 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 28 matching lines @@
 
 DEFINE_GC_TYPE_MARKER(CSSSegmentedFontFace);
 
 CSSSegmentedFontFace::CSSSegmentedFontFace(CSSFontSelector* fontSelector)
     : m_fontSelector(fontSelector)
 {
 }
 
 CSSSegmentedFontFace::~CSSSegmentedFontFace()
 {
-    dispose();
+    pruneTable();
 }
 
+// FIXME(oilpan): Move CSSFontFaceSource and CSSFontSelector to the managed heap.
+// Then all of CSSFontSelector, CSSSegmentedFontFace, CSSFontFace and CSSFontFaceSource
+// die in the same round of GC, and thus we don't need to call dispose().
 void CSSSegmentedFontFace::dispose()
 {
     pruneTable();
     unsigned size = m_fontFaces.size();
     for (unsigned i = 0; i < size; i++)
-        m_fontFaces[i]->removedFromSegmentedFontFace(Handle<CSSSegmentedFontFace>(this));
-    m_fontFaces.clear();
+        m_fontFaces[i]->dispose();
     m_fontSelector = 0;
 }
 
 void CSSSegmentedFontFace::pruneTable()
 {
     // Make sure the glyph page tree prunes out all uses of this custom font.
     if (m_fontDataTable.isEmpty())
         return;
 
     m_fontDataTable.clear();
 }
 
 bool CSSSegmentedFontFace::isValid() const
 {
     // Valid if at least one font face is valid.
     unsigned size = m_fontFaces.size();
     for (unsigned i = 0; i < size; i++) {
         if (m_fontFaces[i]->isValid())
             return true;
     }
     return false;
 }
 
-void CSSSegmentedFontFace::fontLoaded(CSSFontFace*)
+void CSSSegmentedFontFace::fontLoaded(Handle<CSSFontFace>)
 {
     pruneTable();
 
     if (RuntimeEnabledFeatures::fontLoadEventsEnabled() && !isLoading()) {
         Vector<RefPtr<LoadFontCallback> > callbacks;
         m_callbacks.swap(callbacks);
         for (size_t index = 0; index < callbacks.size(); ++index) {
             if (checkFont())
                 callbacks[index]->notifyLoaded(Handle<CSSSegmentedFontFace>(this));
             else
                 callbacks[index]->notifyError(Handle<CSSSegmentedFontFace>(this));
         }
     }
 }
 
-void CSSSegmentedFontFace::appendFontFace(PassRefPtr<CSSFontFace> fontFace)
+void CSSSegmentedFontFace::appendFontFace(Handle<CSSFontFace> fontFace)
 {
     pruneTable();
     fontFace->addedToSegmentedFontFace(Handle<CSSSegmentedFontFace>(this));
     m_fontFaces.append(fontFace);
 }
 
 static void appendFontDataWithInvalidUnicodeRangeIfLoading(SegmentedFontData* newFontData, PassRefPtr<SimpleFontData> prpFaceFontData, const Vector<CSSFontFace::UnicodeRange>& ranges)
 {
     RefPtr<SimpleFontData> faceFontData = prpFaceFontData;
     if (faceFontData->isLoading()) {
@@ skipping 74 matching lines @@
     if (callback) {
         if (isLoading())
             m_callbacks.append(callback);
         else if (checkFont())
             callback->notifyLoaded(Handle<CSSSegmentedFontFace>(this));
         else
             callback->notifyError(Handle<CSSSegmentedFontFace>(this));
     }
 }
 
+void CSSSegmentedFontFace::accept(Visitor* visitor) const
+{
+    visitor->visit(m_fontFaces);
 }
+
+}