[BlackBerry] Use OwnPtr for CredentialBackingStore members
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 8 Mar 2013 08:29:02 +0000 (08:29 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 8 Mar 2013 08:29:02 +0000 (08:29 +0000)
https://bugs.webkit.org/show_bug.cgi?id=111737

Patch by Carlos Garcia Campos <cgarcia@igalia.com> on 2013-03-08
Reviewed by Rob Buis.

It simplifies the code a bit.

* platform/network/blackberry/CredentialBackingStore.cpp:
(WebCore::CredentialBackingStore::~CredentialBackingStore):
(WebCore::CredentialBackingStore::open):
(WebCore::CredentialBackingStore::certMgrWrapper):
* platform/network/blackberry/CredentialBackingStore.h:
(CredentialBackingStore):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@145190 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/platform/network/blackberry/CredentialBackingStore.cpp
Source/WebCore/platform/network/blackberry/CredentialBackingStore.h

index d81b783..2bee0d2 100644 (file)
@@ -1,3 +1,19 @@
+2013-03-08  Carlos Garcia Campos  <cgarcia@igalia.com>
+
+        [BlackBerry] Use OwnPtr for CredentialBackingStore members
+        https://bugs.webkit.org/show_bug.cgi?id=111737
+
+        Reviewed by Rob Buis.
+
+        It simplifies the code a bit.
+
+        * platform/network/blackberry/CredentialBackingStore.cpp:
+        (WebCore::CredentialBackingStore::~CredentialBackingStore):
+        (WebCore::CredentialBackingStore::open):
+        (WebCore::CredentialBackingStore::certMgrWrapper):
+        * platform/network/blackberry/CredentialBackingStore.h:
+        (CredentialBackingStore):
+
 2013-03-07  Gavin Barraclough  <barraclough@apple.com>
 
         MemoryPressureHandler should fully flush the WebCore & NS URL caches
index 83cdbb4..1035896 100644 (file)
  */
 
 #include "config.h"
-
-#if ENABLE(BLACKBERRY_CREDENTIAL_PERSIST)
 #include "CredentialBackingStore.h"
 
+#if ENABLE(BLACKBERRY_CREDENTIAL_PERSIST)
 #include "CredentialStorage.h"
 #include "FileSystem.h"
 #include "KURL.h"
-#include "NotImplemented.h"
 #include "ProtectionSpaceHash.h"
 #include "SQLiteStatement.h"
 #include <BlackBerryPlatformEncryptor.h>
@@ -59,39 +57,11 @@ CredentialBackingStore& credentialBackingStore()
 }
 
 CredentialBackingStore::CredentialBackingStore()
-    : m_addLoginStatement(0)
-    , m_updateLoginStatement(0)
-    , m_hasLoginStatement(0)
-    , m_getLoginStatement(0)
-    , m_removeLoginStatement(0)
-    , m_addNeverRememberStatement(0)
-    , m_hasNeverRememberStatement(0)
-    , m_removeNeverRememberStatement(0)
-    , m_certMgrWrapper(0)
 {
 }
 
 CredentialBackingStore::~CredentialBackingStore()
 {
-    delete m_certMgrWrapper;
-    m_certMgrWrapper = 0;
-    delete m_addLoginStatement;
-    m_addLoginStatement = 0;
-    delete m_updateLoginStatement;
-    m_updateLoginStatement = 0;
-    delete m_hasLoginStatement;
-    m_hasLoginStatement = 0;
-    delete m_getLoginStatement;
-    m_getLoginStatement = 0;
-    delete m_removeLoginStatement;
-    m_removeLoginStatement = 0;
-    delete m_addNeverRememberStatement;
-    m_addNeverRememberStatement = 0;
-    delete m_hasNeverRememberStatement;
-    m_hasNeverRememberStatement = 0;
-    delete m_removeNeverRememberStatement;
-    m_removeNeverRememberStatement = 0;
-
     if (m_database.isOpen())
         m_database.close();
 }
@@ -122,35 +92,35 @@ bool CredentialBackingStore::open(const String& dbPath)
     }
 
     // Prepare the statements.
-    m_addLoginStatement = new SQLiteStatement(m_database, "INSERT OR REPLACE INTO logins (host, port, service_type, realm, auth_scheme, username, password) VALUES (?, ?, ?, ?, ?, ?, ?)");
+    m_addLoginStatement = adoptPtr(new SQLiteStatement(m_database, "INSERT OR REPLACE INTO logins (host, port, service_type, realm, auth_scheme, username, password) VALUES (?, ?, ?, ?, ?, ?, ?)"));
     HANDLE_SQL_EXEC_FAILURE(m_addLoginStatement->prepare() != SQLResultOk,
         false, "Failed to prepare addLogin statement");
 
-    m_updateLoginStatement = new SQLiteStatement(m_database, "UPDATE logins SET username = ?, password = ? WHERE host = ? AND port = ? AND service_type = ? AND realm = ? AND auth_scheme = ?");
+    m_updateLoginStatement = adoptPtr(new SQLiteStatement(m_database, "UPDATE logins SET username = ?, password = ? WHERE host = ? AND port = ? AND service_type = ? AND realm = ? AND auth_scheme = ?"));
     HANDLE_SQL_EXEC_FAILURE(m_updateLoginStatement->prepare() != SQLResultOk,
         false, "Failed to prepare updateLogin statement");
 
-    m_hasLoginStatement = new SQLiteStatement(m_database, "SELECT COUNT(*) FROM logins WHERE host = ? AND port = ? AND service_type = ? AND realm = ? AND auth_scheme = ?");
+    m_hasLoginStatement = adoptPtr(new SQLiteStatement(m_database, "SELECT COUNT(*) FROM logins WHERE host = ? AND port = ? AND service_type = ? AND realm = ? AND auth_scheme = ?"));
     HANDLE_SQL_EXEC_FAILURE(m_hasLoginStatement->prepare() != SQLResultOk,
         false, "Failed to prepare hasLogin statement");
 
