• dgozman's avatar
    Clear navigation_handle early in RenderFrameHostImpl destructor. · 5f8c1ef3
    dgozman authored
    This is to avoid inconsistent (half-destructed) RenderFrameHostImpl instance
    in WebContentsObserver::DidFinishNavigation, acessible through NavigationHandle
    instance.
    
    This happens in DevToolsManagerTest.ReattachOnCancelPendingNavigation, added
    DCHECK to RenderFrameDevToolsAgentHost.
    
    See crrev.com/2544893002 and crrev.com/2387353004 for more context.
    
    BUG=none
    CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
    
    Review-Url: https://codereview.chromium.org/2563863002
    Cr-Commit-Position: refs/heads/master@{#437658}
    5f8c1ef3
render_frame_host_impl.cc 132 KB