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

Unified Diff: chrome/browser/ui/views/desktop_media_picker_views_deprecated_unittest.cc

Issue 1962073002: Desktop Capture Picker New UI: Non Mac Structure Change (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix Unittest Crash Created 4 years, 7 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/desktop_media_picker_views_deprecated_unittest.cc
diff --git a/chrome/browser/ui/views/desktop_media_picker_views_unittest.cc b/chrome/browser/ui/views/desktop_media_picker_views_deprecated_unittest.cc
similarity index 78%
rename from chrome/browser/ui/views/desktop_media_picker_views_unittest.cc
rename to chrome/browser/ui/views/desktop_media_picker_views_deprecated_unittest.cc
index ea3e4bfb36835b7f88ccf1a2fdbe0ec2c6e9c2d1..e7bf0975143e0ab201ded139fdee0bdc9dbd94d7 100644
--- a/chrome/browser/ui/views/desktop_media_picker_views_unittest.cc
+++ b/chrome/browser/ui/views/desktop_media_picker_views_deprecated_unittest.cc
@@ -2,7 +2,7 @@
// 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/desktop_media_picker_views.h"
+#include "chrome/browser/ui/views/desktop_media_picker_views_deprecated.h"
#include <utility>
@@ -21,12 +21,12 @@
#include "ui/views/window/dialog_client_view.h"
#include "ui/views/window/dialog_delegate.h"
-namespace views {
+namespace deprecated {
-class DesktopMediaPickerViewsTest : public testing::Test {
+class DesktopMediaPickerViewsDeprecatedTest : public testing::Test {
public:
- DesktopMediaPickerViewsTest() {}
- ~DesktopMediaPickerViewsTest() override {}
+ DesktopMediaPickerViewsDeprecatedTest() {}
+ ~DesktopMediaPickerViewsDeprecatedTest() override {}
void SetUp() override {
screen_list_ = new FakeDesktopMediaList();
@@ -39,11 +39,11 @@ class DesktopMediaPickerViewsTest : public testing::Test {
base::string16 app_name = base::ASCIIToUTF16("foo");
picker_views_.reset(new DesktopMediaPickerViews());
- picker_views_->Show(NULL, test_helper_.GetContext(), NULL, app_name,
- app_name, std::move(screen_list),
- std::move(window_list), std::move(tab_list), false,
- base::Bind(&DesktopMediaPickerViewsTest::OnPickerDone,
- base::Unretained(this)));
+ picker_views_->Show(
+ NULL, test_helper_.GetContext(), NULL, app_name, app_name,
+ std::move(screen_list), std::move(window_list), std::move(tab_list),
+ false, base::Bind(&DesktopMediaPickerViewsDeprecatedTest::OnPickerDone,
+ base::Unretained(this)));
}
void TearDown() override {
@@ -68,18 +68,19 @@ class DesktopMediaPickerViewsTest : public testing::Test {
std::unique_ptr<DesktopMediaPickerViews> picker_views_;
};
-TEST_F(DesktopMediaPickerViewsTest, DoneCallbackCalledWhenWindowClosed) {
+TEST_F(DesktopMediaPickerViewsDeprecatedTest,
+ DoneCallbackCalledWhenWindowClosed) {
EXPECT_CALL(*this, OnPickerDone(content::DesktopMediaID()));
GetPickerDialogView()->GetWidget()->Close();
base::RunLoop().RunUntilIdle();
}
-TEST_F(DesktopMediaPickerViewsTest, DoneCallbackCalledOnOkButtonPressed) {
+TEST_F(DesktopMediaPickerViewsDeprecatedTest,
+ DoneCallbackCalledOnOkButtonPressed) {
const int kFakeId = 222;
- EXPECT_CALL(*this,
- OnPickerDone(content::DesktopMediaID(
- content::DesktopMediaID::TYPE_WINDOW, kFakeId)));
+ EXPECT_CALL(*this, OnPickerDone(content::DesktopMediaID(
+ content::DesktopMediaID::TYPE_WINDOW, kFakeId)));
window_list_->AddSource(kFakeId);
EXPECT_FALSE(
@@ -95,7 +96,8 @@ TEST_F(DesktopMediaPickerViewsTest, DoneCallbackCalledOnOkButtonPressed) {
// Verifies that a MediaSourceView is selected with mouse left click and
// original selected MediaSourceView gets unselected.
-TEST_F(DesktopMediaPickerViewsTest, SelectMediaSourceViewOnSingleClick) {
+TEST_F(DesktopMediaPickerViewsDeprecatedTest,
+ SelectMediaSourceViewOnSingleClick) {
window_list_->AddSource(0);
window_list_->AddSource(1);
@@ -125,11 +127,10 @@ TEST_F(DesktopMediaPickerViewsTest, SelectMediaSourceViewOnSingleClick) {
EXPECT_TRUE(source_view_1->is_selected());
}
-TEST_F(DesktopMediaPickerViewsTest, DoneCallbackCalledOnDoubleClick) {
+TEST_F(DesktopMediaPickerViewsDeprecatedTest, DoneCallbackCalledOnDoubleClick) {
const int kFakeId = 222;
- EXPECT_CALL(*this,
- OnPickerDone(content::DesktopMediaID(
- content::DesktopMediaID::TYPE_WINDOW, kFakeId)));
+ EXPECT_CALL(*this, OnPickerDone(content::DesktopMediaID(
+ content::DesktopMediaID::TYPE_WINDOW, kFakeId)));
window_list_->AddSource(kFakeId);
@@ -143,11 +144,10 @@ TEST_F(DesktopMediaPickerViewsTest, DoneCallbackCalledOnDoubleClick) {
base::RunLoop().RunUntilIdle();
}
-TEST_F(DesktopMediaPickerViewsTest, DoneCallbackCalledOnDoubleTap) {
+TEST_F(DesktopMediaPickerViewsDeprecatedTest, DoneCallbackCalledOnDoubleTap) {
const int kFakeId = 222;
- EXPECT_CALL(*this,
- OnPickerDone(content::DesktopMediaID(
- content::DesktopMediaID::TYPE_WINDOW, kFakeId)));
+ EXPECT_CALL(*this, OnPickerDone(content::DesktopMediaID(
+ content::DesktopMediaID::TYPE_WINDOW, kFakeId)));
window_list_->AddSource(kFakeId);
ui::GestureEventDetails details(ui::ET_GESTURE_TAP);
@@ -159,14 +159,14 @@ TEST_F(DesktopMediaPickerViewsTest, DoneCallbackCalledOnDoubleTap) {
base::RunLoop().RunUntilIdle();
}
-TEST_F(DesktopMediaPickerViewsTest, CancelButtonAlwaysEnabled) {
+TEST_F(DesktopMediaPickerViewsDeprecatedTest, CancelButtonAlwaysEnabled) {
EXPECT_TRUE(
GetPickerDialogView()->IsDialogButtonEnabled(ui::DIALOG_BUTTON_CANCEL));
}
// Verifies that the MediaSourceView is added or removed when |media_list_| is
// updated.
-TEST_F(DesktopMediaPickerViewsTest, AddAndRemoveMediaSource) {
+TEST_F(DesktopMediaPickerViewsDeprecatedTest, AddAndRemoveMediaSource) {
// No media source at first.
EXPECT_EQ(NULL, GetPickerDialogView()->GetMediaSourceViewForTesting(0));
@@ -183,7 +183,7 @@ TEST_F(DesktopMediaPickerViewsTest, AddAndRemoveMediaSource) {
// Verifies that focusing the MediaSourceView marks it selected and the
// original selected MediaSourceView gets unselected.
-TEST_F(DesktopMediaPickerViewsTest, FocusMediaSourceViewToSelect) {
+TEST_F(DesktopMediaPickerViewsDeprecatedTest, FocusMediaSourceViewToSelect) {
window_list_->AddSource(0);
window_list_->AddSource(1);
@@ -208,7 +208,7 @@ TEST_F(DesktopMediaPickerViewsTest, FocusMediaSourceViewToSelect) {
EXPECT_TRUE(source_view_1->is_selected());
}
-TEST_F(DesktopMediaPickerViewsTest, OkButtonDisabledWhenNoSelection) {
+TEST_F(DesktopMediaPickerViewsDeprecatedTest, OkButtonDisabledWhenNoSelection) {
window_list_->AddSource(111);
EXPECT_FALSE(
@@ -224,8 +224,8 @@ TEST_F(DesktopMediaPickerViewsTest, OkButtonDisabledWhenNoSelection) {
}
// Verifies that the MediaListView get the initial focus.
-TEST_F(DesktopMediaPickerViewsTest, ListViewHasInitialFocus) {
+TEST_F(DesktopMediaPickerViewsDeprecatedTest, ListViewHasInitialFocus) {
EXPECT_TRUE(GetPickerDialogView()->GetMediaListViewForTesting()->HasFocus());
}
-} // namespace views
+} // namespace deprecated

Powered by Google App Engine
This is Rietveld 408576698