Added of new UMA signals in order to be able to discover early if the "save password" feature gets …

chrome/browser/password_manager/password_manager_unittest.cc

 // Copyright (c) 2012 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 <vector>
 
 #include "base/message_loop/message_loop.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/password_manager/mock_password_store.h"
 #include "chrome/browser/password_manager/password_manager.h"
 #include "chrome/browser/password_manager/password_manager_delegate.h"
+#include "chrome/browser/password_manager/password_manager_util.h"
 #include "chrome/browser/password_manager/password_store.h"
 #include "chrome/browser/password_manager/password_store_factory.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/test/base/chrome_render_view_host_test_harness.h"
 #include "chrome/test/base/testing_pref_service_syncable.h"
 #include "chrome/test/base/testing_profile.h"
 #include "content/public/browser/navigation_details.h"
 #include "content/public/common/frame_navigate_params.h"
 #include "content/public/test/test_browser_thread.h"
@@ skipping 506 matching lines @@
   manager()->OnPasswordFormsParsed(observed);  // The post-navigation load.
   manager()->OnPasswordFormsRendered(observed);  // The post-navigation layout.
 }
 
 TEST_F(PasswordManagerTest, SubmissionCallbackTest) {
   manager()->AddSubmissionCallback(SubmissionCallback());
   PasswordForm form = MakeSimpleForm();
   OnPasswordFormSubmitted(form);
   EXPECT_TRUE(FormsAreEqual(form, submitted_form_));
 }
+
+TEST_F(PasswordManagerTest, IsDomainNameMonitoredTest) {
+  std::string domain_name;
+
+  EXPECT_TRUE(password_manager_util::IsDomainNameMonitored(
+      "https://www.linkedin.com", &domain_name));
+  ASSERT_TRUE(domain_name == "linkedin.com");

[Ilya Sherman, 2013/08/29 06:42:21]

nit: EXPECT_EQ

[jdomingos, 2013/08/30 21:09:20]

Done.

+  EXPECT_TRUE(password_manager_util::IsDomainNameMonitored(
+      "https://www.amazon.com", &domain_name));
+  ASSERT_TRUE(domain_name == "amazon.com");
+  EXPECT_FALSE(password_manager_util::IsDomainNameMonitored(
+      "https://www.facebook.com", &domain_name));
+  ASSERT_TRUE(domain_name == "");
+  EXPECT_TRUE(password_manager_util::IsDomainNameMonitored(
+      "wikipedia.org", &domain_name));
+  ASSERT_TRUE(domain_name == "wikipedia.org");
+  EXPECT_FALSE(password_manager_util::IsDomainNameMonitored(
+      "thisisnotwikipedia.org", &domain_name));
+  ASSERT_TRUE(domain_name == "");
+}

[Ilya Sherman, 2013/08/29 06:42:21]

nit: This test should be placed in a file named password_manager_util_unittest.

[jdomingos, 2013/08/30 21:09:20]

Done.