Save passwords for HTTP+FTP authentication....

chrome/browser/password_manager/password_form_manager_unittest.cc

-// Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
+// Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "testing/gtest/include/gtest/gtest.h"
 
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/password_manager/password_form_manager.h"
 #include "chrome/browser/password_manager/password_manager.h"
 #include "chrome/browser/profile_manager.h"
@@ skipping 181 matching lines @@
   EXPECT_FALSE(manager->IsNewLogin());
   // We bless our saved PasswordForm entry with the action URL of the
   // observed form.
   EXPECT_EQ(observed_form()->action,
             GetPendingCredentials(manager.get())->action);
 }
 
 TEST_F(PasswordFormManagerTest, TestValidForms) {
   // User submits credentials for the observed form.
   PasswordForm credentials = *observed_form();
+  credentials.scheme = PasswordForm::SCHEME_HTML;
   credentials.username_value = saved_match()->username_value;
   credentials.password_value = saved_match()->password_value;
-  credentials.preferred = true;
 
+  // Form with both username_element and password_element.
   PasswordFormManager manager1(profile(), NULL, credentials, false);
   SimulateMatchingPhase(&manager1, false);
-  manager1.ProvisionallySave(credentials);
-
-  // Valid form.
   EXPECT_TRUE(manager1.HasValidPasswordForm());
 
+  // Form without a username_element but with a password_element.
   credentials.username_element.clear();
   PasswordFormManager manager2(profile(), NULL, credentials, false);
   SimulateMatchingPhase(&manager2, false);
-  manager2.ProvisionallySave(credentials);
-
-  // Invalid form - no username.
   EXPECT_FALSE(manager2.HasValidPasswordForm());
 
+  // Form without a password_element but with a username_element.
   credentials.username_element = saved_match()->username_element;
   credentials.password_element.clear();
   PasswordFormManager manager3(profile(), NULL, credentials, false);
   SimulateMatchingPhase(&manager3, false);
-  manager3.ProvisionallySave(credentials);
+  EXPECT_FALSE(manager3.HasValidPasswordForm());
 
-  // Invalid form - no password.
-  EXPECT_FALSE(manager3.HasValidPasswordForm());
+  // Form with neither a password_element nor a username_element.
+  credentials.username_element.clear();
+  credentials.password_element.clear();
+  PasswordFormManager manager4(profile(), NULL, credentials, false);
+  SimulateMatchingPhase(&manager4, false);
+  EXPECT_FALSE(manager4.HasValidPasswordForm());
 }
 
+TEST_F(PasswordFormManagerTest, TestValidFormsBasic) {
+  // User submits credentials for the observed form.
+  PasswordForm credentials = *observed_form();
+  credentials.scheme = PasswordForm::SCHEME_BASIC;
+  credentials.username_value = saved_match()->username_value;
+  credentials.password_value = saved_match()->password_value;
+
+  // Form with both username_element and password_element.
+  PasswordFormManager manager1(profile(), NULL, credentials, false);
+  SimulateMatchingPhase(&manager1, false);
+  EXPECT_TRUE(manager1.HasValidPasswordForm());
+
+  // Form without a username_element but with a password_element.
+  credentials.username_element.clear();
+  PasswordFormManager manager2(profile(), NULL, credentials, false);
+  SimulateMatchingPhase(&manager2, false);
+  EXPECT_TRUE(manager2.HasValidPasswordForm());
+
+  // Form without a password_element but with a username_element.
+  credentials.username_element = saved_match()->username_element;
+  credentials.password_element.clear();
+  PasswordFormManager manager3(profile(), NULL, credentials, false);
+  SimulateMatchingPhase(&manager3, false);
+  EXPECT_TRUE(manager3.HasValidPasswordForm());
+
+  // Form with neither a password_element nor a username_element.
+  credentials.username_element.clear();
+  credentials.password_element.clear();
+  PasswordFormManager manager4(profile(), NULL, credentials, false);
+  SimulateMatchingPhase(&manager4, false);
+  EXPECT_TRUE(manager4.HasValidPasswordForm());
+}