Replace fast/multicol/float-paginate-complex with float-big-line.

Replace fast/multicol/float-paginate-complex with float-big-line.

The test float-paginate-complex adds nothing over the other float-* tests,
except the case with a big line box. That case has been split out into
a new reftest: float-big-line.

R=mstensho@opera.com
BUG=386567
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=184876

[mstensho (USE GERRIT), 2014-11-03 13:45:45]

https://codereview.chromium.org/699593003/diff/20001/LayoutTests/TestExpectat...
File LayoutTests/TestExpectations (left):

https://codereview.chromium.org/699593003/diff/20001/LayoutTests/TestExpectat...
LayoutTests/TestExpectations:1046: crbug.com/350853
fast/multicol/newmulticol/float-paginate-complex.html [ Skip ]
This newmulticol/ test should be removed.

https://codereview.chromium.org/699593003/diff/20001/LayoutTests/fast/multico...
File LayoutTests/fast/multicol/float-big-line.html (right):

https://codereview.chromium.org/699593003/diff/20001/LayoutTests/fast/multico...
LayoutTests/fast/multicol/float-big-line.html:8: height: 400px;
You seem to be making assumptions about the line height here. Better set it
explicitly, maybe?

[andersr, 2014-11-04 11:24:24]

https://codereview.chromium.org/699593003/diff/20001/LayoutTests/TestExpectat...
File LayoutTests/TestExpectations (left):

https://codereview.chromium.org/699593003/diff/20001/LayoutTests/TestExpectat...
LayoutTests/TestExpectations:1046: crbug.com/350853
fast/multicol/newmulticol/float-paginate-complex.html [ Skip ]
On 2014/11/03 13:45:45, mstensho wrote:
> This newmulticol/ test should be removed.

Ah, whoops.

Done.

https://codereview.chromium.org/699593003/diff/20001/LayoutTests/fast/multico...
File LayoutTests/fast/multicol/float-big-line.html (right):

https://codereview.chromium.org/699593003/diff/20001/LayoutTests/fast/multico...
LayoutTests/fast/multicol/float-big-line.html:8: height: 400px;
On 2014/11/03 13:45:45, mstensho wrote:
> You seem to be making assumptions about the line height here. Better set it
> explicitly, maybe?

Done.

[mstensho (USE GERRIT), 2014-11-04 21:29:42]

lgtm with one nit (which applies to all three tests).

https://codereview.chromium.org/699593003/diff/40001/LayoutTests/fast/multico...
File LayoutTests/fast/multicol/float-big-line.html (right):

https://codereview.chromium.org/699593003/diff/40001/LayoutTests/fast/multico...
LayoutTests/fast/multicol/float-big-line.html:13: line-height: initial;
Some explicit line-height here would be nice too. Alternatively, just set the
height.

[andersr, 2014-11-05 15:01:29]

https://codereview.chromium.org/699593003/diff/40001/LayoutTests/fast/multico...
File LayoutTests/fast/multicol/float-big-line.html (right):

https://codereview.chromium.org/699593003/diff/40001/LayoutTests/fast/multico...
LayoutTests/fast/multicol/float-big-line.html:13: line-height: initial;
On 2014/11/04 21:29:42, mstensho wrote:
> Some explicit line-height here would be nice too. Alternatively, just set the
> height.

Done.

[andersr, 2014-11-05 15:01:37]

The CQ bit was checked by andersr@opera.com

[commit-bot: I haz the power, 2014-11-05 15:02:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/699593003/60001

[commit-bot: I haz the power, 2014-11-05 15:06:07]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-11-05 15:06:08]

Try jobs failed on following builders:
  android_chromium_gn_compile_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/android_chromium_gn_comp...)
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/3...)
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/3...)

[andersr, 2014-11-05 15:20:44]

Patchset #5 (id:80001) has been deleted

[andersr, 2014-11-05 15:26:38]

The CQ bit was checked by andersr@opera.com

[commit-bot: I haz the power, 2014-11-05 15:26:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/699593003/100001

[commit-bot: I haz the power, 2014-11-05 16:32:44]

Committed patchset #5 (id:100001) as 184876