Implement setter/getter for PaintInfo::outlineObjects.

Source/core/rendering/PaintInfo.h

Index: Source/core/rendering/PaintInfo.h
diff --git a/Source/core/rendering/PaintInfo.h b/Source/core/rendering/PaintInfo.h
index 4e348f2238041484146b0cb0f0111be5153017a9..3f7aa80d2bf9946a7abffd7679592b070526a572 100644
--- a/Source/core/rendering/PaintInfo.h
+++ b/Source/core/rendering/PaintInfo.h
@@ -59,9 +59,9 @@ struct PaintInfo {
         , paintBehavior(newPaintBehavior)
         , paintingRoot(newPaintingRoot)
         , renderRegion(region)
-        , outlineObjects(newOutlineObjects)
         , overlapTestRequests(overlapTestRequests)
         , m_paintContainer(newPaintContainer)
+        , m_outlineObjects(newOutlineObjects)
     {
     }
 
@@ -103,6 +103,9 @@ struct PaintInfo {
     static IntRect infiniteRect() { return IntRect(LayoutRect::infiniteRect()); }
     const RenderLayerModelObject* paintContainer() const { return m_paintContainer; }
 
+    ListHashSet<RenderInline*>* outlineObjects() { return m_outlineObjects; }
+    void setOutlineObjects(ListHashSet<RenderInline*>* objects) { m_outlineObjects = objects; }

[eseidel, 2013/08/15 19:36:40]

We never use the setter it seems?

+
     // FIXME: Introduce setters/getters at some point. Requires a lot of changes throughout rendering/.
     GraphicsContext* context;
     IntRect rect;
@@ -110,12 +113,12 @@ struct PaintInfo {
     PaintBehavior paintBehavior;
     RenderObject* paintingRoot; // used to draw just one element and its visual kids
     RenderRegion* renderRegion;
-    ListHashSet<RenderInline*>* outlineObjects; // used to list outlines that should be painted by a block with inline children
     OverlapTestRequestMap* overlapTestRequests;
 
 private:
 
     const RenderLayerModelObject* m_paintContainer; // the layer object that originates the current painting
+    ListHashSet<RenderInline*>* m_outlineObjects; // used to list outlines that should be painted by a block with inline children

[eseidel, 2013/08/15 19:36:40]

Woh.  So we just have a reference to this thing.  It's not actually owned by
this object.

I don't think it helps much to make this one private.

 };
 
 } // namespace WebCore