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

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

 // Copyright 2016 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 "ash/common/shelf/shelf_delegate.h"
 #include "ash/common/wm_shell.h"
 #include "ash/wm/window_util.h"
 #include "base/macros.h"
 #include "base/run_loop.h"
 #include "chrome/browser/extensions/extension_browsertest.h"
@@ skipping 343 matching lines @@
   SendPackageAdded(true);
   EXPECT_TRUE(app_list_service->IsAppListVisible());
 
   app_list_service->DismissAppList();
   EXPECT_FALSE(app_list_service->IsAppListVisible());
 
   // Send package update event. App list is not shown.
   SendPackageAdded(true);
   EXPECT_FALSE(app_list_service->IsAppListVisible());
 }
+
+// Test AppListControllerDelegate::IsAppOpen for Arc apps.
+IN_PROC_BROWSER_TEST_F(ArcAppLauncherBrowserTest, IsAppOpen) {
+  StartInstance();
+  InstallTestApps(false);
+  SendPackageAdded(true);
+  const std::string app_id = GetTestApp1Id();
+
+  AppListService* service = AppListService::Get();
+  AppListControllerDelegate* delegate = service->GetControllerDelegate();
+  EXPECT_FALSE(delegate->IsAppOpen(app_id));
+  arc::LaunchApp(profile(), app_id);
+  EXPECT_FALSE(delegate->IsAppOpen(app_id));

[khmel, 2016/11/10 01:31:26]

Should not it be EXPECT_TRUE()?

[msw, 2016/11/10 02:39:10]

You're right; good catch! Fixed with a fake task creation.

+}