Avoid use-after-free when InterstitialPageImpl is being torn down.

Avoid use-after-free when InterstitialPageImpl is being torn down.

The InterstitialPageNavigatorImpl was continuing to access parts
of a deleted WebContents (e.g., the NavigationController) after its
corresponding InterstitialPageImpl was disabled.  This CL avoids
the problem by checking for the disabled state.

BUG=730592
TEST=NavigationHandleImpl crashes go away.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2934353002
Cr-Original-Commit-Position: refs/heads/master@{#479544}
Review-Url: https://codereview.chromium.org/2945163002 .
Cr-Commit-Position: refs/branch-heads/3112@{#409}
Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}
Committed: https://chromium.googlesource.com/chromium/src/+/bada48e0f4c8bb29ab5b0125411b860c2a0c3079

[Charlie Reis, 2017-06-20 21:16:21]

creis@chromium.org changed reviewers:
+ nick@chromium.org

[Charlie Reis, 2017-06-20 21:16:22]

FYI: Merging to M60.

[Charlie Reis, 2017-06-20 21:17:00]

Description was changed from

==========
Avoid use-after-free when InterstitialPageImpl is being torn down.

The InterstitialPageNavigatorImpl was continuing to access parts
of a deleted WebContents (e.g., the NavigationController) after its
corresponding InterstitialPageImpl was disabled.  This CL avoids
the problem by checking for the disabled state.

BUG=730592
TEST=NavigationHandleImpl crashes go away.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2934353002
Cr-Commit-Position: refs/heads/master@{#479544}
(cherry picked from commit f1ead56e93a1db602c7cdcf2a569bf8e0f4a8e52)
==========

to

==========
Avoid use-after-free when InterstitialPageImpl is being torn down.

The InterstitialPageNavigatorImpl was continuing to access parts
of a deleted WebContents (e.g., the NavigationController) after its
corresponding InterstitialPageImpl was disabled.  This CL avoids
the problem by checking for the disabled state.

BUG=730592
TEST=NavigationHandleImpl crashes go away.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2934353002
Cr-Original-Commit-Position: refs/heads/master@{#479544}
Review-Url: https://codereview.chromium.org/2945163002 .
Cr-Commit-Position: refs/branch-heads/3112@{#409}
Cr-Branched-From:
b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}
Committed:
https://chromium.googlesource.com/chromium/src/+/bada48e0f4c8bb29ab5b0125411b...
==========

[Charlie Reis, 2017-06-20 21:17:01]

Committed patchset #1 (id:1) manually as
bada48e0f4c8bb29ab5b0125411b860c2a0c3079 (tree was closed).