Remove LauncherItemController::IsOpen, check ShelfItem::status.

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

Index: chrome/browser/ui/ash/launcher/chrome_launcher_controller_impl.cc
diff --git a/chrome/browser/ui/ash/launcher/chrome_launcher_controller_impl.cc b/chrome/browser/ui/ash/launcher/chrome_launcher_controller_impl.cc
index 95943445369070db96873fdf4d94058ea2fe7de6..580c7b09027b8ca561731a2e555435b69f5de81a 100644
--- a/chrome/browser/ui/ash/launcher/chrome_launcher_controller_impl.cc
+++ b/chrome/browser/ui/ash/launcher/chrome_launcher_controller_impl.cc
@@ -460,10 +460,8 @@ void ChromeLauncherControllerImpl::Close(ash::ShelfID id) {
 }
 
 bool ChromeLauncherControllerImpl::IsOpen(ash::ShelfID id) {

[msw, 2016/11/10 01:27:17]

FYI, this is only used by ArcAppContextMenu::BuildMenu.
khmel@ and I think this could be replaced in a followup.

-  LauncherItemController* controller = GetLauncherItemController(id);
-  if (!controller)
-    return false;
-  return controller->IsOpen();
+  const int index = model_->ItemIndexByID(id);
+  return index >= 0 && model_->items()[index].status != ash::STATUS_CLOSED;
 }
 
 bool ChromeLauncherControllerImpl::IsPlatformApp(ash::ShelfID id) {