Don't listen for NOTIFICATION_TAB_CLOSING in TabModalConfirmDialogDelegate

Don't listen for NOTIFICATION_TAB_CLOSING in TabModalConfirmDialogDelegate

WebContentsModalDialogManager closes web contents modal dialogs
when the WebContents is destroyed, so there is no need for
TabModalConfirmDialogDelegate to also close dialogs on tab closing.

BUG=157161
R=sky@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=221219

[sky, 2013-09-04 14:12:51]

LGTM

https://codereview.chromium.org/23820004/diff/1/chrome/browser/ui/tab_modal_c...
File chrome/browser/ui/tab_modal_confirm_dialog_delegate.cc (right):

https://codereview.chromium.org/23820004/diff/1/chrome/browser/ui/tab_modal_c...
chrome/browser/ui/tab_modal_confirm_dialog_delegate.cc:68: DCHECK(type ==
content::NOTIFICATION_LOAD_START);
DCHECK_EQ

[Mike Wittman, 2013-09-04 16:59:28]

https://codereview.chromium.org/23820004/diff/1/chrome/browser/ui/tab_modal_c...
File chrome/browser/ui/tab_modal_confirm_dialog_delegate.cc (right):

https://codereview.chromium.org/23820004/diff/1/chrome/browser/ui/tab_modal_c...
chrome/browser/ui/tab_modal_confirm_dialog_delegate.cc:68: DCHECK(type ==
content::NOTIFICATION_LOAD_START);
On 2013/09/04 14:12:51, sky wrote:
> DCHECK_EQ

Done.

[Mike Wittman, 2013-09-04 17:23:01]

Committed patchset #2 manually as r221219 (presubmit successful).