The column balancer should ignore things outside the bounds of the row.

The column balancer should ignore things outside the bounds of the row.

The column balancer examines one row (fragmentainer group) at a time, and it
needs to ignore things that happen at hard or soft column breaks in other rows.
Do some flow thread coordinate bounds checking to avoid being affected by
pagination struts and hard breaks in other rows.

BUG=556481
R=leviw@chromium.org

Committed: https://crrev.com/904d6b10d8b45a9a3c5933e7252c7d2ee31cb608
Cr-Commit-Position: refs/heads/master@{#364040}

[leviw_travelin_and_unemployed, 2015-12-09 01:03:31]

The CQ bit was checked by leviw@chromium.org to run a CQ dry run

[leviw_travelin_and_unemployed, 2015-12-09 01:05:29]

Once again, LGTM :)

[commit-bot: I haz the power, 2015-12-09 01:35:14]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1501053002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1501053002/20001

[commit-bot: I haz the power, 2015-12-09 03:28:22]

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

[commit-bot: I haz the power, 2015-12-09 03:28:23]

Dry run: This issue passed the CQ dry run.

[mstensho (USE GERRIT), 2015-12-09 08:45:12]

On 2015/12/09 01:05:29, leviw wrote:
> Once again, LGTM :)

You unstoppable reviewer, you! I hadn't even published this. But yeah, it was
ready. :) Thanks.

[mstensho (USE GERRIT), 2015-12-09 08:45:19]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2015-12-09 08:47:10]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1501053002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1501053002/20001

[commit-bot: I haz the power, 2015-12-09 11:35:22]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-09 11:36:05]

Description was changed from

==========
The column balancer should ignore things outside the bounds of the row.

The column balancer examines one row (fragmentainer group) at a time, and it
needs to ignore things that happen at hard or soft column breaks in other rows.
Do some flow thread coordinate bounds checking to avoid being affected by
pagination struts and hard breaks in other rows.

BUG=556481
R=leviw@chromium.org
==========

to

==========
The column balancer should ignore things outside the bounds of the row.

The column balancer examines one row (fragmentainer group) at a time, and it
needs to ignore things that happen at hard or soft column breaks in other rows.
Do some flow thread coordinate bounds checking to avoid being affected by
pagination struts and hard breaks in other rows.

BUG=556481
R=leviw@chromium.org

Committed: https://crrev.com/904d6b10d8b45a9a3c5933e7252c7d2ee31cb608
Cr-Commit-Position: refs/heads/master@{#364040}
==========

[commit-bot: I haz the power, 2015-12-09 11:36:06]

Patchset 2 (id:??) landed as
https://crrev.com/904d6b10d8b45a9a3c5933e7252c7d2ee31cb608
Cr-Commit-Position: refs/heads/master@{#364040}

[leviw_travelin_and_unemployed, 2015-12-09 18:39:24]

On 2015/12/09 at 08:45:12, mstensho wrote:
> On 2015/12/09 01:05:29, leviw wrote:
> > Once again, LGTM :)
> 
> You unstoppable reviewer, you! I hadn't even published this. But yeah, it was
ready. :) Thanks.

I realized I was behind for awhile so I was trying to clear my backlog :)

It *looked* ready!

[mstensho (USE GERRIT), 2015-12-09 18:58:08]

On 2015/12/09 18:39:24, leviw wrote:
> On 2015/12/09 at 08:45:12, mstensho wrote:
> > On 2015/12/09 01:05:29, leviw wrote:
> > > Once again, LGTM :)
> > 
> > You unstoppable reviewer, you! I hadn't even published this. But yeah, it
was
> ready. :) Thanks.
> 
> I realized I was behind for awhile so I was trying to clear my backlog :)
> 
> It *looked* ready!

You even cleared your frontlog. This was supposed to be your treat for today,
but you already ate it yesterday, so now I don't know what to do. :)