Make the App Info Dialog appear modal

chrome/browser/ui/app_list/app_list_service.h

 // Copyright 2013 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_APP_LIST_APP_LIST_SERVICE_H_
 #define CHROME_BROWSER_UI_APP_LIST_APP_LIST_SERVICE_H_
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "chrome/browser/ui/host_desktop.h"
 #include "ui/gfx/native_widget_types.h"
 
 class AppListControllerDelegate;
 class PrefRegistrySimple;
 class Profile;
 
+namespace app_list {
+class AppListView;
+}
+
 namespace base {
 class CommandLine;
 class FilePath;
 }
 
 namespace gfx {
 class ImageSkia;
 }
 
 class AppListService {
@@ skipping 64 matching lines @@
   virtual bool IsAppListVisible() const = 0;
 
   // Enable the app list. What this does specifically will depend on the host
   // operating system and shell.
   virtual void EnableAppList(Profile* initial_profile,
                              AppListEnableSource enable_source) = 0;
 
   // 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;
 
  protected:
   AppListService() {}
   virtual ~AppListService() {}
 
   // Do any once off initialization needed for the app list.
   virtual void Init(Profile* initial_profile) = 0;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(AppListService);
 };
 
 #endif  // CHROME_BROWSER_UI_APP_LIST_APP_LIST_SERVICE_H_