Don't call DirtyStackingContextZOrderLists during document shutdown

Don't call DirtyStackingContextZOrderLists during document shutdown

DirtyStackingContextZOrderLists may try to call LayoutView::Compositor(),
which may be invalid during such times. In any case, DirtyStackingContextZOrderLists
is irrelevant during document shutdown.

BUG=715736
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2840403002
Cr-Commit-Position: refs/heads/master@{#467554}
Committed: https://chromium.googlesource.com/chromium/src/+/d846315f83096c9b86e9206d901e56fa1190710a

[chrishtr, 2017-04-26 23:20:40]

Description was changed from

==========
none

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

to

==========
none

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[chrishtr, 2017-04-26 23:23:29]

Description was changed from

==========
none

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Don't call DirtyStackingContextZOrderLists during document shutdown

DirtyStackingContextZOrderLists may try to call LayoutView::Compositor(),
which may be invalid during such times. In any case,
DirtyStackingContextZOrderLists
is irrelevant during document shutdown.

BUG=715736
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[chrishtr, 2017-04-26 23:23:40]

chrishtr@chromium.org changed reviewers:
+ wkorman@chromium.org

[chrishtr, 2017-04-26 23:23:45]

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

[commit-bot: I haz the power, 2017-04-26 23:25:25]

Dry run: CQ is trying da patch.

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

[wkorman, 2017-04-26 23:27:19]

lgtm

Is there a way to add a test that used to crash and no longer does?

[chrishtr, 2017-04-26 23:38:23]

On 2017/04/26 at 23:27:19, wkorman wrote:
> lgtm
> 
> Is there a way to add a test that used to crash and no longer does?

I was unable to reproduce this crash, unfortunately.

[chrishtr, 2017-04-26 23:39:20]

The CQ bit was unchecked by chrishtr@chromium.org

[chrishtr, 2017-04-26 23:39:22]

The CQ bit was checked by chrishtr@chromium.org

[commit-bot: I haz the power, 2017-04-26 23:40:10]

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, 2017-04-27 02:28:42]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493249962043880, "parent_rev":
"e18c551a2e12066dc4a8b0cb026774b824434eff", "commit_rev":
"d846315f83096c9b86e9206d901e56fa1190710a"}

[commit-bot: I haz the power, 2017-04-27 02:28:49]

Description was changed from

==========
Don't call DirtyStackingContextZOrderLists during document shutdown

DirtyStackingContextZOrderLists may try to call LayoutView::Compositor(),
which may be invalid during such times. In any case,
DirtyStackingContextZOrderLists
is irrelevant during document shutdown.

BUG=715736
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Don't call DirtyStackingContextZOrderLists during document shutdown

DirtyStackingContextZOrderLists may try to call LayoutView::Compositor(),
which may be invalid during such times. In any case,
DirtyStackingContextZOrderLists
is irrelevant during document shutdown.

BUG=715736
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2840403002
Cr-Commit-Position: refs/heads/master@{#467554}
Committed:
https://chromium.googlesource.com/chromium/src/+/d846315f83096c9b86e9206d901e...
==========

[commit-bot: I haz the power, 2017-04-27 02:28:50]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/d846315f83096c9b86e9206d901e...