[oilpan] Move CSSFontFace to the managed heap

Source/core/css/CSSFontFaceSource.h

 /*
  * Copyright (C) 2007, 2008, 2011 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 37 matching lines @@
 class CSSFontFaceSource : public CachedFontClient {
 public:
     CSSFontFaceSource(const String&, CachedFont* = 0);
     virtual ~CSSFontFaceSource();
 
     bool isLoaded() const;
     bool isValid() const;
 
     const AtomicString& string() const { return m_string; }
 
-    void setFontFace(CSSFontFace* face) { m_face = face; }
+    void setFontFace(Handle<CSSFontFace> face) { m_face = face.raw(); }
 
     virtual void fontLoaded(CachedFont*);
 
     PassRefPtr<SimpleFontData> getFontData(const FontDescription&, bool syntheticBold, bool syntheticItalic, CSSFontSelector*);
 
     void pruneTable();
 
 #if ENABLE(SVG_FONTS)
     SVGFontFaceElement* svgFontFaceElement() const;
     void setSVGFontFaceElement(PassRefPtr<SVGFontFaceElement>);
     bool isSVGFontFaceSource() const;
     void setHasExternalSVGFont(bool value) { m_hasExternalSVGFont = value; }
 #endif
 
     bool isDecodeError() const;
     bool ensureFontData();
 
 private:
     void startLoadingTimerFired(Timer<CSSFontFaceSource>*);
 
     AtomicString m_string; // URI for remote, built-in font name for local.
     CachedResourceHandle<CachedFont> m_font; // For remote fonts, a pointer to our cached resource.
+    // FIXME(oilpan): Move CSSFontFaceSource to the managed heap and change this to a Member.
     CSSFontFace* m_face; // Our owning font face.

[Vyacheslav Egorov (Google), 2013/07/10 10:59:58]

I think you can make it a member right away. There is nothing actually
forbidding to have Members in not heap allocated structures.

[haraken, 2013/07/10 11:38:41]

My concern is that using Members in not heap allocated structures will confuse a
verification framework where we want to verify that all Members are visited.
Even without the verification framework, I might want to have a general rule
that all Members have to be visited in order to manually verify that Members are
correctly used.

     HashMap<unsigned, RefPtr<SimpleFontData> > m_fontDataTable; // The hash key is composed of size synthetic styles.
 
 #if ENABLE(SVG_FONTS)
     RefPtr<SVGFontFaceElement> m_svgFontFaceElement;
     RefPtr<SVGFontElement> m_externalSVGFontElement;
     bool m_hasExternalSVGFont;
 #endif
 };
 
 }
 
 #endif