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

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

Issue 2171103002: Add test for WebBluetooth chooser spinner, status text and rescan button (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address comments Created 4 years, 5 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/chooser_dialog_view_browsertest.cc
diff --git a/chrome/browser/ui/views/extensions/chooser_dialog_view_browsertest.cc b/chrome/browser/ui/views/extensions/chooser_dialog_view_browsertest.cc
deleted file mode 100644
index 0fb6111e3110c876b297f6ec8337975c0dd91eb9..0000000000000000000000000000000000000000
--- a/chrome/browser/ui/views/extensions/chooser_dialog_view_browsertest.cc
+++ /dev/null
@@ -1,92 +0,0 @@
-// 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 "chrome/browser/ui/views/extensions/chooser_dialog_view.h"
-
-#include "base/macros.h"
-#include "chrome/browser/chooser_controller/mock_chooser_controller.h"
-#include "chrome/browser/extensions/extension_browsertest.h"
-#include "chrome/browser/platform_util.h"
-#include "chrome/browser/ui/browser_window.h"
-#include "chrome/browser/ui/views/chooser_content_view.h"
-#include "chrome/grit/generated_resources.h"
-#include "testing/gmock/include/gmock/gmock.h"
-#include "testing/gtest/include/gtest/gtest.h"
-#include "ui/base/l10n/l10n_util.h"
-#include "ui/views/controls/styled_label.h"
-#include "ui/views/controls/table/table_view.h"
-#include "ui/views/widget/widget.h"
-#include "ui/views/window/dialog_client_view.h"
-
-class ChooserDialogViewTest : public ExtensionBrowserTest {
- public:
- ChooserDialogViewTest() {}
- ~ChooserDialogViewTest() override {}
-
- void SetUpOnMainThread() override {
- std::unique_ptr<MockChooserController> mock_chooser_controller(
- new MockChooserController(nullptr));
- mock_chooser_controller_ = mock_chooser_controller.get();
- std::unique_ptr<ChooserDialogView> chooser_dialog_view(
- new ChooserDialogView(std::move(mock_chooser_controller)));
- chooser_dialog_view_ = chooser_dialog_view.get();
- table_view_ = chooser_dialog_view_->chooser_content_view_for_test()
- ->table_view_for_test();
- ASSERT_TRUE(table_view_);
- table_model_ = table_view_->model();
- ASSERT_TRUE(table_model_);
- views::Widget* modal_dialog = views::DialogDelegate::CreateDialogWidget(
- chooser_dialog_view.release(), nullptr,
- platform_util::GetViewForWindow(
- browser()->window()->GetNativeWindow()));
- modal_dialog->Show();
- styled_label_ = chooser_dialog_view_->chooser_content_view_for_test()
- ->styled_label_for_test();
- ASSERT_TRUE(styled_label_);
- }
-
- protected:
- MockChooserController* mock_chooser_controller_;
- ChooserDialogView* chooser_dialog_view_;
- views::TableView* table_view_;
- ui::TableModel* table_model_;
- views::StyledLabel* styled_label_;
-
- private:
- DISALLOW_COPY_AND_ASSIGN(ChooserDialogViewTest);
-};
-
-IN_PROC_BROWSER_TEST_F(ChooserDialogViewTest, InitialState) {
- // Since "No devices found." needs to be displayed on the |table_view_|,
- // the number of rows is 1.
- EXPECT_EQ(1, table_view_->RowCount());
- EXPECT_EQ(
- l10n_util::GetStringUTF16(IDS_DEVICE_CHOOSER_NO_DEVICES_FOUND_PROMPT),
- table_model_->GetText(0, 0));
- // |table_view_| should be disabled since there is no option shown.
- EXPECT_FALSE(table_view_->enabled());
- // No option selected.
- EXPECT_EQ(0, table_view_->SelectedRowCount());
- EXPECT_EQ(-1, table_view_->FirstSelectedRow());
-}
-
-IN_PROC_BROWSER_TEST_F(ChooserDialogViewTest, Accept) {
- EXPECT_CALL(*mock_chooser_controller_, Select(testing::_)).Times(1);
- chooser_dialog_view_->Accept();
-}
-
-IN_PROC_BROWSER_TEST_F(ChooserDialogViewTest, Cancel) {
- EXPECT_CALL(*mock_chooser_controller_, Cancel()).Times(1);
- chooser_dialog_view_->Cancel();
-}
-
-IN_PROC_BROWSER_TEST_F(ChooserDialogViewTest, Close) {
- EXPECT_CALL(*mock_chooser_controller_, Close()).Times(1);
- chooser_dialog_view_->Close();
-}
-
-IN_PROC_BROWSER_TEST_F(ChooserDialogViewTest, ClickStyledLabelLink) {
- EXPECT_CALL(*mock_chooser_controller_, OpenHelpCenterUrl()).Times(1);
- styled_label_->LinkClicked(nullptr, 0);
-}

Powered by Google App Engine
This is Rietveld 408576698