Floats following inlines break to next line unncessarily

Floats following inlines break to next line unncessarily

Don't include trailing space in linewidth calculations when deciding
if a float can fit on a line.

Merge http://trac.webkit.org/changeset/148622

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

[rhogan, 2014-06-24 19:33:58]

This is a WebKit line-layout patch I meant to merge a long time ago. There's a
couple after it too, but they depend on this one.
 It came up on www-style,
http://lists.w3.org/Archives/Public/www-style/2014Jun/0232.html, so that's what
reminded me of it.

[leviw_travelin_and_unemployed, 2014-06-24 20:07:46]

Close with a couple nits.

https://codereview.chromium.org/343663004/diff/80001/Source/core/rendering/li...
File Source/core/rendering/line/LineWidth.h (right):

https://codereview.chromium.org/343663004/diff/80001/Source/core/rendering/li...
Source/core/rendering/line/LineWidth.h:50: bool fitsOnLine(float extra, bool
excludeWhitespace) const { return currentWidth() - (excludeWhitespace ?
trailingWhitespaceWidth() : 0) + extra <= m_availableWidth; }
I notice you don't include epsilon here like the above code. That seems wrong
(though maybe we don't need the epsilon anymore?).

Also, can we avoid having a boolean parameter?

[leviw_travelin_and_unemployed, 2014-06-27 19:02:13]

lgtm

[commit-bot: I haz the power, 2014-06-27 19:03:21]

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

[commit-bot: I haz the power, 2014-06-27 19:52:32]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/14173)

[commit-bot: I haz the power, 2014-06-27 20:56:53]

Change committed as 177135