Removed unused methods from PasswordStore

Change-Id: I37527df498582dd0df2a3433d815fc8c303983fc
Bug: 875768
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1823862Reviewed-by: Vadym Doroshenko <dvadym@chromium.org>
Commit-Queue: Matthias Körber <koerber@google.com>
Cr-Commit-Position: refs/heads/master@{#699790}

components/password_manager/core/browser/password_store.cc

@@ -328,12 +328,6 @@ void PasswordStore::RemoveSiteStats(const GURL& origin_domain) {
       base::BindOnce(&PasswordStore::RemoveSiteStatsImpl, this, origin_domain));
 }
 
-void PasswordStore::GetAllSiteStats(PasswordStoreConsumer* consumer) {
-  DCHECK(main_task_runner_->RunsTasksInCurrentSequence());
-  PostStatsTaskAndReplyToConsumerWithResult(
-      consumer, base::BindOnce(&PasswordStore::GetAllSiteStatsImpl, this));
-}
-
 void PasswordStore::GetSiteStats(const GURL& origin_domain,
                                  PasswordStoreConsumer* consumer) {
   DCHECK(main_task_runner_->RunsTasksInCurrentSequence());
@@ -378,11 +372,6 @@ bool PasswordStore::ScheduleTask(base::OnceClosure task) {
   return false;
 }
 
-scoped_refptr<base::SequencedTaskRunner>
-PasswordStore::GetBackgroundTaskRunner() {
-  return background_task_runner_;
-}
-
 bool PasswordStore::IsAbleToSavePasswords() const {
   DCHECK(main_task_runner_->RunsTasksInCurrentSequence());
   return init_status_ == InitStatus::kSuccess;

components/password_manager/core/browser/password_store.h

@@ -236,10 +236,6 @@ class PasswordStore : protected PasswordStoreSync,
   // Removes the statistics for |origin_domain|.
   void RemoveSiteStats(const GURL& origin_domain);
 
-  // Retrieves the statistics for all sites and notifies |consumer| on
-  // completion. The request will be cancelled if the consumer is destroyed.
-  void GetAllSiteStats(PasswordStoreConsumer* consumer);
-
   // Retrieves the statistics for |origin_domain| and notifies |consumer| on
   // completion. The request will be cancelled if the consumer is destroyed.
   void GetSiteStats(const GURL& origin_domain, PasswordStoreConsumer* consumer);
@@ -266,8 +262,6 @@ class PasswordStore : protected PasswordStoreSync,
   // Schedules the given |task| to be run on the PasswordStore's TaskRunner.
   bool ScheduleTask(base::OnceClosure task);
 
-  scoped_refptr<base::SequencedTaskRunner> GetBackgroundTaskRunner();
-
   // Returns true iff initialization was successful.
   virtual bool IsAbleToSavePasswords() const;
 

components/password_manager/core/browser/password_store_default.cc

@@ -296,11 +296,4 @@ void PasswordStoreDefault::ResetLoginDB() {
   login_db_.reset();
 }
 
-#if defined(USE_X11)
-void PasswordStoreDefault::SetLoginDB(std::unique_ptr<LoginDatabase> login_db) {
-  DCHECK(background_task_runner()->RunsTasksInCurrentSequence());
-  login_db_ = std::move(login_db);
-}
-#endif  // defined(USE_X11)
-
 }  // namespace password_manager

components/password_manager/core/browser/password_store_default.h

@@ -25,15 +25,6 @@ class PasswordStoreDefault : public PasswordStore {
 
   void ShutdownOnUIThread() override;
 
-#if defined(USE_X11)
-  // Dispose the current |login_db_| and use |login_db|. |login_db| is expected
-  // to have been initialised. A null value is equivalent to a database which
-  // can't be opened.
-  // TODO(crbug.com/571003) This is only used to migrate Linux to an encrypted
-  // LoginDatabase.
-  void SetLoginDB(std::unique_ptr<LoginDatabase> login_db);
-#endif  // defined(USE_X11)
-
   // To be used only for testing or in subclasses.
   LoginDatabase* login_db() const { return login_db_.get(); }