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

Unified Diff: chrome/renderer/autofill/form_autofill_browsertest.cc

Issue 23140005: Added of new UMA signals in order to be able to discover early if the "save password" feature gets … (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 4 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/renderer/autofill/form_autofill_browsertest.cc
diff --git a/chrome/renderer/autofill/form_autofill_browsertest.cc b/chrome/renderer/autofill/form_autofill_browsertest.cc
index 68fa4c7c3bbecb351b2d386fc8a8bfe1e2823909..a877a1e74f419817a1c1a49343156c55c75f4f95 100644
--- a/chrome/renderer/autofill/form_autofill_browsertest.cc
+++ b/chrome/renderer/autofill/form_autofill_browsertest.cc
@@ -3038,4 +3038,13 @@ TEST_F(FormAutofillTest, SelectOneAsText) {
EXPECT_FORM_FIELD_DATA_EQUALS(expected, fields[2]);
}
+TEST_F(FormAutofillTest, ExtractHostNameFromUrlTest) {
vabr (Chromium) 2013/08/14 14:39:58 The tested function has been removed, right? If so
jdomingos 2013/08/27 18:04:15 Done.
+ EXPECT_EQ(ExtractHostNameFromUrl("www.example.com"), "example");
+ EXPECT_EQ(ExtractHostNameFromUrl("test"), "");
+ EXPECT_EQ(ExtractHostNameFromUrl(".."), "");
+ EXPECT_EQ(ExtractHostNameFromUrl("test..com"), "");
+ EXPECT_EQ(ExtractHostNameFromUrl(""), "");
+ EXPECT_EQ(ExtractHostNameFromUrl("."), "");
+}
+
} // namespace autofill

Powered by Google App Engine
This is Rietveld 408576698