Remove the close button in context menu for IME menu.

chrome/browser/ui/ash/launcher/launcher_context_menu.cc

 // Copyright 2012 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 "chrome/browser/ui/ash/launcher/launcher_context_menu.h"
 
 #include <string>
 
 #include "ash/desktop_background/user_wallpaper_delegate.h"
 #include "ash/metrics/user_metrics_recorder.h"
@@ skipping 133 matching lines @@
     } else if (item_.type == ash::TYPE_DIALOG) {
       AddItem(MENU_CLOSE,
               l10n_util::GetStringUTF16(IDS_LAUNCHER_CONTEXT_MENU_CLOSE));
     } else {
       if (item_.type == ash::TYPE_PLATFORM_APP) {
         AddItem(
             MENU_PIN,
             l10n_util::GetStringUTF16(IDS_LAUNCHER_CONTEXT_MENU_PIN));
       }
       if (controller_->IsOpen(item_.id)) {
-        AddItem(MENU_CLOSE,
-                l10n_util::GetStringUTF16(IDS_LAUNCHER_CONTEXT_MENU_CLOSE));
+        bool show_close_button = true;

[stevenjb, 2016/01/29 17:53:27]

nit (sorry, I didn't notice this before) this could be:
bool show_close_button = controller_->IsOpen(item_.id);

That way we could eliminate one level of indention here.

[Azure Wei, 2016/01/30 09:04:13]

Done. Thank you for pointing this out.

+#if defined(OS_CHROMEOS)
+        if (extension_misc::IsImeMenuExtensionId(
+                controller_->GetAppIDForShelfID(item_.id)))
+          show_close_button = false;

[stevenjb, 2016/01/29 17:53:27]

nit: {}

[Azure Wei, 2016/01/30 09:04:13]

Done.

+#endif
+        if (show_close_button)
+          AddItem(MENU_CLOSE,
+                  l10n_util::GetStringUTF16(IDS_LAUNCHER_CONTEXT_MENU_CLOSE));

[stevenjb, 2016/01/29 17:53:27]

{}

[Azure Wei, 2016/01/30 09:04:13]

Done.

       }
     }
     AddSeparator(ui::NORMAL_SEPARATOR);
     if (item_.type == ash::TYPE_APP_SHORTCUT ||
         item_.type == ash::TYPE_WINDOWED_APP ||
         item_.type == ash::TYPE_PLATFORM_APP) {
       const extensions::MenuItem::ExtensionKey app_key(
           controller_->GetAppIDForShelfID(item_.id));
       if (!app_key.empty()) {
         int index = 0;
@@ skipping 164 matching lines @@
     case MENU_CHANGE_WALLPAPER:
       ash::Shell::GetInstance()->user_wallpaper_delegate()->
           OpenSetWallpaperPage();
       break;
 #endif
     default:
       extension_items_->ExecuteCommand(command_id, NULL,
                                        content::ContextMenuParams());
   }
 }