parent->addChild() can delete parent and any of its siblings

parent->addChild() can delete parent and any of its siblings

Once you call parent->addChild() your pointers to parent and any floating or
out-of-flow siblings it has are no longer safe.

BUG=602271
Committed: https://crrev.com/741853f64fe49635e5b02b641b3799bdeacec37f
Cr-Commit-Position: refs/heads/master@{#386827}

[rhogan, 2016-04-11 18:15:11]

Description was changed from

==========
parent->addChild() can delete parent and any of its siblings

BUG=602771
==========

to

==========
parent->addChild() can delete parent and any of its siblings

Once you call parent->addChild() your pointers to parent and any floating or
out-of-flow siblings it has are no longer safe.

BUG=602771
==========

[rhogan, 2016-04-11 18:15:11]

robhogan@gmail.com changed reviewers:
+ mstensho@opera.com

[mstensho (USE GERRIT), 2016-04-12 19:36:19]

lgtm, but the bug you're referring to doesn't exist. Please fix that before
landing.

[rhogan, 2016-04-12 19:59:44]

Description was changed from

==========
parent->addChild() can delete parent and any of its siblings

Once you call parent->addChild() your pointers to parent and any floating or
out-of-flow siblings it has are no longer safe.

BUG=602771
==========

to

==========
parent->addChild() can delete parent and any of its siblings

Once you call parent->addChild() your pointers to parent and any floating or
out-of-flow siblings it has are no longer safe.

BUG=602271
==========

[rhogan, 2016-04-12 19:59:48]

The CQ bit was checked by robhogan@gmail.com

[commit-bot: I haz the power, 2016-04-12 20:00:21]

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

[commit-bot: I haz the power, 2016-04-12 20:00:22]

This CL has an open dependency (Issue 1868013002 Patch 40001). Please resolve
the dependency and try again.

[rhogan, 2016-04-12 20:54:04]

The CQ bit was checked by robhogan@gmail.com

[commit-bot: I haz the power, 2016-04-12 20:54:31]

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

[commit-bot: I haz the power, 2016-04-12 22:06:31]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-12 22:08:03]

Description was changed from

==========
parent->addChild() can delete parent and any of its siblings

Once you call parent->addChild() your pointers to parent and any floating or
out-of-flow siblings it has are no longer safe.

BUG=602271
==========

to

==========
parent->addChild() can delete parent and any of its siblings

Once you call parent->addChild() your pointers to parent and any floating or
out-of-flow siblings it has are no longer safe.

BUG=602271

Committed: https://crrev.com/741853f64fe49635e5b02b641b3799bdeacec37f
Cr-Commit-Position: refs/heads/master@{#386827}
==========

[commit-bot: I haz the power, 2016-04-12 22:08:04]

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