Eliminate large chunks of nearly duplicated code for table-row-group border calculation.

Eliminate large chunks of nearly duplicated code for table-row-group border calculation.

calcOuterBorderBefore() and calcOuterBorderAfter() were almost identical.
So were calcOuterBorderStart() and calcOuterBorderEnd().

Merge them into calcBlockDirectionOuterBorder() and calcInlineDirectionOuterBorder().

This is just code cleanup; no behavior changes intended.

BUG=none

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

[ojan, 2014-01-24 01:35:13]

lgtm

Please split up the function before committing. No need for another followup
review unless you disagree.

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

https://codereview.chromium.org/145603002/diff/1/Source/core/rendering/Render...
Source/core/rendering/RenderTableSection.cpp:1070: if (side == BorderBefore ||
side == BorderAfter) {
I'd use a separate function for before/after and start/end since they don't
share any code. Not a big deal, but I think it makes the code a bit easier to
follow.

[commit-bot: I haz the power, 2014-01-24 10:31:54]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mstensho@opera.com/145603002/60001

[commit-bot: I haz the power, 2014-01-24 13:48:52]

Change committed as 165746