Make the App Info Dialog appear modal

chrome/browser/ui/app_list/app_list_controller_delegate.h

Index: chrome/browser/ui/app_list/app_list_controller_delegate.h
diff --git a/chrome/browser/ui/app_list/app_list_controller_delegate.h b/chrome/browser/ui/app_list/app_list_controller_delegate.h
index 647dbfa6a8542790ccb4163839b550dda6015e00..93b5534848ae87bd77666aabf4bca9af36a24da3 100644
--- a/chrome/browser/ui/app_list/app_list_controller_delegate.h
+++ b/chrome/browser/ui/app_list/app_list_controller_delegate.h
@@ -24,6 +24,7 @@ class InstallTracker;
 
 namespace gfx {
 class ImageSkia;
+class Rect;
 }
 
 // Interface to allow the view delegate to call out to whatever is controlling
@@ -60,6 +61,11 @@ class AppListControllerDelegate {
   // Get app list window.
   virtual gfx::NativeWindow GetAppListWindow() = 0;
 
+  // Get the content bounds of the app list in the screen. On platforms that
+  // use views, this returns the bounds of the AppListView. Without views, this
+  // returns a 0x0 rectangle.
+  virtual gfx::Rect GetAppListBounds() = 0;

[tapted, 2014/05/21 03:40:57]

I think it's fine to give an implementation in the base clase that returns
gfx::Rect(). This will avoid the duplication in AppListControllerDelegateImpl
and TestAppListControllerDelegate

[sashab, 2014/05/22 07:20:21]

Good thinking. Done.

+
   // Get the application icon to be used, if any, for the app list.
   virtual gfx::ImageSkia GetWindowIcon() = 0;