Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(92)

Unified Diff: chrome/browser/ui/views/extensions/extension_toolbar_icon_surfacing_bubble_views_unittest.cc

Issue 1086973004: [Extensions Mac] Implement developer mode warning on mac (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Avi's Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/extensions/extension_toolbar_icon_surfacing_bubble_views_unittest.cc
diff --git a/chrome/browser/ui/views/extensions/extension_toolbar_icon_surfacing_bubble_views_unittest.cc b/chrome/browser/ui/views/extensions/extension_toolbar_icon_surfacing_bubble_views_unittest.cc
index e3a52fb02146c5453b975303c063ede9bf7adbe3..ab7cbd2bf2c700071c43c402d536ccebfa0a1775 100644
--- a/chrome/browser/ui/views/extensions/extension_toolbar_icon_surfacing_bubble_views_unittest.cc
+++ b/chrome/browser/ui/views/extensions/extension_toolbar_icon_surfacing_bubble_views_unittest.cc
@@ -3,7 +3,8 @@
// found in the LICENSE file.
#include "base/run_loop.h"
-#include "chrome/browser/ui/toolbar/toolbar_actions_bar_bubble_delegate.h"
+#include "base/strings/utf_string_conversions.h"
+#include "chrome/browser/ui/toolbar/test_toolbar_actions_bar_bubble_delegate.h"
#include "chrome/browser/ui/views/extensions/extension_toolbar_icon_surfacing_bubble_views.h"
#include "ui/events/event_utils.h"
#include "ui/events/test/event_generator.h"
@@ -13,33 +14,14 @@
#include "ui/views/test/views_test_base.h"
#include "ui/views/widget/widget.h"
-namespace {
-
using ExtensionToolbarIconSurfacingBubbleTest = views::ViewsTestBase;
-class TestDelegate : public ToolbarActionsBarBubbleDelegate {
- public:
- TestDelegate() {}
- ~TestDelegate() {}
-
- void OnToolbarActionsBarBubbleShown() override {}
- void OnToolbarActionsBarBubbleClosed(CloseAction action) override {
- EXPECT_FALSE(action_);
- action_.reset(new CloseAction(action));
- }
-
- CloseAction* action() const { return action_.get(); }
-
- private:
- scoped_ptr<CloseAction> action_;
-
- DISALLOW_COPY_AND_ASSIGN(TestDelegate);
-};
-
-} // namespace
-
TEST_F(ExtensionToolbarIconSurfacingBubbleTest,
ExtensionToolbarIconSurfacingBubbleTest) {
+ const base::string16 kHeadingString = base::ASCIIToUTF16("Heading");
+ const base::string16 kBodyString = base::ASCIIToUTF16("Body");
+ const base::string16 kActionString = base::ASCIIToUTF16("Action");
+
scoped_ptr<views::Widget> anchor_widget(new views::Widget());
views::Widget::InitParams params =
CreateParams(views::Widget::InitParams::TYPE_WINDOW);
@@ -48,16 +30,19 @@ TEST_F(ExtensionToolbarIconSurfacingBubbleTest,
{
// Test clicking on the button.
- TestDelegate delegate;
+ TestToolbarActionsBarBubbleDelegate delegate(
+ kHeadingString, kBodyString, kActionString);
ExtensionToolbarIconSurfacingBubble* bubble =
new ExtensionToolbarIconSurfacingBubble(
anchor_widget->GetContentsView(),
- &delegate);
+ delegate.GetDelegate());
views::Widget* bubble_widget =
views::BubbleDelegateView::CreateBubble(bubble);
- bubble_widget->Show();
+ EXPECT_FALSE(delegate.shown());
+ bubble->Show();
+ EXPECT_TRUE(delegate.shown());
views::test::TestWidgetObserver bubble_observer(bubble_widget);
- EXPECT_FALSE(delegate.action());
+ EXPECT_FALSE(delegate.close_action());
// Find the button and click on it.
views::View* button = nullptr;
@@ -76,30 +61,32 @@ TEST_F(ExtensionToolbarIconSurfacingBubbleTest,
// The bubble should be closed, and the delegate should be told that the
// button was clicked.
- ASSERT_TRUE(delegate.action());
- EXPECT_EQ(ToolbarActionsBarBubbleDelegate::ACKNOWLEDGED,
- *delegate.action());
+ ASSERT_TRUE(delegate.close_action());
+ EXPECT_EQ(ToolbarActionsBarBubbleDelegate::CLOSE_EXECUTE,
+ *delegate.close_action());
EXPECT_TRUE(bubble_observer.widget_closed());
}
{
// Test dismissing the bubble without clicking the button.
- TestDelegate delegate;
+ TestToolbarActionsBarBubbleDelegate delegate(
+ kHeadingString, kBodyString, kActionString);
ExtensionToolbarIconSurfacingBubble* bubble =
new ExtensionToolbarIconSurfacingBubble(
anchor_widget->GetContentsView(),
- &delegate);
+ delegate.GetDelegate());
views::Widget* bubble_widget =
views::BubbleDelegateView::CreateBubble(bubble);
- bubble_widget->Show();
+ bubble->Show();
views::test::TestWidgetObserver bubble_observer(bubble_widget);
- EXPECT_FALSE(delegate.action());
+ EXPECT_FALSE(delegate.close_action());
// Close the bubble. The delegate should be told it was dismissed.
bubble_widget->Close();
base::RunLoop().RunUntilIdle();
- ASSERT_TRUE(delegate.action());
- EXPECT_EQ(ToolbarActionsBarBubbleDelegate::DISMISSED, *delegate.action());
+ ASSERT_TRUE(delegate.close_action());
+ EXPECT_EQ(ToolbarActionsBarBubbleDelegate::CLOSE_DISMISS,
+ *delegate.close_action());
EXPECT_TRUE(bubble_observer.widget_closed());
}
}

Powered by Google App Engine
This is Rietveld 408576698