Index: content/browser/web_contents/web_contents_impl.cc |
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc |
index 7f45ab817ae157f7ba283b6ff1d8377b1b87c5af..534192cd840cb46243a8874fea4ee6b5fe56b4d5 100644 |
--- a/content/browser/web_contents/web_contents_impl.cc |
+++ b/content/browser/web_contents/web_contents_impl.cc |
@@ -1495,12 +1495,10 @@ void WebContentsImpl::WasUnOccluded() { |
bool WebContentsImpl::NeedToFireBeforeUnload() { |
// TODO(creis): Should we fire even for interstitial pages? |
- // TODO(nasko): it's confusing that this method, per comments and tests that |
- // depend on this behavior, needs to check unload handlers as well. |
return WillNotifyDisconnection() && !ShowingInterstitialPage() && |
!GetRenderViewHost()->SuddenTerminationAllowed() && |
(GetMainFrame()->ShouldDispatchBeforeUnload() || |
- GetMainFrame()->HasUnloadHandler()); |
+ GetMainFrame()->ShouldDispatchUnload()); |
} |
void WebContentsImpl::DispatchBeforeUnload() { |