-    m_getLoginStatement = new SQLiteStatement(m_database, "SELECT username, password FROM logins WHERE host = ? AND port = ? AND service_type = ? AND realm = ? AND auth_scheme = ?");
+    m_getLoginStatement = adoptPtr(new SQLiteStatement(m_database, "SELECT username, password FROM logins WHERE host = ? AND port = ? AND service_type = ? AND realm = ? AND auth_scheme = ?"));
     HANDLE_SQL_EXEC_FAILURE(m_getLoginStatement->prepare() != SQLResultOk,
         false, "Failed to prepare getLogin statement");
 
-    m_removeLoginStatement = new SQLiteStatement(m_database, "DELETE FROM logins WHERE host = ? AND port = ? AND service_type = ? AND realm = ? AND auth_scheme = ?");
+    m_removeLoginStatement = adoptPtr(new SQLiteStatement(m_database, "DELETE FROM logins WHERE host = ? AND port = ? AND service_type = ? AND realm = ? AND auth_scheme = ?"));
     HANDLE_SQL_EXEC_FAILURE(m_removeLoginStatement->prepare() != SQLResultOk,
         false, "Failed to prepare removeLogin statement");
 
-    m_addNeverRememberStatement = new SQLiteStatement(m_database, "INSERT OR REPLACE INTO never_remember (host, port, service_type, realm, auth_scheme) VALUES (?, ?, ?, ?, ?)");
+    m_addNeverRememberStatement = adoptPtr(new SQLiteStatement(m_database, "INSERT OR REPLACE INTO never_remember (host, port, service_type, realm, auth_scheme) VALUES (?, ?, ?, ?, ?)"));
     HANDLE_SQL_EXEC_FAILURE(m_addNeverRememberStatement->prepare() != SQLResultOk,
         false, "Failed to prepare addNeverRemember statement");
 
-    m_hasNeverRememberStatement = new SQLiteStatement(m_database, "SELECT COUNT(*) FROM never_remember WHERE host = ? AND port = ? AND service_type = ? AND realm = ? AND auth_scheme = ?");
+    m_hasNeverRememberStatement = adoptPtr(new SQLiteStatement(m_database, "SELECT COUNT(*) FROM never_remember WHERE host = ? AND port = ? AND service_type = ? AND realm = ? AND auth_scheme = ?"));
     HANDLE_SQL_EXEC_FAILURE(m_hasNeverRememberStatement->prepare() != SQLResultOk,
         false, "Failed to prepare hasNeverRemember statement");
 
-    m_removeNeverRememberStatement = new SQLiteStatement(m_database, "DELETE FROM never_remember WHERE host = ? AND port = ? AND service_type = ? AND realm = ? AND auth_scheme = ?");
+    m_removeNeverRememberStatement = adoptPtr(new SQLiteStatement(m_database, "DELETE FROM never_remember WHERE host = ? AND port = ? AND service_type = ? AND realm = ? AND auth_scheme = ?"));
     HANDLE_SQL_EXEC_FAILURE(m_removeNeverRememberStatement->prepare() != SQLResultOk,
         false, "Failed to prepare removeNeverRemember statement");
 
@@ -430,9 +400,9 @@ String CredentialBackingStore::decryptedString(const String& cipherText) const
 BlackBerry::Platform::CertMgrWrapper* CredentialBackingStore::certMgrWrapper()
 {
     if (!m_certMgrWrapper)
-        m_certMgrWrapper = new BlackBerry::Platform::CertMgrWrapper();
+        m_certMgrWrapper = adoptPtr(new BlackBerry::Platform::CertMgrWrapper());
 
-    return m_certMgrWrapper;
+    return m_certMgrWrapper.get();
 }
 
 
index 30c6062..c5d868c 100644 (file)
@@ -25,6 +25,7 @@
 #include "SQLiteDatabase.h"
 
 #include <BlackBerryPlatformMisc.h>
+#include <wtf/OwnPtr.h>
 
 namespace BlackBerry {
 namespace Platform {
@@ -61,16 +62,16 @@ private:
     BlackBerry::Platform::CertMgrWrapper* certMgrWrapper();
 
     SQLiteDatabase m_database;
-    SQLiteStatement* m_addLoginStatement;
-    SQLiteStatement* m_updateLoginStatement;
-    SQLiteStatement* m_hasLoginStatement;
-    SQLiteStatement* m_getLoginStatement;
-    SQLiteStatement* m_removeLoginStatement;
-    SQLiteStatement* m_addNeverRememberStatement;
-    SQLiteStatement* m_hasNeverRememberStatement;
-    SQLiteStatement* m_removeNeverRememberStatement;
-
-    BlackBerry::Platform::CertMgrWrapper* m_certMgrWrapper;
+    OwnPtr<SQLiteStatement> m_addLoginStatement;
+    OwnPtr<SQLiteStatement> m_updateLoginStatement;
+    OwnPtr<SQLiteStatement> m_hasLoginStatement;
+    OwnPtr<SQLiteStatement> m_getLoginStatement;
+    OwnPtr<SQLiteStatement> m_removeLoginStatement;
+    OwnPtr<SQLiteStatement> m_addNeverRememberStatement;
+    OwnPtr<SQLiteStatement> m_hasNeverRememberStatement;
+    OwnPtr<SQLiteStatement> m_removeNeverRememberStatement;
+
+    OwnPtr<BlackBerry::Platform::CertMgrWrapper> m_certMgrWrapper;
 
     DISABLE_COPY(CredentialBackingStore)
 };