Refactor duplicate code into RenderFlexibleBox::mainAxisContentExtent
[WebKit-https.git] / Source / WebCore / ChangeLog
index eb043a7..7584f4a 100644 (file)
@@ -1,3 +1,22 @@
+2012-09-13  Tony Chang  <tony@chromium.org>
+
+        Refactor duplicate code into RenderFlexibleBox::mainAxisContentExtent
+        https://bugs.webkit.org/show_bug.cgi?id=96676
+
+        Reviewed by Eric Seidel.
+
+        There was computeAvailableFreeSpace() and lineBreakLength() shared some very similar code.
+        Merge this code into mainAxisContentExtent().
+
+        No new tests, this is just a refactor.
+
+        * rendering/RenderFlexibleBox.cpp:
+        (WebCore::RenderFlexibleBox::mainAxisContentExtent): Take contentLogicalHeight as a parameter,
+        which is needed when determining the logical height.
+        (WebCore::RenderFlexibleBox::computeAvailableFreeSpace): Use mainAxisContentExtent() and inline.
+        (WebCore::RenderFlexibleBox::computeNextFlexLine): Use mainAxisContentExtent().
+        * rendering/RenderFlexibleBox.h:
+
 2012-09-13  Glenn Hartmann  <hartmanng@chromium.org>
 
         Add new accelerated compositing for overflow scroll setting.