chrome: Add TabModalConfirmDialog interface.

chrome/browser/ui/tab_modal_confirm_dialog.h

+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_UI_TAB_MODAL_CONFIRM_DIALOG_H_
+#define CHROME_BROWSER_UI_TAB_MODAL_CONFIRM_DIALOG_H_
+
+class TabContents;
+class TabModalConfirmDialogDelegate;
+
+class TabModalConfirmDialog {
+ public:
+  static TabModalConfirmDialog* Create(TabModalConfirmDialogDelegate* delegate,

[Peter Kasting, 2012/09/25 00:39:20]

This serves the same purpose as ShowTabModalConfirmDialog().  Pick one or the
other (personally I slightly prefer this Create() function), not both.

This will likely require you to also touch the TabModalConfirmDialogWebUI
implementation.

[tfarina, 2012/09/25 02:03:38]

Can I do this in a follow up, or this should be done within this patch?

[Peter Kasting, 2012/09/25 02:10:32]

It probably makes sense to do it in this patch because it will mean making more
classes derive from TabModalConfirmDialog, and that in turn may influence the
rest of the design here.

[tfarina, 2012/09/25 02:10:42]

Would you be fine if I add a Show() function here?

virtual void Show() = 0;

And then in the clients we do:

dialog = TabModalConfirmDialog::Create(...);
dialog->Show();

+                                       TabContents* tab_contents);
+
+  virtual ~TabModalConfirmDialog() {}

[Peter Kasting, 2012/09/25 00:39:20]

Nit: Should probably be protected, I doubt we want people deleting through this
interface

[tfarina, 2012/09/25 02:03:38]

Done.

+
+  // Accepts the dialog.
+  virtual void AcceptTabModalDialog() = 0;
+
+  // Cancels the dialog.
+  virtual void CancelTabModalDialog() = 0;
+};
+
+#endif  // CHROME_BROWSER_UI_TAB_MODAL_CONFIRM_DIALOG_H_