Clean Obsolete HTTP Data from the Password Store

components/password_manager/core/browser/mock_password_store.h

Index: components/password_manager/core/browser/mock_password_store.h
diff --git a/components/password_manager/core/browser/mock_password_store.h b/components/password_manager/core/browser/mock_password_store.h
index 7d05812bf30d20e9879d03914230768c4d8336e5..d2f8244e3ea194c1cc025150fd1bc7570462fade 100644
--- a/components/password_manager/core/browser/mock_password_store.h
+++ b/components/password_manager/core/browser/mock_password_store.h
@@ -19,6 +19,9 @@ class MockPasswordStore : public PasswordStore {
  public:
   MockPasswordStore();
 
+  bool Init(const syncer::SyncableService::StartSyncFlare& flare) override {
+    return true;
+  };

[vasilii, 2017/03/23 17:39:26]

The style guide says "Stop inlining virtual methods". It's for performance
reasons which are not applicable here but still..

[vabr (Chromium), 2017/03/24 07:02:42]

The advantage of having the method here defined in the class definition is that
it's clear that it does nothing. If you declare it only (and define elsewhere),
there should be a comment saying: "This just returns true." The current code
seems to express the same with less lines, so I would suggest keeping it (but
won't fight either version).

   MOCK_METHOD1(RemoveLogin, void(const autofill::PasswordForm&));
   MOCK_METHOD2(GetLogins,
                void(const PasswordStore::FormDigest&, PasswordStoreConsumer*));