MacViews: Harmony for toolbar actions bubbles.

chrome/browser/ui/views/extensions/extension_message_bubble_view_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/macros.h"
 #include "chrome/browser/ui/extensions/extension_message_bubble_browsertest.h"
-#include "chrome/browser/ui/toolbar/toolbar_actions_bar.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/test/test_browser_dialog.h"
 #include "chrome/browser/ui/views/toolbar/toolbar_actions_bar_bubble_views.h"
-#include "chrome/browser/ui/views/toolbar/toolbar_view.h"
+#include "ui/base/ui_base_switches.h"
 #include "ui/views/bubble/bubble_dialog_delegate.h"
 #include "ui/views/controls/link.h"
 #include "ui/views/controls/link_listener.h"
 #include "ui/views/window/dialog_client_view.h"
 
 namespace {
 
-// Returns the ToolbarView for the given |browser|.
-ToolbarView* GetToolbarViewForBrowser(Browser* browser) {
-  return BrowserView::GetBrowserViewForBrowser(browser)->toolbar();
-}
-
-ToolbarActionsBarBubbleViews* GetBubbleForBrowser(Browser* browser) {
-  return static_cast<ToolbarActionsBarBubbleViews*>(
-      GetToolbarViewForBrowser(browser)->browser_actions()->active_bubble());
-}
-
-// Checks that the |bubble| is using the |expected_reference_view|, and is in
-// approximately the correct position.
-void CheckBubbleAndReferenceView(views::BubbleDialogDelegateView* bubble,
-                                 views::View* expected_reference_view) {
+// Checks that the |bubble| is showing. Uses |reference_bounds| to ensure it is
+// in approximately the correct position.
+void CheckBubbleAgainstReferenceBounds(views::BubbleDialogDelegateView* bubble,
+                                       const gfx::Rect& reference_bounds) {
   ASSERT_TRUE(bubble);
-  ASSERT_TRUE(expected_reference_view);
-  EXPECT_EQ(expected_reference_view, bubble->GetAnchorView());
 
   // Do a rough check that the bubble is in the right place.
   gfx::Rect bubble_bounds = bubble->GetWidget()->GetWindowBoundsInScreen();
-  gfx::Rect reference_bounds = expected_reference_view->GetBoundsInScreen();
   // It should be below the reference view, but not too far below.
   EXPECT_GE(bubble_bounds.y(), reference_bounds.y());
   // The arrow should be poking into the anchor.
   const int kShadowWidth = 1;
   EXPECT_LE(bubble_bounds.y(), reference_bounds.bottom() + kShadowWidth);
   // The bubble should intersect the reference view somewhere along the x-axis.
   EXPECT_FALSE(bubble_bounds.x() > reference_bounds.right());
   EXPECT_FALSE(reference_bounds.x() > bubble_bounds.right());
 
   // And, of course, the bubble should be visible...
   EXPECT_TRUE(bubble->visible());
   // ... as should its Widget.
   EXPECT_TRUE(bubble->GetWidget()->IsVisible());
 }
 
 }  // namespace
 
 class ExtensionMessageBubbleViewBrowserTest
-    : public ExtensionMessageBubbleBrowserTest {
+    : public SupportsTestDialog<ExtensionMessageBubbleBrowserTest> {
  protected:
   ExtensionMessageBubbleViewBrowserTest() {}
   ~ExtensionMessageBubbleViewBrowserTest() override {}
 
+  // ExtensionMessageBubbleBrowserTest:
+  void SetUpCommandLine(base::CommandLine* command_line) override;
+
+  // TestBrowserDialog:
+  void ShowDialog(const std::string& name) override;
+
  private:
   // ExtensionMessageBubbleBrowserTest:
+  void CloseBubble(Browser* browser) override;
   void CheckBubbleNative(Browser* browser, AnchorPosition anchor) override;
   void CloseBubbleNative(Browser* browser) override;
   void CheckBubbleIsNotPresentNative(Browser* browser) override;
   void ClickLearnMoreButton(Browser* browser) override;
   void ClickActionButton(Browser* browser) override;
   void ClickDismissButton(Browser* browser) override;
 
+  // Whether to ignore requests from ExtensionMessageBubbleBrowserTest to
+  // CloseBubble().
+  bool block_close_ = false;
+
   DISALLOW_COPY_AND_ASSIGN(ExtensionMessageBubbleViewBrowserTest);
 };
 
 class ExtensionMessageBubbleViewBrowserTestLegacy
     : public ExtensionMessageBubbleViewBrowserTest {
  protected:
   void SetUpCommandLine(base::CommandLine* command_line) override {
     ExtensionMessageBubbleViewBrowserTest::SetUpCommandLine(command_line);
     override_redesign_.reset();
     override_redesign_.reset(new extensions::FeatureSwitch::ScopedOverride(
         extensions::FeatureSwitch::extension_action_redesign(), false));
   }
 };
 
+void ExtensionMessageBubbleViewBrowserTest::SetUpCommandLine(
+    base::CommandLine* command_line) {
+  ExtensionMessageBubbleBrowserTest::SetUpCommandLine(command_line);
+  // MD is required on Mac to get a Views bubble. Enable for all platforms.

[Devlin, 2017/03/09 16:18:51]

nit: maybe expand to
// This shouldn't affect the behavior of the bubble, so enable for all
platforms.

[tapted, 2017/03/10 04:57:40]

Done.

+  command_line->AppendSwitch(switches::kExtendMdToSecondaryUi);
+}
+
+void ExtensionMessageBubbleViewBrowserTest::ShowDialog(
+    const std::string& name) {
+  EXPECT_EQ("devmode", name);  // TODO(tapted): Add cases for all bubbles types.

[Devlin, 2017/03/09 16:18:51]

how descriptive are these names supposed to be?  "devmode" is pretty vague,
"developer mode extensions warning bubble" is probably too specific, "dev mode
extensions bubble" is somewhere in between.  You know the naming convention
best, so leaving it up to you. :)

[tapted, 2017/03/10 04:57:40]

I went with "devmode_warning" and added a comment above the test case:

// BrowserDialogTest for the warning bubble that appears at startup when there
// are extensions installed in developer mode. Can be invoked interactively with
// --gtest_filter=BrowserDialogTest.Invoke.
IN_PROC_BROWSER_TEST_F(ExtensionMessageBubbleView

+
+  // When invoked this way, the dialog test harness must close the bubble.
+  block_close_ = true;
+  TestBubbleAnchoredToExtensionAction();
+  block_close_ = false;
+}
+
+void ExtensionMessageBubbleViewBrowserTest::CloseBubble(Browser* browser) {
+  if (!block_close_)
+    ExtensionMessageBubbleBrowserTest::CloseBubble(browser);
+}
+
 void ExtensionMessageBubbleViewBrowserTest::CheckBubbleNative(
     Browser* browser,
     AnchorPosition anchor) {
-  ToolbarView* toolbar_view = GetToolbarViewForBrowser(browser);
-  BrowserActionsContainer* container = toolbar_view->browser_actions();
-  views::BubbleDialogDelegateView* bubble = container->active_bubble();
-  views::View* anchor_view = nullptr;
-  switch (anchor) {
-    case ANCHOR_BROWSER_ACTION:
-      DCHECK_GT(container->num_toolbar_actions(), 0u);
-      anchor_view = container->GetToolbarActionViewAt(0);
-      break;
-    case ANCHOR_APP_MENU:
-      anchor_view = toolbar_view->app_menu_button();
-      break;
-  }
-  CheckBubbleAndReferenceView(bubble, anchor_view);
+  gfx::Rect reference_bounds =
+      GetAnchorReferenceBoundsForBrowser(browser, anchor);
+  CheckBubbleAgainstReferenceBounds(GetViewsBubbleForBrowser(browser),
+                                    reference_bounds);
 }
 
 void ExtensionMessageBubbleViewBrowserTest::CloseBubbleNative(
     Browser* browser) {
-  BrowserActionsContainer* container =
-      GetToolbarViewForBrowser(browser)->browser_actions();
-  views::BubbleDialogDelegateView* bubble = container->active_bubble();
+  views::BubbleDialogDelegateView* bubble = GetViewsBubbleForBrowser(browser);
   ASSERT_TRUE(bubble);
   bubble->GetWidget()->Close();
-  EXPECT_EQ(nullptr, container->active_bubble());
+  EXPECT_EQ(nullptr, GetViewsBubbleForBrowser(browser));
 }
 
 void ExtensionMessageBubbleViewBrowserTest::CheckBubbleIsNotPresentNative(
     Browser* browser) {
-  EXPECT_EQ(
-      nullptr,
-      GetToolbarViewForBrowser(browser)->browser_actions()->active_bubble());
+  EXPECT_EQ(nullptr, GetViewsBubbleForBrowser(browser));
 }
 
 void ExtensionMessageBubbleViewBrowserTest::ClickLearnMoreButton(
     Browser* browser) {
-  ToolbarActionsBarBubbleViews* bubble = GetBubbleForBrowser(browser);
+  ToolbarActionsBarBubbleViews* bubble = GetViewsBubbleForBrowser(browser);
   static_cast<views::LinkListener*>(bubble)->LinkClicked(
       const_cast<views::Link*>(bubble->learn_more_button()), 0);
 }
 
 void ExtensionMessageBubbleViewBrowserTest::ClickActionButton(
     Browser* browser) {
-  ToolbarActionsBarBubbleViews* bubble = GetBubbleForBrowser(browser);
+  ToolbarActionsBarBubbleViews* bubble = GetViewsBubbleForBrowser(browser);
   bubble->GetDialogClientView()->AcceptWindow();
 }
 
 void ExtensionMessageBubbleViewBrowserTest::ClickDismissButton(
     Browser* browser) {
-  ToolbarActionsBarBubbleViews* bubble = GetBubbleForBrowser(browser);
+  ToolbarActionsBarBubbleViews* bubble = GetViewsBubbleForBrowser(browser);
   bubble->GetDialogClientView()->CancelWindow();
 }
 
 IN_PROC_BROWSER_TEST_F(ExtensionMessageBubbleViewBrowserTest,
                        ExtensionBubbleAnchoredToExtensionAction) {
   TestBubbleAnchoredToExtensionAction();
 }
 
 IN_PROC_BROWSER_TEST_F(ExtensionMessageBubbleViewBrowserTestLegacy,
                        ExtensionBubbleAnchoredToAppMenu) {
@@ skipping 77 matching lines @@
 
 IN_PROC_BROWSER_TEST_F(ExtensionMessageBubbleViewBrowserTest,
                        TestClickingActionButton) {
   TestClickingActionButton();
 }
 
 IN_PROC_BROWSER_TEST_F(ExtensionMessageBubbleViewBrowserTest,
                        TestClickingDismissButton) {
   TestClickingDismissButton();
 }
+
+IN_PROC_BROWSER_TEST_F(ExtensionMessageBubbleViewBrowserTest,
+                       InvokeDialog_devmode) {
+  RunDialog();
+}