[webkit-changes] [158963] trunk/Source/WebCore

Fri, 08 Nov 2013 15:23:44 -0800

Title: [158963] trunk/Source/WebCore
Revision
158963
Author
[email protected]
Date
2013-11-08 15:24:40 -0800 (Fri, 08 Nov 2013)

Log Message

Use references instead of pointers to RenderBlockFlow in FloatingObjects and ComputeFloatOffsetAdapter
https://bugs.webkit.org/show_bug.cgi?id=124074

Reviewed by Sam Weinig.

Just a straightforward conversion from const pointers to const references.

Also, remove unneeded argument from FloatingObjects constructor.

No new tests, no behavior change.

* rendering/FloatingObjects.cpp:
(WebCore::ComputeFloatOffsetAdapter::ComputeFloatOffsetAdapter):
(WebCore::FloatingObjects::findNextFloatLogicalBottomBelow):
(WebCore::FloatingObjects::findNextFloatLogicalBottomBelowForBlock):
(WebCore::FloatingObjects::FloatingObjects):
(WebCore::FloatingObjects::clearLineBoxTreePointers):
(WebCore::FloatingObjects::computePlacedFloatsTree):
(WebCore::shapeInfoForFloat):
(WebCore::::updateOffsetIfNeeded):
(WebCore::::collectIfNeeded):
(WebCore::::heightRemaining):
* rendering/FloatingObjects.h:
* rendering/RenderBlockFlow.cpp:
(WebCore::RenderBlockFlow::createFloatingObjects):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (158962 => 158963)


--- trunk/Source/WebCore/ChangeLog	2013-11-08 23:23:16 UTC (rev 158962)
+++ trunk/Source/WebCore/ChangeLog	2013-11-08 23:24:40 UTC (rev 158963)
@@ -1,3 +1,31 @@
+2013-11-08  Bem Jones-Bey  <[email protected]>
+
+        Use references instead of pointers to RenderBlockFlow in FloatingObjects and ComputeFloatOffsetAdapter
+        https://bugs.webkit.org/show_bug.cgi?id=124074
+
+        Reviewed by Sam Weinig.
+
+        Just a straightforward conversion from const pointers to const references.
+
+        Also, remove unneeded argument from FloatingObjects constructor.
+
+        No new tests, no behavior change.
+
+        * rendering/FloatingObjects.cpp:
+        (WebCore::ComputeFloatOffsetAdapter::ComputeFloatOffsetAdapter):
+        (WebCore::FloatingObjects::findNextFloatLogicalBottomBelow):
+        (WebCore::FloatingObjects::findNextFloatLogicalBottomBelowForBlock):
+        (WebCore::FloatingObjects::FloatingObjects):
+        (WebCore::FloatingObjects::clearLineBoxTreePointers):
+        (WebCore::FloatingObjects::computePlacedFloatsTree):
+        (WebCore::shapeInfoForFloat):
+        (WebCore::::updateOffsetIfNeeded):
+        (WebCore::::collectIfNeeded):
+        (WebCore::::heightRemaining):
+        * rendering/FloatingObjects.h:
+        * rendering/RenderBlockFlow.cpp:
+        (WebCore::RenderBlockFlow::createFloatingObjects):
+
 2013-11-08  Sam Weinig  <[email protected]>
 
         Teach SubframeLoader a bit about references

Modified: trunk/Source/WebCore/rendering/FloatingObjects.cpp (158962 => 158963)


--- trunk/Source/WebCore/rendering/FloatingObjects.cpp	2013-11-08 23:23:16 UTC (rev 158962)
+++ trunk/Source/WebCore/rendering/FloatingObjects.cpp	2013-11-08 23:24:40 UTC (rev 158963)
@@ -125,7 +125,7 @@
 public:
     typedef FloatingObjectInterval IntervalType;
 
-    ComputeFloatOffsetAdapter(const RenderBlockFlow* renderer, LayoutUnit lineTop, LayoutUnit lineBottom, LayoutUnit offset)
+    ComputeFloatOffsetAdapter(const RenderBlockFlow& renderer, LayoutUnit lineTop, LayoutUnit lineBottom, LayoutUnit offset)
         : m_renderer(renderer)
         , m_lineTop(roundToInt(lineTop))
         , m_lineBottom(roundToInt(lineBottom))
