Oilpan: move TextFinder to the heap.

Source/web/WebLocalFrameImpl.cpp

 /*
  * Copyright (C) 2009 Google 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 1519 matching lines @@
 {
     Platform::current()->incrementStatsCounter(webFrameActiveCount);
     frameCount++;
 }
 
 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);
     WebFrame::traceFrames(visitor, this);
 }
 #endif
 
 void WebLocalFrameImpl::setCoreFrame(PassRefPtrWillBeRawPtr<LocalFrame> frame)
 {
     m_frame = frame;
@@ skipping 388 matching lines @@
 
 void WebLocalFrameImpl::invalidateAll() const
 {
     ASSERT(frame() && frame()->view());
     FrameView* view = frame()->view();
     view->invalidateRect(view->frameRect());
     invalidateScrollbar();
 }
 
 } // namespace blink