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 |
index 1b65c6ca5d52618aa25ca88e2d1584ac10e095ce..0fb6111e3110c876b297f6ec8337975c0dd91eb9 100644 |
--- a/chrome/browser/ui/views/extensions/chooser_dialog_view_browsertest.cc |
+++ b/chrome/browser/ui/views/extensions/chooser_dialog_view_browsertest.cc |
@@ -9,11 +9,12 @@ |
#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/gfx/range/range.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" |
@@ -30,7 +31,8 @@ class ChooserDialogViewTest : public ExtensionBrowserTest { |
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_->table_view_for_test(); |
+ 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_); |
@@ -39,6 +41,9 @@ class ChooserDialogViewTest : public ExtensionBrowserTest { |
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: |
@@ -46,6 +51,7 @@ class ChooserDialogViewTest : public ExtensionBrowserTest { |
ChooserDialogView* chooser_dialog_view_; |
views::TableView* table_view_; |
ui::TableModel* table_model_; |
+ views::StyledLabel* styled_label_; |
private: |
DISALLOW_COPY_AND_ASSIGN(ChooserDialogViewTest); |
@@ -54,15 +60,15 @@ class ChooserDialogViewTest : public ExtensionBrowserTest { |
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(table_view_->RowCount(), 1); |
+ EXPECT_EQ(1, table_view_->RowCount()); |
EXPECT_EQ( |
- table_model_->GetText(0, 0), |
- l10n_util::GetStringUTF16(IDS_DEVICE_CHOOSER_NO_DEVICES_FOUND_PROMPT)); |
+ 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(table_view_->SelectedRowCount(), 0); |
- EXPECT_EQ(table_view_->FirstSelectedRow(), -1); |
+ EXPECT_EQ(0, table_view_->SelectedRowCount()); |
+ EXPECT_EQ(-1, table_view_->FirstSelectedRow()); |
} |
IN_PROC_BROWSER_TEST_F(ChooserDialogViewTest, Accept) { |
@@ -82,5 +88,5 @@ IN_PROC_BROWSER_TEST_F(ChooserDialogViewTest, Close) { |
IN_PROC_BROWSER_TEST_F(ChooserDialogViewTest, ClickStyledLabelLink) { |
EXPECT_CALL(*mock_chooser_controller_, OpenHelpCenterUrl()).Times(1); |
- chooser_dialog_view_->StyledLabelLinkClicked(nullptr, gfx::Range(), 0); |
+ styled_label_->LinkClicked(nullptr, 0); |
} |