Skips Chrome icon in Athena home card.

athena/extensions/extension_app_model_builder.cc

 // Copyright 2014 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.
 
 #include "athena/extensions/public/extension_app_model_builder.h"
 
 #include "athena/activity/public/activity_factory.h"
 #include "athena/activity/public/activity_manager.h"
 #include "athena/extensions/public/extensions_delegate.h"
 #include "extensions/browser/extension_icon_image.h"
 #include "extensions/common/constants.h"
 #include "extensions/common/extension_set.h"
 #include "extensions/common/manifest_handlers/icons_handler.h"
 #include "extensions/grit/extensions_browser_resources.h"
 #include "ui/app_list/app_list_item.h"
 #include "ui/app_list/app_list_model.h"
 #include "ui/base/resource/resource_bundle.h"
 
 namespace athena {
 
 namespace {
 
+// Copied from chrome/common/extensions/extension_constants.h
+// TODO(mukai): move constants to src/extensions
+const char kChromeAppId[] = "mgndgikekgjfcpckkfioiadnlibdjbkf";
+
 class AppItem : public app_list::AppListItem {
  public:
   AppItem(scoped_refptr<const extensions::Extension> extension,
           content::BrowserContext* browser_context)
       : app_list::AppListItem(extension->id()),
         extension_(extension),
         browser_context_(browser_context),
         icon_image_(browser_context_,
                     extension.get(),
                     extensions::IconsInfo::GetIcons(extension.get()),
@@ skipping 28 matching lines @@
 
 ExtensionAppModelBuilder::~ExtensionAppModelBuilder() {
 }
 
 void ExtensionAppModelBuilder::PopulateApps(app_list::AppListModel* model) {
   ExtensionsDelegate* bridge = ExtensionsDelegate::Get(browser_context_);
   const extensions::ExtensionSet& extensions = bridge->GetInstalledExtensions();
   for (extensions::ExtensionSet::const_iterator iter = extensions.begin();
        iter != extensions.end();
        ++iter) {
+    // Chrome icon is currently disabled for homecard since it's not meaningful.
+    // http://crbug.com/421677
     // TODO(mukai): use chrome/browser/extension_ui_util.
-    if ((*iter)->ShouldDisplayInAppLauncher()) {
+    if ((*iter)->ShouldDisplayInAppLauncher() &&
+        (*iter)->id() != kChromeAppId) {
       model->AddItem(scoped_ptr<app_list::AppListItem>(
           new AppItem(*iter, browser_context_)));
     }
   }
 }
 
 }  // namespace athena