Attachment 241403 Details for Bug 138646 – Patch

[patch] Patch

bug-138646-20141112110226.patch (text/plain), 21.49 KB, created by Gyuyoung Kim on 2014-11-11 18:02:24 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Gyuyoung Kim

Created: 2014-11-11 18:02:24 PST

Size: 21.49 KB

patch

obsolete

>Subversion Revision: 175987
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 65db1e96865dcb84386c367074747b59c08982ad..1cf628707ad8ba248d82ed7cf7dfb10dae5cafd5 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,50 @@
>+2014-11-11  Gyuyoung Kim  <gyuyoung.kim@samsung.com>
>+
>+        Move WebCore/loader to std::unique_ptr
>+        https://bugs.webkit.org/show_bug.cgi?id=138646
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Replace OwnPtr with std::unique_ptr<> and std::make_unique<>.
>+        However this patch leaves ResourceRequest::adopt() because this need
>+        to be handled by different patch.
>+
>+        No new tests, no behaviro changes.
>+
>+        * loader/DocumentLoader.cpp:
>+        (WebCore::DocumentLoader::DocumentLoader):
>+        (WebCore::DocumentLoader::addAllArchiveResources):
>+        (WebCore::DocumentLoader::addArchiveResource):
>+        (WebCore::DocumentLoader::clearArchiveResources):
>+        (WebCore::DocumentLoader::startLoadingMainResource):
>+        * loader/DocumentLoader.h:
>+        * loader/DocumentThreadableLoader.cpp:
>+        (WebCore::DocumentThreadableLoader::makeCrossOriginAccessRequest):
>+        (WebCore::DocumentThreadableLoader::preflightSuccess):
>+        * loader/DocumentThreadableLoader.h:
>+        * loader/SubresourceLoader.cpp:
>+        (WebCore::SubresourceLoader::SubresourceLoader):
>+        (WebCore::SubresourceLoader::didReceiveResponse):
>+        (WebCore::SubresourceLoader::notifyDone):
>+        * loader/SubresourceLoader.h:
>+        * loader/TextTrackLoader.h:
>+        * loader/appcache/ApplicationCacheHost.h:
>+        * loader/appcache/ApplicationCacheStorage.cpp:
>+        (WebCore::ApplicationCacheStorage::storeCopyOfCache):
>+        * loader/archive/cf/LegacyWebArchive.cpp:
>+        (WebCore::LegacyWebArchive::create):
>+        * loader/cache/CachedResourceLoader.cpp:
>+        (WebCore::CachedResourceLoader::requestPreload):
>+        (WebCore::CachedResourceLoader::clearPreloads):
>+        (WebCore::CachedResourceLoader::printPreloadStats):
>+        * loader/cache/CachedResourceLoader.h:
>+        * loader/cache/MemoryCache.h:
>+        * loader/icon/IconDatabase.cpp:
>+        (WebCore::IconDatabase::deleteAllPreparedStatements):
>+        (WebCore::readySQLiteStatement):
>+        * loader/icon/IconDatabase.h:
>+        * loader/icon/IconRecord.h:
>+
> 2014-11-11  Eric Carlson  <eric.carlson@apple.com>
> 
>         [iOS] video is sometimes allowed to play from the background
>diff --git a/Source/WebCore/loader/DocumentLoader.cpp b/Source/WebCore/loader/DocumentLoader.cpp
>index 1575431f90d578e092e64809ef4167f6514296a9..f032f2aefe1e85d16ea40ff1f56189f94d52a8da 100644
>--- a/Source/WebCore/loader/DocumentLoader.cpp
>+++ b/Source/WebCore/loader/DocumentLoader.cpp
>@@ -142,7 +142,7 @@ DocumentLoader::DocumentLoader(const ResourceRequest& req, const SubstituteData&
>     , m_dataLoadTimer(this, &DocumentLoader::handleSubstituteDataLoadNow)
>     , m_waitingForContentPolicy(false)
>     , m_subresourceLoadersArePageCacheAcceptable(false)
>-    , m_applicationCacheHost(adoptPtr(new ApplicationCacheHost(*this)))
>+    , m_applicationCacheHost(std::make_unique<ApplicationCacheHost>(*this))
> {
> }
> 
>@@ -1019,8 +1019,8 @@ void DocumentLoader::setArchive(PassRefPtr<Archive> archive)
> void DocumentLoader::addAllArchiveResources(Archive* archive)
> {
>     if (!m_archiveResourceCollection)
>-        m_archiveResourceCollection = adoptPtr(new ArchiveResourceCollection);
>-        
>+        m_archiveResourceCollection = std::make_unique<ArchiveResourceCollection>();
>+
>     ASSERT(archive);
>     if (!archive)
>         return;
>@@ -1033,7 +1033,7 @@ void DocumentLoader::addAllArchiveResources(Archive* archive)
> void DocumentLoader::addArchiveResource(PassRefPtr<ArchiveResource> resource)
> {
>     if (!m_archiveResourceCollection)
>-        m_archiveResourceCollection = adoptPtr(new ArchiveResourceCollection);
>+        m_archiveResourceCollection = std::make_unique<ArchiveResourceCollection>();
>         
>     ASSERT(resource);
>     if (!resource)
>@@ -1049,7 +1049,7 @@ PassRefPtr<Archive> DocumentLoader::popArchiveForSubframe(const String& frameNam
> 
> void DocumentLoader::clearArchiveResources()
> {
>-    m_archiveResourceCollection.clear();
>+    m_archiveResourceCollection = nullptr;
>     m_substituteResourceDeliveryTimer.stop();
> }
> 
>@@ -1439,7 +1439,7 @@ void DocumentLoader::startLoadingMainResource()
>         // If the load was aborted by clearing m_request, it's possible the ApplicationCacheHost
>         // is now in a state where starting an empty load will be inconsistent. Replace it with
>         // a new ApplicationCacheHost.
>-        m_applicationCacheHost = adoptPtr(new ApplicationCacheHost(*this));
>+        m_applicationCacheHost = std::make_unique<ApplicationCacheHost>(*this);
>         maybeLoadEmpty();
>         return;
>     }
>diff --git a/Source/WebCore/loader/DocumentLoader.h b/Source/WebCore/loader/DocumentLoader.h
>index 50f04a1ce822ba63f03116736c30508910b0a1fa..85762858c8c9c684ef517e8f82103f815d0a660c 100644
>--- a/Source/WebCore/loader/DocumentLoader.h
>+++ b/Source/WebCore/loader/DocumentLoader.h
>@@ -388,7 +388,7 @@ namespace WebCore {
>         SubstituteResourceMap m_pendingSubstituteResources;
>         Timer m_substituteResourceDeliveryTimer;
> 
>-        OwnPtr<ArchiveResourceCollection> m_archiveResourceCollection;
>+        std::unique_ptr<ArchiveResourceCollection> m_archiveResourceCollection;
> #if ENABLE(WEB_ARCHIVE) || ENABLE(MHTML)
>         RefPtr<Archive> m_archive;
>         RefPtr<SharedBuffer> m_parsedArchiveData;
>@@ -415,7 +415,7 @@ namespace WebCore {
>         bool m_subresourceLoadersArePageCacheAcceptable;
> 
>         friend class ApplicationCacheHost;  // for substitute resource delivery
>-        OwnPtr<ApplicationCacheHost> m_applicationCacheHost;
>+        std::unique_ptr<ApplicationCacheHost> m_applicationCacheHost;
> 
> #if ENABLE(CONTENT_FILTERING)
>         std::unique_ptr<ContentFilter> m_contentFilter;
>diff --git a/Source/WebCore/loader/DocumentThreadableLoader.cpp b/Source/WebCore/loader/DocumentThreadableLoader.cpp
>index 58c5b9f60fa1b0ce1436871ab9488436922c87c2..00064794555a4dc55debfd2bc3fc6113b1671c68 100644
>--- a/Source/WebCore/loader/DocumentThreadableLoader.cpp
>+++ b/Source/WebCore/loader/DocumentThreadableLoader.cpp
>@@ -98,14 +98,14 @@ void DocumentThreadableLoader::makeCrossOriginAccessRequest(const ResourceReques
> {
>     ASSERT(m_options.crossOriginRequestPolicy == UseAccessControl);
> 
>-    OwnPtr<ResourceRequest> crossOriginRequest = adoptPtr(new ResourceRequest(request));
>+    auto crossOriginRequest = std::make_unique<ResourceRequest>(request);
>     updateRequestForAccessControl(*crossOriginRequest, securityOrigin(), m_options.allowCredentials());
> 
>     if ((m_options.preflightPolicy == ConsiderPreflight && isSimpleCrossOriginAccessRequest(crossOriginRequest->httpMethod(), crossOriginRequest->httpHeaderFields())) || m_options.preflightPolicy == PreventPreflight)
>         makeSimpleCrossOriginAccessRequest(*crossOriginRequest);
>     else {
>         m_simpleRequest = false;
>-        m_actualRequest = crossOriginRequest.release();
>+        m_actualRequest = WTF::move(crossOriginRequest);
> 
>         if (CrossOriginPreflightResultCache::shared().canSkipPreflight(securityOrigin()->toString(), m_actualRequest->url(), m_options.allowCredentials(), m_actualRequest->httpMethod(), m_actualRequest->httpHeaderFields()))
>             preflightSuccess();
>@@ -335,7 +335,7 @@ void DocumentThreadableLoader::didFail(unsigned long identifier, const ResourceE
> 
> void DocumentThreadableLoader::preflightSuccess()
> {
>-    OwnPtr<ResourceRequest> actualRequest;
>+    std::unique_ptr<ResourceRequest> actualRequest;
>     actualRequest.swap(m_actualRequest);
> 
>     actualRequest->setHTTPOrigin(securityOrigin()->toString());
>diff --git a/Source/WebCore/loader/DocumentThreadableLoader.h b/Source/WebCore/loader/DocumentThreadableLoader.h
>index fabcd3d26d5a8c868a6993c8d48cefbabdbd7be9..2a3f2f0271656fec2d083e6ad56b32d16b40d333 100644
>--- a/Source/WebCore/loader/DocumentThreadableLoader.h
>+++ b/Source/WebCore/loader/DocumentThreadableLoader.h
>@@ -36,7 +36,6 @@
> #include "FrameLoaderTypes.h"
> #include "ThreadableLoader.h"
> #include <wtf/Forward.h>
>-#include <wtf/OwnPtr.h>
> #include <wtf/PassRefPtr.h>
> #include <wtf/RefCounted.h>
> #include <wtf/RefPtr.h>
>@@ -106,7 +105,7 @@ namespace WebCore {
>         bool m_sameOriginRequest;
>         bool m_simpleRequest;
>         bool m_async;
>-        OwnPtr<ResourceRequest> m_actualRequest;  // non-null during Access Control preflight checks
>+        std::unique_ptr<ResourceRequest> m_actualRequest; // non-null during Access Control preflight checks
>     };
> 
> } // namespace WebCore
>diff --git a/Source/WebCore/loader/SubresourceLoader.cpp b/Source/WebCore/loader/SubresourceLoader.cpp
>index 206dd22b6332ac5f10962e86942b79089e3932ce..e15bdfbcd5fd3a2c547612efdbe846bfd6d7a0ea 100644
>--- a/Source/WebCore/loader/SubresourceLoader.cpp
>+++ b/Source/WebCore/loader/SubresourceLoader.cpp
>@@ -68,7 +68,7 @@ SubresourceLoader::SubresourceLoader(Frame* frame, CachedResource* resource, con
>     , m_resource(resource)
>     , m_loadingMultipartContent(false)
>     , m_state(Uninitialized)
>-    , m_requestCountTracker(adoptPtr(new RequestCountTracker(frame->document()->cachedResourceLoader(), resource)))
>+    , m_requestCountTracker(std::make_unique<RequestCountTracker>(frame->document()->cachedResourceLoader(), resource))
> {
> #ifndef NDEBUG
>     subresourceLoaderCounter.increment();
>@@ -225,7 +225,7 @@ void SubresourceLoader::didReceiveResponse(const ResourceResponse& response)
>         m_loadingMultipartContent = true;
> 
>         // We don't count multiParts in a CachedResourceLoader's request count
>-        m_requestCountTracker.clear();
>+        m_requestCountTracker = nullptr;
>         if (!m_resource->isImage()) {
>             cancel();
>             return;
>@@ -380,7 +380,7 @@ void SubresourceLoader::notifyDone()
>     if (reachedTerminalState())
>         return;
> 
>-    m_requestCountTracker.clear();
>+    m_requestCountTracker = nullptr;
> #if PLATFORM(IOS)
>     m_documentLoader->cachedResourceLoader().loadDone(m_resource, m_state != CancelledWhileInitializing);
> #else
>diff --git a/Source/WebCore/loader/SubresourceLoader.h b/Source/WebCore/loader/SubresourceLoader.h
>index 303ffd9abd7b1bb4d092d18019d6b1115e914289..c21f1293426a795be4c14d1bd0e21996770579f9 100644
>--- a/Source/WebCore/loader/SubresourceLoader.h
>+++ b/Source/WebCore/loader/SubresourceLoader.h
>@@ -121,7 +121,7 @@ private:
>     CachedResource* m_resource;
>     bool m_loadingMultipartContent;
>     SubresourceLoaderState m_state;
>-    OwnPtr<RequestCountTracker> m_requestCountTracker;
>+    std::unique_ptr<RequestCountTracker> m_requestCountTracker;
> };
> 
> }
>diff --git a/Source/WebCore/loader/TextTrackLoader.h b/Source/WebCore/loader/TextTrackLoader.h
>index 187ffbefb28e4bcc3f830ce09393c28e539d4a49..82754dfe5010298a1f7044d2d5cdbff490ae171b 100644
>--- a/Source/WebCore/loader/TextTrackLoader.h
>+++ b/Source/WebCore/loader/TextTrackLoader.h
>@@ -33,7 +33,6 @@
> #include "Timer.h"
> #include "WebVTTParser.h"
> #include <memory>
>-#include <wtf/OwnPtr.h>
> 
> namespace WebCore {
> 
>diff --git a/Source/WebCore/loader/appcache/ApplicationCacheHost.h b/Source/WebCore/loader/appcache/ApplicationCacheHost.h
>index 86c17740e9b54c093128183fdb014a0ec12cf7df..3ddf6941513ceefa8593f6e93cd1be8082daec3c 100644
>--- a/Source/WebCore/loader/appcache/ApplicationCacheHost.h
>+++ b/Source/WebCore/loader/appcache/ApplicationCacheHost.h
>@@ -33,7 +33,6 @@
> 
> #include "URL.h"
> #include <wtf/Deque.h>
>-#include <wtf/OwnPtr.h>
> #include <wtf/PassRefPtr.h>
> #include <wtf/RefPtr.h>
> #include <wtf/Vector.h>
>diff --git a/Source/WebCore/loader/appcache/ApplicationCacheStorage.cpp b/Source/WebCore/loader/appcache/ApplicationCacheStorage.cpp
>index f079a3bd60f35174a8dad88f16fb1136d628d3ea..ebb23acc36ee619d1852d177ef5fcc85a38d6012 100644
>--- a/Source/WebCore/loader/appcache/ApplicationCacheStorage.cpp
>+++ b/Source/WebCore/loader/appcache/ApplicationCacheStorage.cpp
>@@ -1359,7 +1359,7 @@ bool ApplicationCacheStorage::storeCopyOfCache(const String& cacheDirectory, App
>     }
>     
>     // Now create a new cache group.
>-    OwnPtr<ApplicationCacheGroup> groupCopy(adoptPtr(new ApplicationCacheGroup(cache->group()->manifestURL(), true)));
>+    std::unique_ptr<ApplicationCacheGroup> groupCopy(std::make_unique<ApplicationCacheGroup>(cache->group()->manifestURL(), true));
>     
>     groupCopy->setNewestCache(cacheCopy);
>     
>diff --git a/Source/WebCore/loader/archive/cf/LegacyWebArchive.cpp b/Source/WebCore/loader/archive/cf/LegacyWebArchive.cpp
>index 972346a0083ba6e44e3e24824ceaa34d0a371e24..5c94069bbd26b3e47194a741f8d465c1a1768099 100644
>--- a/Source/WebCore/loader/archive/cf/LegacyWebArchive.cpp
>+++ b/Source/WebCore/loader/archive/cf/LegacyWebArchive.cpp
>@@ -437,12 +437,12 @@ PassRefPtr<LegacyWebArchive> LegacyWebArchive::create(Node* node, std::function<
>     // If the page was loaded with javascript enabled, we don't want to archive <noscript> tags
>     // In practice we don't actually know whether scripting was enabled when the page was originally loaded
>     // but we can approximate that by checking if scripting is enabled right now.
>-    OwnPtr<Vector<QualifiedName>> tagNamesToFilter;
>+    std::unique_ptr<Vector<QualifiedName>> tagNamesToFilter;
>     if (frame->page() && frame->page()->settings().isScriptEnabled()) {
>-        tagNamesToFilter = adoptPtr(new Vector<QualifiedName>);
>+        tagNamesToFilter = std::make_unique<Vector<QualifiedName>>();
>         tagNamesToFilter->append(HTMLNames::noscriptTag);
>     }
>-        
>+
>     Vector<Node*> nodeList;
>     String markupString = createMarkup(*node, IncludeNode, &nodeList, DoNotResolveURLs, tagNamesToFilter.get());
>     Node::NodeType nodeType = node->nodeType();
>diff --git a/Source/WebCore/loader/cache/CachedResourceLoader.cpp b/Source/WebCore/loader/cache/CachedResourceLoader.cpp
>index a70c8625203048c740547d0dd87e05f9fabce23c..ab590df3d26d510115390272b3d542b30af96744 100644
>--- a/Source/WebCore/loader/cache/CachedResourceLoader.cpp
>+++ b/Source/WebCore/loader/cache/CachedResourceLoader.cpp
>@@ -877,7 +877,7 @@ void CachedResourceLoader::requestPreload(CachedResource::Type type, CachedResou
>     resource->increasePreloadCount();
> 
>     if (!m_preloads)
>-        m_preloads = adoptPtr(new ListHashSet<CachedResource*>);
>+        m_preloads = std::make_unique<ListHashSet<CachedResource*>>();
>     m_preloads->add(resource.get());
> 
> #if PRELOAD_DEBUG
>@@ -923,7 +923,7 @@ void CachedResourceLoader::clearPreloads()
>         if (!deleted && res->preloadResult() == CachedResource::PreloadNotReferenced)
>             memoryCache()->remove(res);
>     }
>-    m_preloads.clear();
>+    m_preloads = nullptr;
> }
> 
> void CachedResourceLoader::clearPendingPreloads()
>@@ -969,7 +969,7 @@ void CachedResourceLoader::printPreloadStats()
>         
>         res->decreasePreloadCount();
>     }
>-    m_preloads.clear();
>+    m_preloads = nullptr;
>     
>     if (scripts)
>         printf("SCRIPTS: %d (%d hits, hit rate %d%%)\n", scripts, scripts - scriptMisses, (scripts - scriptMisses) * 100 / scripts);
>diff --git a/Source/WebCore/loader/cache/CachedResourceLoader.h b/Source/WebCore/loader/cache/CachedResourceLoader.h
>index 66a812dd029f62f0872d641b0316a66a5faf3c5c..e4ed2feb9d4ab8c9a527edb5d96a492718120bf7 100644
>--- a/Source/WebCore/loader/cache/CachedResourceLoader.h
>+++ b/Source/WebCore/loader/cache/CachedResourceLoader.h
>@@ -165,7 +165,7 @@ private:
>     
>     int m_requestCount;
>     
>-    OwnPtr<ListHashSet<CachedResource*>> m_preloads;
>+    std::unique_ptr<ListHashSet<CachedResource*>> m_preloads;
>     struct PendingPreload {
>         CachedResource::Type m_type;
>         CachedResourceRequest m_request;
>diff --git a/Source/WebCore/loader/cache/MemoryCache.h b/Source/WebCore/loader/cache/MemoryCache.h
>index 1bbad1f4287a4350ef0fe995d3c20e1cc3b36deb..86840f625f4108abb3bedb54f4911cc0fa53f961 100644
>--- a/Source/WebCore/loader/cache/MemoryCache.h
>+++ b/Source/WebCore/loader/cache/MemoryCache.h
>@@ -68,7 +68,7 @@ public:
> 
> #if ENABLE(CACHE_PARTITIONING)
>     typedef HashMap<String, CachedResource*> CachedResourceItem;
>-    typedef HashMap<String, OwnPtr<CachedResourceItem>> CachedResourceMap;
>+    typedef HashMap<String, std::unique_ptr<CachedResourceItem>> CachedResourceMap;
> #else
>     typedef HashMap<String, CachedResource*> CachedResourceMap;
> #endif
>diff --git a/Source/WebCore/loader/icon/IconDatabase.cpp b/Source/WebCore/loader/icon/IconDatabase.cpp
>index abcb0cb345afe6096d0f408148b3f925afbedc8c..f10c413d34a1815d426b70dfa0eba1a9cde666ae 100644
>--- a/Source/WebCore/loader/icon/IconDatabase.cpp
>+++ b/Source/WebCore/loader/icon/IconDatabase.cpp
>@@ -1758,20 +1758,20 @@ void IconDatabase::deleteAllPreparedStatements()
> {
>     ASSERT_ICON_SYNC_THREAD();
>     
>-    m_setIconIDForPageURLStatement.clear();
>-    m_removePageURLStatement.clear();
>-    m_getIconIDForIconURLStatement.clear();
>-    m_getImageDataForIconURLStatement.clear();
>-    m_addIconToIconInfoStatement.clear();
>-    m_addIconToIconDataStatement.clear();
>-    m_getImageDataStatement.clear();
>-    m_deletePageURLsForIconURLStatement.clear();
>-    m_deleteIconFromIconInfoStatement.clear();
>-    m_deleteIconFromIconDataStatement.clear();
>-    m_updateIconInfoStatement.clear();
>-    m_updateIconDataStatement.clear();
>-    m_setIconInfoStatement.clear();
>-    m_setIconDataStatement.clear();
>+    m_setIconIDForPageURLStatement = nullptr;
>+    m_removePageURLStatement = nullptr;
>+    m_getIconIDForIconURLStatement = nullptr;
>+    m_getImageDataForIconURLStatement = nullptr;
>+    m_addIconToIconInfoStatement = nullptr;
>+    m_addIconToIconDataStatement = nullptr;
>+    m_getImageDataStatement = nullptr;
>+    m_deletePageURLsForIconURLStatement = nullptr;
>+    m_deleteIconFromIconInfoStatement = nullptr;
>+    m_deleteIconFromIconDataStatement = nullptr;
>+    m_updateIconInfoStatement = nullptr;
>+    m_updateIconDataStatement  = nullptr;
>+    m_setIconInfoStatement = nullptr;
>+    m_setIconDataStatement = nullptr;
> }
> 
> void* IconDatabase::cleanupSyncThread()
>@@ -1810,15 +1810,15 @@ void* IconDatabase::cleanupSyncThread()
> // 1 - If the SQLDatabase& argument is different, the statement must be destroyed and remade.  This happens when the user
> //     switches to and from private browsing
> // 2 - Lazy construction of the Statement in the first place, in case we've never made this query before
>-inline void readySQLiteStatement(OwnPtr<SQLiteStatement>& statement, SQLiteDatabase& db, const String& str)
>+inline void readySQLiteStatement(std::unique_ptr<SQLiteStatement>& statement, SQLiteDatabase& db, const String& str)
> {
>     if (statement && (statement->database() != &db || statement->isExpired())) {
>         if (statement->isExpired())
>             LOG(IconDatabase, "SQLiteStatement associated with %s is expired", str.ascii().data());
>-        statement.clear();
>+        statement = nullptr;
>     }
>     if (!statement) {
>-        statement = adoptPtr(new SQLiteStatement(db, str));
>+        statement = std::make_unique<SQLiteStatement>(db, str);
>         if (statement->prepare() != SQLResultOk)
>             LOG_ERROR("Preparing statement %s failed", str.ascii().data());
>     }
>diff --git a/Source/WebCore/loader/icon/IconDatabase.h b/Source/WebCore/loader/icon/IconDatabase.h
>index 4e0490431eeb9991d5e5fc0a31b81a94882382bd..c7d92f2dd921f7e39498f85742470075bfae03be 100644
>--- a/Source/WebCore/loader/icon/IconDatabase.h
>+++ b/Source/WebCore/loader/icon/IconDatabase.h
>@@ -231,20 +231,20 @@ private:
>     
>     SQLiteDatabase m_syncDB;
>     
>-    OwnPtr<SQLiteStatement> m_setIconIDForPageURLStatement;
>-    OwnPtr<SQLiteStatement> m_removePageURLStatement;
>-    OwnPtr<SQLiteStatement> m_getIconIDForIconURLStatement;
>-    OwnPtr<SQLiteStatement> m_getImageDataForIconURLStatement;
>-    OwnPtr<SQLiteStatement> m_addIconToIconInfoStatement;
>-    OwnPtr<SQLiteStatement> m_addIconToIconDataStatement;
>-    OwnPtr<SQLiteStatement> m_getImageDataStatement;
>-    OwnPtr<SQLiteStatement> m_deletePageURLsForIconURLStatement;
>-    OwnPtr<SQLiteStatement> m_deleteIconFromIconInfoStatement;
>-    OwnPtr<SQLiteStatement> m_deleteIconFromIconDataStatement;
>-    OwnPtr<SQLiteStatement> m_updateIconInfoStatement;
>-    OwnPtr<SQLiteStatement> m_updateIconDataStatement;
>-    OwnPtr<SQLiteStatement> m_setIconInfoStatement;
>-    OwnPtr<SQLiteStatement> m_setIconDataStatement;
>+    std::unique_ptr<SQLiteStatement> m_setIconIDForPageURLStatement;
>+    std::unique_ptr<SQLiteStatement> m_removePageURLStatement;
>+    std::unique_ptr<SQLiteStatement> m_getIconIDForIconURLStatement;
>+    std::unique_ptr<SQLiteStatement> m_getImageDataForIconURLStatement;
>+    std::unique_ptr<SQLiteStatement> m_addIconToIconInfoStatement;
>+    std::unique_ptr<SQLiteStatement> m_addIconToIconDataStatement;
>+    std::unique_ptr<SQLiteStatement> m_getImageDataStatement;
>+    std::unique_ptr<SQLiteStatement> m_deletePageURLsForIconURLStatement;
>+    std::unique_ptr<SQLiteStatement> m_deleteIconFromIconInfoStatement;
>+    std::unique_ptr<SQLiteStatement> m_deleteIconFromIconDataStatement;
>+    std::unique_ptr<SQLiteStatement> m_updateIconInfoStatement;
>+    std::unique_ptr<SQLiteStatement> m_updateIconDataStatement;
>+    std::unique_ptr<SQLiteStatement> m_setIconInfoStatement;
>+    std::unique_ptr<SQLiteStatement> m_setIconDataStatement;
> };
> 
> #endif // !ENABLE(ICONDATABASE)
>diff --git a/Source/WebCore/loader/icon/IconRecord.h b/Source/WebCore/loader/icon/IconRecord.h
>index bc99b6518186b35f5be572ba42a418367dbf4b25..d40859393482ccf89f31b2e520920acebff91e21 100644
>--- a/Source/WebCore/loader/icon/IconRecord.h
>+++ b/Source/WebCore/loader/icon/IconRecord.h
>@@ -32,7 +32,6 @@
> #include "PageURLRecord.h"
> #include "SharedBuffer.h"
> #include <wtf/HashSet.h>
>-#include <wtf/OwnPtr.h>
> #include <wtf/RefCounted.h>
> #include <wtf/text/StringHash.h>
> #include <wtf/text/WTFString.h>

Actions: View | Formatted Diff | Diff

Attachments on bug 138646: 241403 | 241414 | 241453 | 241459 | 241523 | 241528 | 241538 | 241551 | 241553 | 241692