Add new DialogDelegate for dialogs in the App List

chrome/browser/ui/views/apps/app_info_dialog/app_info_dialog_views.h

Index: chrome/browser/ui/views/apps/app_info_dialog/app_info_dialog_views.h
diff --git a/chrome/browser/ui/views/apps/app_info_dialog/app_info_dialog_views.h b/chrome/browser/ui/views/apps/app_info_dialog/app_info_dialog_views.h
index c42035b1823e241cc00cb310f442b6c8247dcd48..599b97a0a8fbe19ce936d52ce6e3908ee591c63d 100644
--- a/chrome/browser/ui/views/apps/app_info_dialog/app_info_dialog_views.h
+++ b/chrome/browser/ui/views/apps/app_info_dialog/app_info_dialog_views.h
@@ -5,44 +5,21 @@
 #ifndef CHROME_BROWSER_UI_VIEWS_APPS_APP_INFO_DIALOG_APP_INFO_DIALOG_VIEWS_H_
 #define CHROME_BROWSER_UI_VIEWS_APPS_APP_INFO_DIALOG_APP_INFO_DIALOG_VIEWS_H_
 
-#include "ui/gfx/native_widget_types.h"
-#include "ui/views/window/dialog_delegate.h"
+#include "chrome/browser/ui/views/apps/app_info_dialog/app_info_tab.h"
 
 class Profile;
 
 namespace extensions {
 class Extension;
 }
-namespace views {
-class TabbedPane;
-}
 
 // View the information about a particular chrome application.
-class AppInfoDialog : public views::DialogDelegateView {
+class AppInfoDialog : public AppInfoTab {

[tapted, 2014/06/16 06:11:56]

I don't think this should inherit from AppInfoTab - the tabs should be children
of this. views::View should work fine I think

[sashab, 2014/06/16 07:01:20]

Ahh yup, this is a mistake. Done.

  public:
-  AppInfoDialog(gfx::NativeWindow parent_window,
-                Profile* profile,
-                const extensions::Extension* app,
-                const base::Closure& close_callback);
-
+  AppInfoDialog(Profile* profile, const extensions::Extension* app);
   virtual ~AppInfoDialog();
 
  private:
-  // Overridden from views::View:
-  virtual gfx::Size GetPreferredSize() const OVERRIDE;
-
-  // Overridden from views::DialogDelegate:
-  virtual bool Cancel() OVERRIDE;
-  virtual int GetDialogButtons() const OVERRIDE;
-
-  // Overridden from views::WidgetDelegate:
-  virtual ui::ModalType GetModalType() const OVERRIDE;
-
-  gfx::NativeWindow parent_window_;
-  Profile* profile_;
-  const extensions::Extension* app_;
-  base::Closure close_callback_;
-
   DISALLOW_COPY_AND_ASSIGN(AppInfoDialog);
 };