Make the App Info Dialog appear modal

chrome/browser/ui/app_list/app_list_service.h

Index: chrome/browser/ui/app_list/app_list_service.h
diff --git a/chrome/browser/ui/app_list/app_list_service.h b/chrome/browser/ui/app_list/app_list_service.h
index 91fbe7f3ef88170a05365bbcca4c242cdccd33fa..70872f4015698bad17c3ba9d7099454b1b9224d8 100644
--- a/chrome/browser/ui/app_list/app_list_service.h
+++ b/chrome/browser/ui/app_list/app_list_service.h
@@ -16,6 +16,10 @@ class AppListControllerDelegate;
 class PrefRegistrySimple;
 class Profile;
 
+namespace app_list {
+class AppListView;
+}
+
 namespace base {
 class CommandLine;
 class FilePath;
@@ -100,6 +104,9 @@ class AppListService {
   // Get the window the app list is in, or NULL if the app list isn't visible.
   virtual gfx::NativeWindow GetAppListWindow() = 0;
 
+  // Get app list view, or NULL if the app list isn't visible.
+  virtual app_list::AppListView* GetAppListView() = 0;

[calamity, 2014/05/16 05:54:32]

Hmm. Mac doesn't have an AppListView which is likely why GetAppListWindow()
exists here.

In fact, after looking around, it looks like you can just implement
DoShowAppInfoFlow in AppListControllerDelegateViews (it's currently in
app_list_controller_delegate.cc). Given that it's not implemented on other
platforms, there's no real need to have DoShowAppInfoFlow anywhere else?

From AppListControllerDelegateViews, you can access AppListServiceViews which
will provide you with shower().app_list().

[sashab, 2014/05/19 07:24:53]

Great suggestion!! It was even easier than that - I put the call to
ShowAppListOverlay() in OnShowExtensionPrompt() and OnCloseExtensionPrompt() in
AppListControllerDelegateViews, so now it applies to all popups in the app list.

+
   // Returns a pointer to the platform specific AppListControllerDelegate.
   virtual AppListControllerDelegate* GetControllerDelegate() = 0;