Oilpan: move TextFinder to the heap.

Source/web/WebLocalFrameImpl.cpp

Index: Source/web/WebLocalFrameImpl.cpp
diff --git a/Source/web/WebLocalFrameImpl.cpp b/Source/web/WebLocalFrameImpl.cpp
index 2abeb0c510ca064da97fdbc9bf836a31ed26c4b4..d6842ae647fc39e6479030ac9e128cd0cdb208d3 100644
--- a/Source/web/WebLocalFrameImpl.cpp
+++ b/Source/web/WebLocalFrameImpl.cpp
@@ -1537,13 +1537,16 @@ WebLocalFrameImpl::~WebLocalFrameImpl()
     Platform::current()->decrementStatsCounter(webFrameActiveCount);
     frameCount--;
 
+#if !ENABLE(OILPAN)
     cancelPendingScopingEffort();

[haraken, 2014/10/24 01:18:21]

Just to confirm: It is ok not to call cancelPendingScopingEffort() (which
changes TextFinder::m_activeMatchIndexInCurrentFrame etc) because the
WebLocalFrameImpl and TextFinder die together, right?

[sof, 2014/10/24 05:32:00]

Correct - the TextFinder is owned&controlled by WebLocalFrameImpl; they die
together. 

+#endif
 }
 
 #if ENABLE(OILPAN)
 void WebLocalFrameImpl::trace(Visitor* visitor)
 {
     visitor->trace(m_frame);
+    visitor->trace(m_textFinder);
     visitor->trace(m_printContext);
     visitor->trace(m_geolocationClientProxy);
     visitor->registerWeakMembers<WebFrame, &WebFrame::clearWeakFrames>(this);