Show media router toolbar icon ephemerally for active local routes and issues

chrome/browser/ui/views/media_router/media_router_ui_browsertest.cc

 // Copyright 2015 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 "base/bind.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "chrome/browser/extensions/browser_action_test_util.h"
+#include "chrome/browser/media/router/media_router_ui_service.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_commands.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
+#include "chrome/browser/ui/toolbar/component_toolbar_actions_factory.h"
 #include "chrome/browser/ui/toolbar/media_router_action.h"
+#include "chrome/browser/ui/toolbar/media_router_action_controller.h"
 #include "chrome/browser/ui/toolbar/toolbar_action_view_delegate.h"
 #include "chrome/browser/ui/views/frame/browser_view.h"
 #include "chrome/browser/ui/views/toolbar/app_menu_button.h"
 #include "chrome/browser/ui/views/toolbar/browser_actions_container.h"
 #include "chrome/browser/ui/views/toolbar/toolbar_action_view.h"
 #include "chrome/browser/ui/views/toolbar/toolbar_view.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "chrome/test/base/ui_test_utils.h"
 #include "content/public/browser/web_contents.h"
@@ skipping 25 matching lines @@
     toolbar_action_view_widget_ = new views::Widget();
     views::Widget::InitParams params(views::Widget::InitParams::TYPE_POPUP);
     params.ownership = views::Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
     toolbar_action_view_widget_->Init(params);
     toolbar_action_view_widget_->Show();
 
     // Sets delegate on |media_router_action_|.
     toolbar_action_view_ = new ToolbarActionView(media_router_action_.get(),
                                                  browser_actions_container);
     toolbar_action_view_widget_->SetContentsView(toolbar_action_view_);
+
+    action_controller_ =
+        &MediaRouterUIService::Get(browser()->profile())->action_controller_;
+
+    issue_.reset(new Issue(
+        "title notification", "message notification",
+        media_router::IssueAction(media_router::IssueAction::TYPE_DISMISS),
+        std::vector<media_router::IssueAction>(), "route_id",
+        media_router::Issue::NOTIFICATION, false, -1));
+
+    routes_ = {MediaRoute("routeId1", MediaSource("sourceId"), "sinkId1",
+                          "description", true, std::string(), true)};
   }
 
   void TearDownOnMainThread() override {
     toolbar_action_view_widget_->Close();
     media_router_action_.reset();
     browser_action_test_util_.reset();
     InProcessBrowserTest::TearDownOnMainThread();
   }
 
   void OpenMediaRouterDialogAndWaitForNewWebContents() {
@@ skipping 18 matching lines @@
     ASSERT_EQ(chrome::kChromeUIMediaRouterURL,
         nav_observer.last_navigation_url().spec());
     nav_observer.StopWatchingNewWebContents();
   }
 
   void ExecuteMediaRouterAction(AppMenuButton* app_menu_button) {
     EXPECT_TRUE(app_menu_button->IsMenuShowing());
     media_router_action_->ExecuteAction(true);
   }
 
+  bool ActionExists() {
+    return ToolbarActionsModel::Get(browser()->profile())
+        ->HasComponentAction(
+            ComponentToolbarActionsFactory::kMediaRouterActionId);
+  }
+
+  void SetAlwaysShowActionPref(bool always_show) {
+    return ToolbarActionsModel::Get(browser()->profile())
+        ->component_migration_helper()
+        ->SetComponentActionPref(
+            ComponentToolbarActionsFactory::kMediaRouterActionId, always_show);
+  }
+
  protected:
   // Must be initialized after |InProcessBrowserTest::SetUpOnMainThread|.
   std::unique_ptr<BrowserActionTestUtil> browser_action_test_util_;
   std::unique_ptr<MediaRouterAction> media_router_action_;
 
   // ToolbarActionView constructed to set the delegate on
   // |media_router_action_|.
-  ToolbarActionView* toolbar_action_view_;
+  ToolbarActionView* toolbar_action_view_ = nullptr;
 
   // Hosts the |toolbar_action_view_|.
-  views::Widget* toolbar_action_view_widget_;
+  views::Widget* toolbar_action_view_widget_ = nullptr;
+
+  std::unique_ptr<Issue> issue_;
+
+  // A vector of MediaRoutes that includes a local route.
+  std::vector<MediaRoute> routes_;
+
+  MediaRouterActionController* action_controller_ = nullptr;
 };
 
 IN_PROC_BROWSER_TEST_F(MediaRouterUIBrowserTest,
                        OpenDialogWithMediaRouterAction) {
   // We start off at about:blank page.
   // Make sure there is 1 tab and media router is enabled.
   ASSERT_EQ(1, browser()->tab_strip_model()->count());
 
   OpenMediaRouterDialogAndWaitForNewWebContents();
 
   // Reload the browser and wait.
   content::TestNavigationObserver reload_observer(
       browser()->tab_strip_model()->GetActiveWebContents());
   chrome::Reload(browser(), WindowOpenDisposition::CURRENT_TAB);
   reload_observer.Wait();
 
   // The reload should have removed the previously created dialog.
   // We expect a new dialog WebContents to be created by calling this.
   OpenMediaRouterDialogAndWaitForNewWebContents();
 
   // Navigate away.
   ui_test_utils::NavigateToURL(browser(), GURL("about:blank"));
 
   // The navigation should have removed the previously created dialog.
   // We expect a new dialog WebContents to be created by calling this.
   OpenMediaRouterDialogAndWaitForNewWebContents();
 }
 
+IN_PROC_BROWSER_TEST_F(MediaRouterUIBrowserTest, EphemeralToolbarIcon) {
+  action_controller_->OnIssueUpdated(issue_.get());
+  EXPECT_TRUE(ActionExists());
+  action_controller_->OnIssueUpdated(nullptr);
+  EXPECT_FALSE(ActionExists());
+
+  action_controller_->OnRoutesUpdated(routes_, std::vector<MediaRoute::Id>());
+  EXPECT_TRUE(ActionExists());
+  action_controller_->OnRoutesUpdated(std::vector<MediaRoute>(),
+                                      std::vector<MediaRoute::Id>());
+  EXPECT_FALSE(ActionExists());
+
+  SetAlwaysShowActionPref(true);
+  EXPECT_TRUE(ActionExists());
+  SetAlwaysShowActionPref(false);
+  EXPECT_FALSE(ActionExists());
+}
+
+IN_PROC_BROWSER_TEST_F(MediaRouterUIBrowserTest,
+                       EphemeralToolbarIconWithMultipleWindows) {
+  action_controller_->OnRoutesUpdated(routes_, std::vector<MediaRoute::Id>());
+  EXPECT_TRUE(ActionExists());
+
+  // Opening and closing a window shouldn't affect the state of the ephemeral
+  // icon. Creating and removing the icon with multiple windows open should also
+  // work.
+  Browser* browser2 = CreateBrowser(browser()->profile());
+  EXPECT_TRUE(ActionExists());
+  action_controller_->OnRoutesUpdated(std::vector<MediaRoute>(),
+                                      std::vector<MediaRoute::Id>());
+  EXPECT_FALSE(ActionExists());
+  action_controller_->OnRoutesUpdated(routes_, std::vector<MediaRoute::Id>());
+  EXPECT_TRUE(ActionExists());
+  browser2->window()->Close();
+  EXPECT_TRUE(ActionExists());
+}
+
 }  // namespace media_router