Avoid calling a virtual method on a null document in lifecycle().

Avoid calling a virtual method on a null document in lifecycle().

The bug was found by UBSan Vptr bot.

BUG=667521
Committed: https://crrev.com/ed07d372b9ec31e3b47074e5b95941c52c9c9fca
Cr-Commit-Position: refs/heads/master@{#433688}

[krasin1, 2016-11-21 21:02:49]

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

[commit-bot: I haz the power, 2016-11-21 21:03:43]

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

[krasin1, 2016-11-21 21:04:46]

The CQ bit was unchecked by krasin@chromium.org

[krasin1, 2016-11-21 22:53:34]

krasin@chromium.org changed reviewers:
+ bokan@chromium.org

[bokan, 2016-11-21 22:56:22]

Is there a bug we can put in the BUG= line?

lgtm otherwise.

[krasin1, 2016-11-21 23:13:39]

Description was changed from

==========
Avoid calling a virtual method on a null document in lifecycle().

The bug was found by UBSan Vptr bot.

BUG=
==========

to

==========
Avoid calling a virtual method on a null document in lifecycle().

The bug was found by UBSan Vptr bot.

BUG=667521
==========

[krasin1, 2016-11-21 23:14:52]

On 2016/11/21 22:56:22, bokan wrote:
> Is there a bug we can put in the BUG= line?
> 
> lgtm otherwise.

I have filed https://crbug.com/667521. Generally, it's just a buildbot
regression (may be not very recent, as we had another compile-time issue due to
a bad skia roll last week).

[krasin1, 2016-11-21 23:14:57]

The CQ bit was checked by krasin@chromium.org

[commit-bot: I haz the power, 2016-11-21 23:15:30]

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

[krasin1, 2016-11-21 23:15:48]

And thank you for the instant response!

[commit-bot: I haz the power, 2016-11-21 23:23:27]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1479770097981530, "parent_rev":
"f2eb2cc0ea4ca477d2186c3e654580788c26e725", "commit_rev":
"64315856497e333688b1a089fb67b2f7c7f3ad0d"}

[commit-bot: I haz the power, 2016-11-21 23:23:56]

Description was changed from

==========
Avoid calling a virtual method on a null document in lifecycle().

The bug was found by UBSan Vptr bot.

BUG=667521
==========

to

==========
Avoid calling a virtual method on a null document in lifecycle().

The bug was found by UBSan Vptr bot.

BUG=667521
==========

[commit-bot: I haz the power, 2016-11-21 23:23:57]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-21 23:26:58]

Description was changed from

==========
Avoid calling a virtual method on a null document in lifecycle().

The bug was found by UBSan Vptr bot.

BUG=667521
==========

to

==========
Avoid calling a virtual method on a null document in lifecycle().

The bug was found by UBSan Vptr bot.

BUG=667521

Committed: https://crrev.com/ed07d372b9ec31e3b47074e5b95941c52c9c9fca
Cr-Commit-Position: refs/heads/master@{#433688}
==========

[commit-bot: I haz the power, 2016-11-21 23:26:59]

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