Use centered app list position whenever virtual keyboard is enabled.

ui/app_list/app_list_switches.h

Index: ui/app_list/app_list_switches.h
diff --git a/ui/app_list/app_list_switches.h b/ui/app_list/app_list_switches.h
index 17b6716c149978303603787af3eb12c1e84d3118..2e027d61c036f4c5a8e07ce10fd2219d29f0c70c 100644
--- a/ui/app_list/app_list_switches.h
+++ b/ui/app_list/app_list_switches.h
@@ -29,6 +29,10 @@ bool APP_LIST_EXPORT IsAppInfoEnabled();
 
 bool APP_LIST_EXPORT IsExperimentalAppListEnabled();

[varkha, 2014/04/30 16:20:15]

This should also not be experimental anymore.

[Matt Giuca, 2014/05/05 06:18:05]

This one is still experimental (it is different to
IsExperimentalAppListPositionEnabled; now IsCenteredAppListEnabled -- this one
also introduces a bunch of new experiments including switchable top-level
views).

 
+// Determines whether either command-line switch was given for enabling the
+// experimental app list position. Do not use this when positioning the app
+// list; instead use AppListViewDelegate::UseExperimentalAppListPosition. It
+// checks a superset of the conditions that trigger the position.
 bool APP_LIST_EXPORT IsExperimentalAppListPositionEnabled();

[benwells, 2014/04/30 06:40:30]

Let's remove all traces of 'experimental' from the code for this - it is no
longer experimental with your change. E.g. this could be
IsCenteredAppListEnabled.

[Matt Giuca, 2014/04/30 06:53:38]

Should we also change the flag name, or is that frowned upon?

[Matt Giuca, 2014/05/05 06:18:05]

Done.

 
 }  // namespace switches