[Password Manager] Add UMA stats for custom passphrase users.

components/password_manager/core/browser/password_generation_manager_unittest.cc

 // Copyright 2014 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/prefs/pref_registry_simple.h"
 #include "base/prefs/pref_service.h"
 #include "base/prefs/testing_pref_service.h"
 #include "base/strings/utf_string_conversions.h"
 #include "components/autofill/core/browser/autofill_field.h"
 #include "components/autofill/core/browser/autofill_metrics.h"
 #include "components/autofill/core/browser/form_structure.h"
 #include "components/autofill/core/common/form_data.h"
 #include "components/autofill/core/common/form_field_data.h"
 #include "components/password_manager/core/browser/password_autofill_manager.h"
 #include "components/password_manager/core/browser/password_generation_manager.h"
 #include "components/password_manager/core/browser/password_manager.h"
 #include "components/password_manager/core/browser/stub_password_manager_client.h"
 #include "components/password_manager/core/browser/stub_password_manager_driver.h"
+#include "components/password_manager/core/browser/test_password_store.h"
 #include "components/password_manager/core/common/password_manager_pref_names.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "url/gurl.h"
 
 using base::ASCIIToUTF16;
 
 namespace password_manager {
 
 namespace {
 
@@ skipping 32 matching lines @@
   PasswordManager password_manager_;
   PasswordGenerationManager password_generation_manager_;
   PasswordAutofillManager password_autofill_manager_;
   std::vector<autofill::FormData> found_account_creation_forms_;
   bool is_off_the_record_;
 };
 
 class TestPasswordManagerClient : public StubPasswordManagerClient {
  public:
   TestPasswordManagerClient(scoped_ptr<PrefService> prefs)
-      : prefs_(prefs.Pass()), driver_(this), is_sync_enabled_(false) {}
+      : prefs_(prefs.Pass()),
+        store_(new TestPasswordStore),
+        driver_(this),
+        is_sync_enabled_(false) {}
 
-  PasswordStore* GetPasswordStore() override { return NULL; }
+  ~TestPasswordManagerClient() {

[vabr (Chromium), 2014/10/24 06:30:28]

Should this also be 'override'?

[Garrett Casto, 2014/10/24 18:25:03]

Good catch. Done.

+    store_->Shutdown();
+  }
+
+  PasswordStore* GetPasswordStore() override { return store_.get(); }
   PrefService* GetPrefs() override { return prefs_.get(); }
   PasswordManagerDriver* GetDriver() override { return &driver_; }
-  bool IsPasswordSyncEnabled() override { return is_sync_enabled_; }
+  bool IsPasswordSyncEnabled(CustomPassphraseState state) override {
+    return is_sync_enabled_;
+  }
 
   void set_is_password_sync_enabled(bool enabled) {
     is_sync_enabled_ = enabled;
   }
 
  private:
   scoped_ptr<PrefService> prefs_;
+  scoped_refptr<TestPasswordStore> store_;
   TestPasswordManagerDriver driver_;
   bool is_sync_enabled_;
 };
 
 // Unlike the base AutofillMetrics, exposes copy and assignment constructors,
 // which are handy for briefer test code.  The AutofillMetrics class is
 // stateless, so this is safe.
 class TestAutofillMetrics : public autofill::AutofillMetrics {
  public:
   TestAutofillMetrics() {}
@@ skipping 112 matching lines @@
   // be disabled.
   GetTestDriver()->set_is_off_the_record(true);
   PrefService* prefs = client_->GetPrefs();
   prefs->SetBoolean(prefs::kPasswordManagerSavingEnabled, true);
   client_->set_is_password_sync_enabled(true);
 
   EXPECT_FALSE(IsGenerationEnabled());
 }
 
 }  // namespace password_manager