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

Unified Diff: chrome/browser/ui/autofill/save_card_bubble_controller_impl_unittest.cc

Issue 2488743002: Remove legal messages from save card bubble before offering local save. (Closed)
Patch Set: Fix test crash Created 4 years, 1 month 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
« no previous file with comments | « chrome/browser/ui/autofill/save_card_bubble_controller_impl.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/autofill/save_card_bubble_controller_impl_unittest.cc
diff --git a/chrome/browser/ui/autofill/save_card_bubble_controller_impl_unittest.cc b/chrome/browser/ui/autofill/save_card_bubble_controller_impl_unittest.cc
index 15e165e14f380462e50b472986f51695c6544d19..95f7a046a9c20eab5774218adf99d32bd15d4589 100644
--- a/chrome/browser/ui/autofill/save_card_bubble_controller_impl_unittest.cc
+++ b/chrome/browser/ui/autofill/save_card_bubble_controller_impl_unittest.cc
@@ -125,6 +125,15 @@ class SaveCardBubbleControllerImplTest : public BrowserWithTestWindowTest {
DISALLOW_COPY_AND_ASSIGN(SaveCardBubbleControllerImplTest);
};
+// Tests that the legal message lines vector is empty when doing a local save so
+// that no legal messages will be shown to the user in that case.
+TEST_F(SaveCardBubbleControllerImplTest, LegalMessageLinesEmptyOnLocalSave) {
+ ShowUploadBubble();
+ controller()->OnBubbleClosed();
+ ShowLocalBubble();
+ EXPECT_TRUE(controller()->GetLegalMessageLines().empty());
+}
+
TEST_F(SaveCardBubbleControllerImplTest, Metrics_Local_FirstShow_ShowBubble) {
base::HistogramTester histogram_tester;
ShowLocalBubble();
« no previous file with comments | « chrome/browser/ui/autofill/save_card_bubble_controller_impl.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698