Minor Refactor in RenderFlexbox

Minor Refactor in RenderFlexbox

This is a minor refactoring which moves the case of calculation
of cross-size of flex line for single line flex-container from
RenderFlexibleBox::repositionLogicalHeightDependentFlexItems
to RenderFlexibleBox::alignFlexLines which is more appropriate
and saves us from doing unnecessary work.  The old code was doing 
align-content for a single line, then reposition 
everything when aligning children.

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

[harpreet.sk, 2014-06-17 12:38:42]

Minor change. PTAL..

[tony, 2014-06-17 16:11:13]

This looks correct to me.  I would mention in the change description that this
saves us from doing unnecessary work.  The old code would do align-content for a
single line, then reposition everything when aligning children.  Christian or
Ojan should also take a look.

[cbiesinger, 2014-06-17 22:56:15]

lgtm

but I'm not an owner, so tony/ojan will have to review or rubberstamp this.

[tony, 2014-06-17 22:59:23]

On 2014/06/17 22:56:15, cbiesinger wrote:
> lgtm
> 
> but I'm not an owner, so tony/ojan will have to review or rubberstamp this.

I'm not in the owners file either.  Ojan?

[leviw_travelin_and_unemployed, 2014-06-17 23:23:57]

I am. rslgtm.

[harpreet.sk, 2014-06-18 05:13:25]

The CQ bit was checked by harpreet.sk@samsung.com

[commit-bot: I haz the power, 2014-06-18 05:13:40]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/harpreet.sk@samsung.com/340513004/1

[commit-bot: I haz the power, 2014-06-18 06:54:03]

Change committed as 176399