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

Unified Diff: chrome/browser/ui/passwords/manage_passwords_bubble_model_unittest.cc

Issue 765073004: Adding skeleton code for showing Bubble, asking user to submit URL when Password Manager fails (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years 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/passwords/manage_passwords_bubble_model_unittest.cc
diff --git a/chrome/browser/ui/passwords/manage_passwords_bubble_model_unittest.cc b/chrome/browser/ui/passwords/manage_passwords_bubble_model_unittest.cc
index 50a52f264a9da72aa49acec58fce80cea1a8def2..80e02c10bd0a4498c9cd6d0c06f52c148920333e 100644
--- a/chrome/browser/ui/passwords/manage_passwords_bubble_model_unittest.cc
+++ b/chrome/browser/ui/passwords/manage_passwords_bubble_model_unittest.cc
@@ -35,6 +35,14 @@ class ManagePasswordsBubbleModelTest : public testing::Test {
void TearDown() override { model_.reset(); }
+ void PretendNeedToAskUserToSubmitURL() {
+ model_->set_state(
+ password_manager::ui::ASK_USER_TO_REPORT_URL_AND_BUBBLE_STATE);
+ model_->OnBubbleShown(ManagePasswordsBubble::AUTOMATIC);
+ controller()->SetState(
+ password_manager::ui::ASK_USER_TO_REPORT_URL_AND_BUBBLE_STATE);
+ }
+
void PretendPasswordWaiting() {
model_->set_state(password_manager::ui::PENDING_PASSWORD_AND_BUBBLE_STATE);
model_->OnBubbleShown(ManagePasswordsBubble::AUTOMATIC);
@@ -260,6 +268,36 @@ TEST_F(ManagePasswordsBubbleModelTest, ClickCancelCredential) {
1);
}
+TEST_F(ManagePasswordsBubbleModelTest, ClickCollectURL) {
+ base::HistogramTester histogram_tester;
+ PretendNeedToAskUserToSubmitURL();
+ model_->OnCollectURLClicked();
+ model_->OnBubbleHidden();
+ EXPECT_EQ(model_->dismissal_reason(),
+ password_manager::metrics_util::CLICKED_COLLECT_URL);
+ EXPECT_EQ(password_manager::ui::ASK_USER_TO_REPORT_URL_AND_BUBBLE_STATE,
+ model_->state());
+
+ histogram_tester.ExpectUniqueSample(
+ kUIDismissalReasonMetric,
+ password_manager::metrics_util::CLICKED_COLLECT_URL, 1);
+}
+
+TEST_F(ManagePasswordsBubbleModelTest, ClickDoNotCollectURL) {
+ base::HistogramTester histogram_tester;
+ PretendNeedToAskUserToSubmitURL();
+ model_->OnDoNotCollectURLClicked();
+ model_->OnBubbleHidden();
+ EXPECT_EQ(model_->dismissal_reason(),
+ password_manager::metrics_util::CLICKED_DO_NOT_COLLECT_URL);
+ EXPECT_EQ(password_manager::ui::ASK_USER_TO_REPORT_URL_AND_BUBBLE_STATE,
+ model_->state());
+
+ histogram_tester.ExpectUniqueSample(
+ kUIDismissalReasonMetric,
+ password_manager::metrics_util::CLICKED_DO_NOT_COLLECT_URL, 1);
+}
+
TEST_F(ManagePasswordsBubbleModelTest, DismissCredential) {
base::HistogramTester histogram_tester;
PretendCredentialsWaiting();

Powered by Google App Engine
This is Rietveld 408576698