Ignore continuations when collapsing an anonymous block.

Ignore continuations when collapsing an anonymous block.

When we decide to promote all children of an anonymous block and then
nuke it, ignore any continuation set on the new parent of the children,
to make sure that the children are in fact inserted as children of said
new parent, and not somewhere else in the tree.

This is a hack that's needed as long as we keep the old multicol
implementation. The root cause is probably buggy block continuation
handling, but fixing that would be risky.

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

[mstensho (USE GERRIT), 2014-10-15 18:41:56]

mstensho@opera.com changed reviewers:
+ pdr@chromium.org

[pdr., 2014-10-15 22:56:43]

pdr@chromium.org changed reviewers:
+ aarya@google.com, inferno@chromium.org

[pdr., 2014-10-15 22:56:44]

+cc Abhishek, can you please review this?

[inferno, 2014-10-16 06:34:11]

On 2014/10/15 22:56:44, pdr wrote:
> +cc Abhishek, can you please review this?

lgtm. but definitely see and analyze c#13 in bug.

[pdr., 2014-10-16 06:35:51]

The CQ bit was checked by pdr@chromium.org

[commit-bot: I haz the power, 2014-10-16 06:36:18]

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

[pdr., 2014-10-16 06:36:32]

The CQ bit was unchecked by pdr@chromium.org

[mstensho (USE GERRIT), 2014-10-16 07:45:15]

On 2014/10/16 06:34:11, inferno wrote:
> On 2014/10/15 22:56:44, pdr wrote:
> > +cc Abhishek, can you please review this?
> 
> lgtm. but definitely see and analyze c#13 in bug.

Thanks. Read and replied.

[aarya, 2014-10-16 15:37:21]

On 2014/10/16 07:45:15, mstensho wrote:
> On 2014/10/16 06:34:11, inferno wrote:
> > On 2014/10/15 22:56:44, pdr wrote:
> > > +cc Abhishek, can you please review this?
> > 
> > lgtm. but definitely see and analyze c#13 in bug.
> 
> Thanks. Read and replied.

feel free to cq when you get a chance. this bug is hitting a lot on ClusterFuzz
(in different variants), so wanna clear them all.

[mstensho (USE GERRIT), 2014-10-16 15:53:28]

On 2014/10/16 15:37:21, aarya wrote:
> On 2014/10/16 07:45:15, mstensho wrote:
> > On 2014/10/16 06:34:11, inferno wrote:
> > > On 2014/10/15 22:56:44, pdr wrote:
> > > > +cc Abhishek, can you please review this?
> > > 
> > > lgtm. but definitely see and analyze c#13 in bug.
> > 
> > Thanks. Read and replied.
> 
> feel free to cq when you get a chance. this bug is hitting a lot on
ClusterFuzz
> (in different variants), so wanna clear them all.

Sure thing! Just wanted to wait a little and see if people wanted a nicer fix.
But we can always deal with that later.

CQing now.

[mstensho (USE GERRIT), 2014-10-16 15:53:32]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2014-10-16 15:54:19]

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

[commit-bot: I haz the power, 2014-10-16 15:55:25]

Committed patchset #1 (id:1) as 183823