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

Unified Diff: components/autofill/core/common/save_password_progress_logger_unittest.cc

Issue 365783002: Autofill: don't require POST method. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: remove method_ member Created 6 years, 5 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: components/autofill/core/common/save_password_progress_logger_unittest.cc
diff --git a/components/autofill/core/common/save_password_progress_logger_unittest.cc b/components/autofill/core/common/save_password_progress_logger_unittest.cc
index bb5f4d085a96e8bca20108f83debbb43feab5a0a..0de91ac06225003fe6420c685ece2a4a6c4b0295 100644
--- a/components/autofill/core/common/save_password_progress_logger_unittest.cc
+++ b/components/autofill/core/common/save_password_progress_logger_unittest.cc
@@ -86,13 +86,11 @@ TEST(SavePasswordProgressLoggerTest, LogHTMLForm) {
TestLogger logger;
logger.LogHTMLForm(SavePasswordProgressLogger::STRING_MESSAGE,
"form_name",
- "post",
GURL("http://example.org/verysecret?verysecret"));
SCOPED_TRACE(testing::Message() << "Log string = ["
<< logger.accumulated_log() << "]");
EXPECT_TRUE(logger.LogsContainSubstring(kTestString));
EXPECT_TRUE(logger.LogsContainSubstring("form_name"));
- EXPECT_TRUE(logger.LogsContainSubstring("POST"));
EXPECT_TRUE(logger.LogsContainSubstring("http://example.org"));
EXPECT_FALSE(logger.LogsContainSubstring("verysecret"));
}

Powered by Google App Engine
This is Rietveld 408576698