previousRegionLogicalHeight not getting updated

previousRegionLogicalHeight not getting updated

previousRegionLogicalHeight() was not getting updated in RenderFlowThread::validateRegions() and 
previousRegionLogicalWidth() was updated but not used.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=170589

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

lgtm

+Mihnea as it probably interest him.

Let's remove the empty BUG= from the description (I know that gclient generates
it for you but we don't leave it)

https://codereview.chromium.org/147653002/diff/1/Source/core/rendering/Render...
File Source/core/rendering/RenderFlowThread.cpp (right):

https://codereview.chromium.org/147653002/diff/1/Source/core/rendering/Render...
Source/core/rendering/RenderFlowThread.cpp:186: previousRegionLogicalWidth =
regionLogicalWidth;
This variable is never used so we should remove it!

[mihnea, 2014-03-28 14:56:04]

On 2014/03/26 00:13:43, Julien Chaffraix - PST wrote:
> lgtm
> 
> +Mihnea as it probably interest him.
> 

Thanks a lot Julien :)

> Let's remove the empty BUG= from the description (I know that gclient
generates
> it for you but we don't leave it)
> 
>
https://codereview.chromium.org/147653002/diff/1/Source/core/rendering/Render...
> File Source/core/rendering/RenderFlowThread.cpp (right):
> 
>
https://codereview.chromium.org/147653002/diff/1/Source/core/rendering/Render...
> Source/core/rendering/RenderFlowThread.cpp:186: previousRegionLogicalWidth =
> regionLogicalWidth;
> This variable is never used so we should remove it!

[rhogan, 2014-04-01 17:29:48]

The CQ bit was checked by robhogan@gmail.com

[commit-bot: I haz the power, 2014-04-01 17:29:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/robhogan@gmail.com/147653002/50001

[commit-bot: I haz the power, 2014-04-01 18:16:47]

Change committed as 170589