@@ -145,7 +145,7 @@
 private:
     bool updateOffsetIfNeeded(const FloatingObject*);
 
-    const RenderBlockFlow* m_renderer;
+    const RenderBlockFlow& m_renderer;
     int m_lineTop;
     int m_lineBottom;
     LayoutUnit m_offset;
@@ -207,7 +207,7 @@
 
 LayoutUnit FloatingObjects::findNextFloatLogicalBottomBelow(LayoutUnit logicalHeight)
 {
-    FindNextFloatLogicalBottomAdapter adapter(*m_renderer, logicalHeight);
+    FindNextFloatLogicalBottomAdapter adapter(m_renderer, logicalHeight);
     placedFloatsTree().allOverlapsWithAdapter(adapter);
 
     return adapter.nextShapeLogicalBottom();
@@ -215,17 +215,17 @@
 
 LayoutUnit FloatingObjects::findNextFloatLogicalBottomBelowForBlock(LayoutUnit logicalHeight)
 {
-    FindNextFloatLogicalBottomAdapter adapter(*m_renderer, logicalHeight);
+    FindNextFloatLogicalBottomAdapter adapter(m_renderer, logicalHeight);
     placedFloatsTree().allOverlapsWithAdapter(adapter);
 
     return adapter.nextLogicalBottom();
 }
 
-FloatingObjects::FloatingObjects(const RenderBlockFlow* renderer, bool horizontalWritingMode)
+FloatingObjects::FloatingObjects(const RenderBlockFlow& renderer)
     : m_placedFloatsTree(UninitializedTree)
     , m_leftObjectsCount(0)
     , m_rightObjectsCount(0)
-    , m_horizontalWritingMode(horizontalWritingMode)
+    , m_horizontalWritingMode(renderer.isHorizontalWritingMode())
     , m_renderer(renderer)
 {
 }
@@ -238,7 +238,7 @@
 {
     // Clear references to originating lines, since the lines are being deleted
     for (auto it = m_set.begin(), end = m_set.end(); it != end; ++it) {
-        ASSERT(!((*it)->originatingLine()) || &((*it)->originatingLine()->renderer()) == m_renderer);
+        ASSERT(!((*it)->originatingLine()) || &((*it)->originatingLine()->renderer()) == &m_renderer);
         (*it)->setOriginatingLine(0);
     }
 }
@@ -341,7 +341,7 @@
     ASSERT(!m_placedFloatsTree.isInitialized());
     if (m_set.isEmpty())
         return;
-    m_placedFloatsTree.initIfNeeded(m_renderer->view().intervalArena());
+    m_placedFloatsTree.initIfNeeded(m_renderer.view().intervalArena());
     for (auto it = m_set.begin(), end = m_set.end(); it != end; ++it) {
         FloatingObject* floatingObject = it->get();
         if (floatingObject->isPlaced())
@@ -357,11 +357,11 @@
 }
 
 #if ENABLE(CSS_SHAPES)
-static inline ShapeOutsideInfo* shapeInfoForFloat(const FloatingObject* floatingObject, const RenderBlockFlow* containingBlock, LayoutUnit lineTop, LayoutUnit lineBottom)
+static inline ShapeOutsideInfo* shapeInfoForFloat(const FloatingObject* floatingObject, const RenderBlockFlow& containingBlock, LayoutUnit lineTop, LayoutUnit lineBottom)
 {
     if (floatingObject) {
         if (ShapeOutsideInfo* shapeOutside = floatingObject->renderer().shapeOutsideInfo()) {
-            shapeOutside->updateDeltasForContainingBlockLine(containingBlock, floatingObject, lineTop, lineBottom - lineTop);
+            shapeOutside->updateDeltasForContainingBlockLine(&containingBlock, floatingObject, lineTop, lineBottom - lineTop);
             return shapeOutside;
         }
     }
@@ -433,7 +433,7 @@
 template<>
 inline bool ComputeFloatOffsetAdapter<FloatingObject::FloatLeft>::updateOffsetIfNeeded(const FloatingObject* floatingObject)
 {
-    LayoutUnit logicalRight = m_renderer->logicalRightForFloat(floatingObject);
+    LayoutUnit logicalRight = m_renderer.logicalRightForFloat(floatingObject);
     if (logicalRight > m_offset) {
         m_offset = logicalRight;
         return true;
@@ -444,7 +444,7 @@
 template<>
 inline bool ComputeFloatOffsetAdapter<FloatingObject::FloatRight>::updateOffsetIfNeeded(const FloatingObject* floatingObject)
 {
-    LayoutUnit logicalLeft = m_renderer->logicalLeftForFloat(floatingObject);
+    LayoutUnit logicalLeft = m_renderer.logicalLeftForFloat(floatingObject);
     if (logicalLeft < m_offset) {
         m_offset = logicalLeft;
         return true;
@@ -461,7 +461,7 @@
 
     // All the objects returned from the tree should be already placed.
     ASSERT(floatingObject->isPlaced());
-    ASSERT(rangesIntersect(m_renderer->pixelSnappedLogicalTopForFloat(floatingObject), m_renderer->pixelSnappedLogicalBottomForFloat(floatingObject), m_lineTop, m_lineBottom));
+    ASSERT(rangesIntersect(m_renderer.pixelSnappedLogicalTopForFloat(floatingObject), m_renderer.pixelSnappedLogicalBottomForFloat(floatingObject), m_lineTop, m_lineBottom));
 
     bool floatIsNewExtreme = updateOffsetIfNeeded(floatingObject);
     if (floatIsNewExtreme)
@@ -471,7 +471,7 @@
 template <FloatingObject::Type FloatTypeValue>
 LayoutUnit ComputeFloatOffsetAdapter<FloatTypeValue>::heightRemaining() const
 {
-    return m_outermostFloat ? m_renderer->logicalBottomForFloat(m_outermostFloat) - m_lineTop : LayoutUnit(1);
+    return m_outermostFloat ? m_renderer.logicalBottomForFloat(m_outermostFloat) - m_lineTop : LayoutUnit(1);
 }
 
 #ifndef NDEBUG

Modified: trunk/Source/WebCore/rendering/FloatingObjects.h (158962 => 158963)


--- trunk/Source/WebCore/rendering/FloatingObjects.h	2013-11-08 23:23:16 UTC (rev 158962)
+++ trunk/Source/WebCore/rendering/FloatingObjects.h	2013-11-08 23:24:40 UTC (rev 158963)
@@ -124,7 +124,7 @@
 class FloatingObjects {
     WTF_MAKE_NONCOPYABLE(FloatingObjects); WTF_MAKE_FAST_ALLOCATED;
 public:
-    FloatingObjects(const RenderBlockFlow*, bool horizontalWritingMode);
+    FloatingObjects(const RenderBlockFlow&);
     ~FloatingObjects();
 
     void clear();
@@ -161,7 +161,7 @@
     unsigned m_leftObjectsCount;
     unsigned m_rightObjectsCount;
     bool m_horizontalWritingMode;
-    const RenderBlockFlow* m_renderer;
+    const RenderBlockFlow& m_renderer;
 };
 
 #ifndef NDEBUG

Modified: trunk/Source/WebCore/rendering/RenderBlockFlow.cpp (158962 => 158963)


--- trunk/Source/WebCore/rendering/RenderBlockFlow.cpp	2013-11-08 23:23:16 UTC (rev 158962)
+++ trunk/Source/WebCore/rendering/RenderBlockFlow.cpp	2013-11-08 23:24:40 UTC (rev 158963)
@@ -1927,7 +1927,7 @@
 
 void RenderBlockFlow::createFloatingObjects()
 {
-    m_floatingObjects = adoptPtr(new FloatingObjects(this, isHorizontalWritingMode()));
+    m_floatingObjects = adoptPtr(new FloatingObjects(*this));
 }
 
 void RenderBlockFlow::removeFloatingObjects()

_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to