Minor fixes in login_database.* and sql/statement.*

components/password_manager/core/browser/login_database.h

Index: components/password_manager/core/browser/login_database.h
diff --git a/components/password_manager/core/browser/login_database.h b/components/password_manager/core/browser/login_database.h
index c992f3eb159e7c18882efab2cf68e33db957d121..47b1a87b429d600ac4e1355e627721c5e67a3b92 100644
--- a/components/password_manager/core/browser/login_database.h
+++ b/components/password_manager/core/browser/login_database.h
@@ -36,7 +36,7 @@ extern const int kCompatibleVersionNumber;
 // the login information.
 class LoginDatabase {
  public:
-  LoginDatabase(const base::FilePath& db_path);
+  explicit LoginDatabase(const base::FilePath& db_path);
   virtual ~LoginDatabase();
 
   // Actually creates/opens the database. If false is returned, no other method
@@ -179,7 +179,7 @@ class LoginDatabase {
   // ENCRYPTION_RESULT_SUCCESS, |form| is not filled.
   static EncryptionResult InitPasswordFormFromStatement(
       autofill::PasswordForm* form,
-      sql::Statement& s);
+      sql::Statement* s);

[msramek, 2016/07/11 11:28:08]

Should we make this const?

(which would actually allow us to keep the reference if we want to)

[vabr (Chromium), 2016/07/11 12:42:32]

Done, including the yak-shaving to make it possible :).

 
   // Gets all blacklisted or all non-blacklisted (depending on |blacklisted|)
   // credentials. On success returns true and overwrites |forms| with the