previousRegionLogicalHeight not getting updated

Source/core/rendering/RenderFlowThread.cpp

 /*
  * Copyright (C) 2011 Adobe Systems Incorporated. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1. Redistributions of source code must retain the above
  *    copyright notice, this list of conditions and the following
  *    disclaimer.
@@ skipping 165 matching lines @@
 
                 if (!firstRegionVisited) {
                     firstRegionVisited = true;
                 } else {
                     if (m_regionsHaveUniformLogicalWidth && previousRegionLogicalWidth != regionLogicalWidth)
                         m_regionsHaveUniformLogicalWidth = false;
                     if (m_regionsHaveUniformLogicalHeight && previousRegionLogicalHeight != regionLogicalHeight)
                         m_regionsHaveUniformLogicalHeight = false;
                 }
 
                 previousRegionLogicalWidth = regionLogicalWidth;

[Julien - ping for review, 2014/03/26 00:13:44]

This variable is never used so we should remove it!

+                previousRegionLogicalHeight = regionLogicalHeight;
             }
         }
     }
 
     updateLogicalWidth(); // Called to get the maximum logical width for the region.
     updateRegionsFlowThreadPortionRect();
 }
 
 void RenderFlowThread::layout()
 {
@@ skipping 978 matching lines @@
 {
     if (!m_renderFlowThread)
         return;
     RenderView* view = m_renderFlowThread->view();
     ASSERT(view->flowThreadController()->currentRenderFlowThread() == m_renderFlowThread);
     view->flowThreadController()->setCurrentRenderFlowThread(m_previousRenderFlowThread);
 }
 
 
 } // namespace WebCore