Add new DialogDelegate for dialogs in the App List

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

Index: chrome/browser/ui/views/apps/app_info_dialog/app_info_tab.h
diff --git a/chrome/browser/ui/views/apps/app_info_dialog/app_info_tab.h b/chrome/browser/ui/views/apps/app_info_dialog/app_info_tab.h
index a3df03dbc690d427ca941702f3f2a9881ffd8adf..8a19b39ac63d791bbabe56593d2aef4463a04922 100644
--- a/chrome/browser/ui/views/apps/app_info_dialog/app_info_tab.h
+++ b/chrome/browser/ui/views/apps/app_info_dialog/app_info_tab.h
@@ -5,10 +5,8 @@
 #ifndef CHROME_BROWSER_UI_VIEWS_APPS_APP_INFO_DIALOG_APP_INFO_TAB_H_
 #define CHROME_BROWSER_UI_VIEWS_APPS_APP_INFO_DIALOG_APP_INFO_TAB_H_
 
-#include "ui/gfx/native_widget_types.h"
 #include "ui/views/view.h"
 
-class AppInfoView;
 class Profile;
 
 namespace extensions {
@@ -22,18 +20,13 @@ class View;
 // profile and app. Tabs in the App Info dialog extend this class.
 class AppInfoTab : public views::View {
  public:
-  AppInfoTab(gfx::NativeWindow parent_window,
-             Profile* profile,
-             const extensions::Extension* app,
-             const base::Closure& close_callback);
+  AppInfoTab(Profile* profile, const extensions::Extension* app);
 
   virtual ~AppInfoTab();
 
  protected:

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

protected data members are only allowed in test harnesses --
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Access_Control

But this is probably better to fix in a follow-up CL, since it's orthogonal.

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

I copied this 'pattern' from CreateApplicationShortcutView, which also uses
profile_ as a protected member. But yes, if it says so in the style guide, I'll
fix this in a follow-up CL.

-  gfx::NativeWindow parent_window_;
   Profile* profile_;
   const extensions::Extension* app_;
-  const base::Closure& close_callback_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(AppInfoTab);