Notify launcher page with onTransitionChanged event

chrome/browser/apps/custom_launcher_page_browsertest_views.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 <string>
 
 #include "base/command_line.h"
 #include "chrome/browser/apps/app_browsertest_util.h"
 #include "chrome/browser/ui/app_list/app_list_service.h"
+#include "chrome/browser/ui/app_list/app_list_service_views.h"
+#include "chrome/browser/ui/app_list/app_list_shower_views.h"
 #include "extensions/common/extension.h"
 #include "extensions/common/switches.h"
+#include "extensions/test/extension_test_message_listener.h"
 #include "ui/app_list/app_list_switches.h"
+#include "ui/app_list/views/app_list_main_view.h"
+#include "ui/app_list/views/app_list_view.h"
+#include "ui/app_list/views/contents_view.h"
 
 namespace {
 
 // The path of the test application within the "platform_apps" directory.
 const char kCustomLauncherPagePath[] = "custom_launcher_page";
 
 // The app ID of the test application.
 const char kCustomLauncherPageID[] = "lmadimbbgapmngbiclpjjngmdickadpl";
 
 }  // namespace
@@ skipping 24 matching lines @@
     AppListService* service =
         AppListService::Get(chrome::HOST_DESKTOP_TYPE_NATIVE);
     DCHECK(service);
     service->ShowForProfile(browser()->profile());
   }
 
  private:
   DISALLOW_COPY_AND_ASSIGN(CustomLauncherPageBrowserTest);
 };
 
-IN_PROC_BROWSER_TEST_F(CustomLauncherPageBrowserTest, LoadPageAndOpenLauncher) {
+IN_PROC_BROWSER_TEST_F(CustomLauncherPageBrowserTest,
+                       OpenLauncherAndSwitchToCustomPage) {
   LoadAndLaunchPlatformApp(kCustomLauncherPagePath, "Launched");
+
+  AppListServiceViews* service = static_cast<AppListServiceViews*>(
+      AppListService::Get(chrome::HOST_DESKTOP_TYPE_NATIVE));
+  // The app list should have loaded instantly since the profile is already
+  // loaded.
+  EXPECT_TRUE(service->IsAppListVisible());
+
+  app_list::ContentsView* contents_view =
+      service->shower().app_list()->app_list_main_view()->contents_view();
+
+  ASSERT_TRUE(
+      contents_view->IsStateActive(app_list::AppListModel::STATE_START));
+
+  {
+    ExtensionTestMessageListener listener("onPageExpanded", false);
+    contents_view->SetActivePage(contents_view->GetPageIndexForState(
+        app_list::AppListModel::STATE_CUSTOM_LAUNCHER_PAGE));
+
+    listener.WaitUntilSatisfied();
+  }
+  {
+    ExtensionTestMessageListener listener("onPageCollapsed", false);
+    contents_view->SetActivePage(contents_view->GetPageIndexForState(

[tapted, 2014/11/07 05:34:56]

You might run into a problem here on ChromeOS that could make this test flaky.
There's a run loop spun while doing listener.WaitUntilSatisfied(), during which
another window could take focus, which will dismiss the applist and destroy
contents_view, leading to an invalid access here.

You might also have a problem earlier, depending whether dismissing the app list
mid-animation ends up in the onPageExpanded state. I think in that case the test
will hang (on any platform).

Suff like this keeps coming up... I think we just need a
ScopedKeepAppListOpenForTest that works on all platforms.

[calamity, 2014/11/10 03:38:30]

Added and implemented the disable-app-list-on-blur command line flag for now.

+        app_list::AppListModel::STATE_START));
+
+    listener.WaitUntilSatisfied();
+  }
 }