There's no class A break point before or after a float.

There's no class A break point before or after a float.

LayoutBox::classABreakPointValue() should only be called at class A break
points. Document the assertion there, and make sure that
needsForcedBreakBefore() only calls it when at in-flow objects.

BUG=619152
R=eae@chromium.org

Committed: https://crrev.com/194b775563c7c3ecff9ba92187aa7ca3d330b303
Cr-Commit-Position: refs/heads/master@{#410703}

[eae, 2016-08-09 13:38:07]

LGTM

Welcome back, we missed you!

[mstensho (USE GERRIT), 2016-08-09 14:10:44]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2016-08-09 14:10:58]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-09 14:18:42]

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

[commit-bot: I haz the power, 2016-08-09 14:18:43]

Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[mstensho (USE GERRIT), 2016-08-09 14:47:09]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2016-08-09 14:47:25]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-09 16:44:56]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-09 16:46:14]

Description was changed from

==========
There's no class A break point before or after a float.

LayoutBox::classABreakPointValue() should only be called at class A break
points. Document the assertion there, and make sure that
needsForcedBreakBefore() only calls it when at in-flow objects.

BUG=619152
R=eae@chromium.org
==========

to

==========
There's no class A break point before or after a float.

LayoutBox::classABreakPointValue() should only be called at class A break
points. Document the assertion there, and make sure that
needsForcedBreakBefore() only calls it when at in-flow objects.

BUG=619152
R=eae@chromium.org

Committed: https://crrev.com/194b775563c7c3ecff9ba92187aa7ca3d330b303
Cr-Commit-Position: refs/heads/master@{#410703}
==========

[commit-bot: I haz the power, 2016-08-09 16:46:15]

Patchset 1 (id:??) landed as
https://crrev.com/194b775563c7c3ecff9ba92187aa7ca3d330b303
Cr-Commit-Position: refs/heads/master@{#410703}