When app shortcuts are enabled, do a once-off creation of all shortcuts.

apps/pref_names.cc

Index: apps/pref_names.cc
diff --git a/apps/pref_names.cc b/apps/pref_names.cc
index b4d3176a594112a72114f76b23587bff3b327b81..16531bd340939b98f4d92f61b83a64c08bd3222d 100644
--- a/apps/pref_names.cc
+++ b/apps/pref_names.cc
@@ -27,6 +27,11 @@ const char kAppLauncherHasBeenEnabled[] =
 const char kAppLauncherIsEnabled[] =
     "apps.app_launcher.should_show_apps_page";
 
+// A boolean that indicates whether app shortcuts are enabled.
+// On a transition from false to true, shortcuts are created for all apps.
+const char kAppShortcutsEnabled[] =
+    "apps.app_shortcuts_enabled";

[Matt Giuca, 2013/06/20 10:19:18]

Should fit on one line.

[Matt Giuca, 2013/06/20 10:19:18]

"enabled" sounds like it's a choice you can turn either on or off. How about
"apps.shortcuts_have_been_created", which gives it a "one time" feel, and
follows the pattern of apps.app_launcher_has_been_enabled.

(Edit: Haha I see Ben made almost exactly the same suggestion later.)

[jackhou1, 2013/06/20 11:03:22]

Done.

[jackhou1, 2013/06/20 11:03:22]

Done.

+
 // A boolean identifying if we should show the app launcher promo or not.
 const char kShowAppLauncherPromo[] = "app_launcher.show_promo";