Add new DialogDelegate for dialogs in the App List

chrome/browser/ui/app_list/app_list_controller_delegate.cc

 // 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.
 
 #include "chrome/browser/ui/app_list/app_list_controller_delegate.h"
 
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extension_util.h"
 #include "chrome/browser/extensions/install_tracker_factory.h"
 #include "chrome/browser/extensions/launch_util.h"
@@ skipping 83 matching lines @@
     Profile* profile,
     const std::string& extension_id) {
   DCHECK(CanDoShowAppInfoFlow());
   ExtensionService* service =
       extensions::ExtensionSystem::Get(profile)->extension_service();
   DCHECK(service);
   const extensions::Extension* extension = service->GetInstalledExtension(
       extension_id);
   DCHECK(extension);
 
-  gfx::NativeWindow parent_window = GetAppListWindow();
-  if (!parent_window)
-    return;
+  OnShowChildDialog();
 
-  // For the centered app list, inset the dialog further so it appears as a
-  // vertical column in the center of the app list.
-  const int kAppListDialogHorizontalBorderInsets =
-      app_list::switches::IsCenteredAppListEnabled() ? 110 : 10;
-  const int kAppListDialogVerticalBorderInsets = 10;
-
-  gfx::Rect dialog_bounds = GetAppListBounds();
-  dialog_bounds.Inset(kAppListDialogHorizontalBorderInsets,
-                      kAppListDialogVerticalBorderInsets);
-
-  OnShowChildDialog();
-  ShowAppInfoDialog(parent_window,
-                    dialog_bounds,
-                    profile,
-                    extension,
-                    base::Bind(&AppListControllerDelegate::OnCloseChildDialog,
-                               base::Unretained(this)));
+  // Since the AppListControllerDelegate is a leaky singleton, passing its
+  // raw pointer around is OK.
+  ShowAppInfoDialog(this, profile, extension);
 }
 
 void AppListControllerDelegate::UninstallApp(Profile* profile,
                                              const std::string& app_id) {
   // ExtensionUninstall deletes itself when done or aborted.
   ExtensionUninstaller* uninstaller =
       new ExtensionUninstaller(profile, app_id, this);
   uninstaller->Run();
 }
 
@@ skipping 87 matching lines @@
   out_apps->InsertAll(registry->enabled_extensions());
   out_apps->InsertAll(registry->disabled_extensions());
   out_apps->InsertAll(registry->terminated_extensions());
 }
 
 void AppListControllerDelegate::OnSearchStarted() {
 #if defined(ENABLE_RLZ)
   RLZTracker::RecordAppListSearch();
 #endif
 }