Don't show shortcut arrow on hosted apps on ChromeOS.

chrome/browser/ui/app_list/extension_app_item.cc

Index: chrome/browser/ui/app_list/extension_app_item.cc
diff --git a/chrome/browser/ui/app_list/extension_app_item.cc b/chrome/browser/ui/app_list/extension_app_item.cc
index 9ecbdab008de4f9fd29fc5593541833d05a388ca..f75fb52fbc5fd608d59f40b1027e20c87cdc1fc2 100644
--- a/chrome/browser/ui/app_list/extension_app_item.cc
+++ b/chrome/browser/ui/app_list/extension_app_item.cc
@@ -145,14 +145,12 @@ ExtensionAppItem::~ExtensionAppItem() {
 }
 
 bool ExtensionAppItem::NeedsOverlay() const {
-  // The overlay icon is disabled for hosted apps in windowed mode with
-  // streamlined hosted apps.
-  bool streamlined_hosted_apps =
-      extensions::util::IsStreamlinedHostedAppsEnabled();
 #if defined(OS_CHROMEOS)
-  if (!streamlined_hosted_apps)
-    return false;
+  // The overlay is disabled completely in ChromeOS.
+  return false;
 #endif

[Matt Giuca, 2015/01/22 06:39:54]

nit: Blank line after this.

[benwells, 2015/01/22 07:03:23]

Done.

+  // The overlay icon is disabled for hosted apps in windowed mode with
+  // streamlined hosted apps.

[Matt Giuca, 2015/01/22 06:39:54]

Move this comment to just above the return.

[benwells, 2015/01/22 07:03:23]

Done.

   extensions::LaunchType launch_type =
       GetExtension()
           ? extensions::GetLaunchType(extensions::ExtensionPrefs::Get(profile_),
@@ -160,7 +158,7 @@ bool ExtensionAppItem::NeedsOverlay() const {
           : extensions::LAUNCH_TYPE_WINDOW;
 
   return !is_platform_app_ && extension_id_ != extension_misc::kChromeAppId &&
-      (!streamlined_hosted_apps ||
+      (!extensions::util::IsStreamlinedHostedAppsEnabled() ||
        launch_type != extensions::LAUNCH_TYPE_WINDOW);
 }