[Oilpan]: Moving the FontSelector/FontCacheClient, CSSSegmentedFontFace, and FontFaceCache to the o…

Source/core/css/CSSFontSelector.cpp

Index: Source/core/css/CSSFontSelector.cpp
diff --git a/Source/core/css/CSSFontSelector.cpp b/Source/core/css/CSSFontSelector.cpp
index 5ff2d00ac56d4a8157bfb853f548a7dcb59d96c0..08e1a099e832157abd7594e5af4a0249e71a7d90 100644
--- a/Source/core/css/CSSFontSelector.cpp
+++ b/Source/core/css/CSSFontSelector.cpp
@@ -227,4 +227,10 @@ void CSSFontSelector::updateGenericFontFamilySettings(Document& document)
     m_genericFontFamilySettings = document.settings()->genericFontFamilySettings();
 }
 
+void CSSFontSelector::trace(Visitor* visitor)
+{
+    visitor->trace(m_fontFaceCache);

[haraken, 2014/04/07 14:10:12]

You need to trace m_fontLoader. I hope the plugin catches the error :)

[wibling-chromium, 2014/04/08 09:35:59]

It didn't. Thanks for catching this.
I just talked to Ian and the plugin will complain only if there is a member
field or superclass in the object that needs tracing which is not the case here.
The reason to not complain in this case was that it became close to impossible
since an object could have a trace method without it being called in e.g. the
non-oilpan build.

+    visitor->trace(m_clients);
+}
+
 }