Clean up Storage code
[WebKit-https.git] / Source / WebKit / Storage / StorageNamespaceImpl.cpp
index 7d85c1c..56a0192 100644 (file)
@@ -33,7 +33,9 @@
 #include <wtf/NeverDestroyed.h>
 #include <wtf/text/StringHash.h>
 
-namespace WebCore {
+using namespace WebCore;
+
+namespace WebKit {
 
 static HashMap<String, StorageNamespaceImpl*>& localStorageNamespaceMap()
 {
@@ -42,21 +44,21 @@ static HashMap<String, StorageNamespaceImpl*>& localStorageNamespaceMap()
     return localStorageNamespaceMap;
 }
 
-RefPtr<StorageNamespaceImpl> StorageNamespaceImpl::createSessionStorageNamespace(unsigned quota)
+Ref<StorageNamespaceImpl> StorageNamespaceImpl::createSessionStorageNamespace(unsigned quota)
 {
-    return adoptRef(new StorageNamespaceImpl(SessionStorage, String(), quota));
+    return adoptRef(*new StorageNamespaceImpl(SessionStorage, String(), quota));
 }
 
-RefPtr<StorageNamespaceImpl> StorageNamespaceImpl::getOrCreateLocalStorageNamespace(const String& databasePath, unsigned quota)
+Ref<StorageNamespaceImpl> StorageNamespaceImpl::getOrCreateLocalStorageNamespace(const String& databasePath, unsigned quota)
 {
     ASSERT(!databasePath.isNull());
 
     auto& slot = localStorageNamespaceMap().add(databasePath, nullptr).iterator->value;
     if (slot)
-        return slot;
+        return *slot;
 
-    RefPtr<StorageNamespaceImpl> storageNamespace = adoptRef(new StorageNamespaceImpl(LocalStorage, databasePath, quota));
-    slot = storageNamespace.get();
+    Ref<StorageNamespaceImpl> storageNamespace = adoptRef(*new StorageNamespaceImpl(LocalStorage, databasePath, quota));
+    slot = storageNamespace.ptr();
 
     return storageNamespace;
 }
@@ -105,11 +107,11 @@ RefPtr<StorageArea> StorageNamespaceImpl::storageArea(RefPtr<SecurityOrigin>&& o
     ASSERT(!m_isShutdown);
 
     RefPtr<StorageAreaImpl> storageArea;
-    if ((storageArea = m_storageAreaMap.get(origin)))
+    if ((storageArea = m_storageAreaMap.get(origin.get())))
         return storageArea;
 
-    storageArea = StorageAreaImpl::create(m_storageType, origin.copyRef(), m_syncManager, m_quota);
-    m_storageAreaMap.set(WTFMove(origin), storageArea);
+    storageArea = StorageAreaImpl::create(m_storageType, origin.get(), m_syncManager.get(), m_quota);
+    m_storageAreaMap.set(origin, storageArea.get());
     return storageArea;
 }