Adding skeleton code for showing Bubble, asking user to submit URL when Password Manager fails

chrome/browser/password_manager/chrome_password_manager_client_unittest.cc

Index: chrome/browser/password_manager/chrome_password_manager_client_unittest.cc
diff --git a/chrome/browser/password_manager/chrome_password_manager_client_unittest.cc b/chrome/browser/password_manager/chrome_password_manager_client_unittest.cc
index fb35ebb2b5c7aab85ab9f0004f38beec859ceaa7..efff1bbda43a2b40ba76651a9b484182207fedd4 100644
--- a/chrome/browser/password_manager/chrome_password_manager_client_unittest.cc
+++ b/chrome/browser/password_manager/chrome_password_manager_client_unittest.cc
@@ -228,7 +228,14 @@ TEST_F(ChromePasswordManagerClientTest,
   ChromePasswordManagerClient* client = GetClient();
   // TODO(melandory) Since "Ask user to submit URL" functionality is currently
   // in development, so the user should not be asked to submit a URL.
-  EXPECT_FALSE(client->ShouldAskUserToSubmitURL());
+  EXPECT_FALSE(client->ShouldAskUserToSubmitURL(GURL("https://hostname.com/")));
+}
+
+TEST_F(ChromePasswordManagerClientTest, ShouldAskUserToSubmitURLEmptyURL) {
+  ChromePasswordManagerClient* client = GetClient();
+  // TODO(melandory) Since "Ask user to submit URL" functionality is currently

[vabr (Chromium), 2014/12/04 13:25:32]

The TODO comment here seems like a copy&paste mistake. Here we should always
EXPECT_FALSE, no?

[melandory, 2014/12/05 12:56:52]

Done.

+  // in development, so the user should not be asked to submit a URL.
+  EXPECT_FALSE(client->ShouldAskUserToSubmitURL(GURL::EmptyGURL()));
 }
 
 TEST_F(ChromePasswordManagerClientTest, ShouldFilterAutofillResult_Reauth) {