Oilpan: Reduce FIXME(OILPAN) from the code base

This CL removes FIXME(OILPAN) that is out-dated or doesn't make much sense.

BUG=

Review URL: https://codereview.chromium.org/1327793002

git-svn-id: svn://svn.chromium.org/blink/trunk@201639 bbb929c8-8fbe-4397-9dbb-9b2b20218538

third_party/WebKit/Source/core/css/ElementRuleCollector.h

@@ -62,11 +62,7 @@ public:
     }
 
 private:
-    // FIXME: Oilpan: RuleData is in the oilpan heap and this pointer
-    // really should be traced. However, RuleData objects are
-    // allocated inside larger TerminatedArray objects and we cannot
-    // trace a raw rule data pointer at this point.
-    const RuleData* m_ruleData;
+    RawPtrWillBeMember<const RuleData> m_ruleData;
     unsigned m_specificity;
     uint64_t m_position;
     RawPtrWillBeMember<const CSSStyleSheet> m_parentStyleSheet;

third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceLinearGradient.h

@@ -48,10 +48,6 @@ public:
 
 private:
 #if ENABLE(OILPAN)
-    // FIXME: Oilpan: the LayoutObject hierarchy isn't on the heap, but
-    // the SVG property hierarchy is. And the two meet here, so keep a
-    // persistent reference to the SVG part object by way of a heap-allocated
-    // wrapper object.
     Persistent<LinearGradientAttributesWrapper> m_attributesWrapper;
 
     LinearGradientAttributes& mutableAttributes() { return m_attributesWrapper->attributes(); }

third_party/WebKit/Source/core/svg/graphics/SVGImageChromeClient.cpp

@@ -102,8 +102,7 @@ void SVGImageChromeClient::animationTimerFired(Timer<SVGImageChromeClient>*)
     // below, including this object and its timer. For code simplicity, the
     // object protection isn't made the conditional on Oilpan.
     //
-    // FIXME: Oilpan: move this and other ChromeClients to the Oilpan heap
-    // to render this protection redundant.
+    // TODO(oilpan): move SVGImage to the Oilpan heap and remove this protection.
     RefPtr<SVGImage> protect(m_image);
     m_image->frameView()->page()->animator().serviceScriptedAnimations(monotonicallyIncreasingTime());
     m_image->frameView()->updateAllLifecyclePhases();

third_party/WebKit/Source/platform/heap/Heap.cpp

@@ -574,11 +574,7 @@ void Heap::globalWeakProcessing(Visitor* visitor)
 
 void Heap::collectAllGarbage()
 {
-    // FIXME: Oilpan: we should perform a single GC and everything
-    // should die. Unfortunately it is not the case for all objects
-    // because the hierarchy was not completely moved to the heap and
-    // some heap allocated objects own objects that contain persistents
-    // pointing to other heap allocated objects.
+    // We need to run multiple GCs to collect a chain of persistent handles.
     size_t previousLiveObjects = 0;
     for (int i = 0; i < 5; ++i) {
         collectGarbage(ThreadState::NoHeapPointersOnStack, ThreadState::GCWithSweep, ForcedGC);

third_party/WebKit/Source/wtf/RawPtr.h

@@ -75,7 +75,6 @@ public:
 
     T* get() const { return m_ptr; }
     void clear() { m_ptr = 0; }
-    // FIXME: oilpan: Remove release and leakRef once we remove RefPtrWillBeRawPtr.
     RawPtr<T> release()
     {
         RawPtr<T> tmp = m_ptr;