Mark private method layoutBlockFlow as inline

Mark private method layoutBlockFlow as inline

Mark private method layoutBlockFlow as inline since it is only used in
the implementation file and this appears to be performance critical code.  This
is an attempt to fix a regression introduced by r165254.  This also reduces the
binary size on clang by 40 bytes.

BUG=33574
R=esprehn@chromium.org, ch.dumez@samsung.com

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

[atreat, 2014-01-20 18:53:05]

Please have a look.

[Inactive, 2014-01-20 18:57:49]

lgtm

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

https://codereview.chromium.org/140323004/diff/1/Source/core/rendering/Render...
Source/core/rendering/RenderBlockFlow.cpp:254: inline void
RenderBlockFlow::layoutBlockFlow(bool relayoutChildren, LayoutUnit
pageLogicalHeight)
Yes, looking at the original patch, it seems like the extra function call in the
layoutBlock() virtual function is the only difference that might impact
performance.
Hopefully, inlining it will fix the regression.

[commit-bot: I haz the power, 2014-01-20 18:58:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/adam.treat@samsung.com/140323004/1

[Inactive, 2014-01-20 19:10:47]

On 2014/01/20 18:58:02, I haz the power (commit-bot) wrote:
> CQ is trying da patch. Follow status at
> https://chromium-status.appspot.com/cq/adam.treat%40samsung.com/140323004/1

Fingers crossed! If it doesn't fix it, then we'll likely have to revert both
patches as we have no hardware to debug mac-specific regressions :/

[atreat, 2014-01-20 19:14:20]

On 2014/01/20 19:10:47, Chris Dumez wrote:
> On 2014/01/20 18:58:02, I haz the power (commit-bot) wrote:
> > CQ is trying da patch. Follow status at
> >
https://chromium-status.appspot.com/cq/adam.treat%2540samsung.com/140323004/1
> 
> Fingers crossed! If it doesn't fix it, then we'll likely have to revert both
> patches as we have no hardware to debug mac-specific regressions :/

Even worse, we can't run those tests as non-googlers.

[commit-bot: I haz the power, 2014-01-20 20:08:54]

Change committed as 165419