Title: [145328] trunk/Source/WebKit2
Revision
145328
Author
[email protected]
Date
2013-03-10 10:41:13 -0700 (Sun, 10 Mar 2013)

Log Message

WebPluginSiteDataManager should use OwnPtr.
<http://webkit.org/b/111940>

Reviewed by Anders Carlsson.

* UIProcess/Plugins/WebPluginSiteDataManager.cpp:
(WebKit::WebPluginSiteDataManager::invalidate):
(WebKit::WebPluginSiteDataManager::getSitesWithData):
(WebKit::WebPluginSiteDataManager::clearSiteData):
(WebKit::WebPluginSiteDataManager::didGetSitesWithDataForAllPlugins):
(WebKit::WebPluginSiteDataManager::didClearSiteDataForAllPlugins):
* UIProcess/Plugins/WebPluginSiteDataManager.h:
(WebPluginSiteDataManager):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (145327 => 145328)


--- trunk/Source/WebKit2/ChangeLog	2013-03-10 16:46:16 UTC (rev 145327)
+++ trunk/Source/WebKit2/ChangeLog	2013-03-10 17:41:13 UTC (rev 145328)
@@ -1,3 +1,19 @@
+2013-03-10  Andreas Kling  <[email protected]>
+
+        WebPluginSiteDataManager should use OwnPtr.
+        <http://webkit.org/b/111940>
+
+        Reviewed by Anders Carlsson.
+
+        * UIProcess/Plugins/WebPluginSiteDataManager.cpp:
+        (WebKit::WebPluginSiteDataManager::invalidate):
+        (WebKit::WebPluginSiteDataManager::getSitesWithData):
+        (WebKit::WebPluginSiteDataManager::clearSiteData):
+        (WebKit::WebPluginSiteDataManager::didGetSitesWithDataForAllPlugins):
+        (WebKit::WebPluginSiteDataManager::didClearSiteDataForAllPlugins):
+        * UIProcess/Plugins/WebPluginSiteDataManager.h:
+        (WebPluginSiteDataManager):
+
 2013-03-09  Manuel Rego Casasnovas  <[email protected]>
 
         [GTK][WK2] Fix compilation warning missing-field-initializers in webkit-2.0 stable branch

Modified: trunk/Source/WebKit2/UIProcess/Plugins/WebPluginSiteDataManager.cpp (145327 => 145328)


--- trunk/Source/WebKit2/UIProcess/Plugins/WebPluginSiteDataManager.cpp	2013-03-10 16:46:16 UTC (rev 145327)
+++ trunk/Source/WebKit2/UIProcess/Plugins/WebPluginSiteDataManager.cpp	2013-03-10 17:41:13 UTC (rev 145328)
@@ -139,9 +139,7 @@
     invalidateCallbackMap(m_arrayCallbacks);
 
 #if ENABLE(PLUGIN_PROCESS)
-    deleteAllValues(m_pendingGetSitesWithData);
     m_pendingGetSitesWithData.clear();
-    deleteAllValues(m_pendingClearSiteData);
     m_pendingClearSiteData.clear();
 #endif
 }
@@ -162,7 +160,7 @@
     ASSERT(!m_pendingGetSitesWithData.contains(callbackID));
 
     GetSitesWithDataState* state = new GetSitesWithDataState(this, callbackID);
-    m_pendingGetSitesWithData.set(callbackID, state);
+    m_pendingGetSitesWithData.set(callbackID, adoptPtr(state));
     state->getSitesWithDataForNextPlugin();
 #else
     Vector<PluginModuleInfo> plugins = m_webContext->pluginInfoStore().plugins();
@@ -222,7 +220,7 @@
     ASSERT(!m_pendingClearSiteData.contains(callbackID));
 
     ClearSiteDataState* state = new ClearSiteDataState(this, sitesVector, flags, maxAgeInSeconds, callbackID);
-    m_pendingClearSiteData.set(callbackID, state);
+    m_pendingClearSiteData.set(callbackID, adoptPtr(state));
     state->clearSiteDataForNextPlugin();
 #else
     Vector<PluginModuleInfo> plugins = m_webContext->pluginInfoStore().plugins();
@@ -267,7 +265,7 @@
 
 void WebPluginSiteDataManager::didGetSitesWithDataForAllPlugins(const Vector<String>& sites, uint64_t callbackID)
 {
-    OwnPtr<GetSitesWithDataState> state = adoptPtr(m_pendingGetSitesWithData.take(callbackID));
+    OwnPtr<GetSitesWithDataState> state = m_pendingGetSitesWithData.take(callbackID);
     ASSERT(state);
 
     didGetSitesWithData(sites, callbackID);
@@ -283,7 +281,7 @@
 
 void WebPluginSiteDataManager::didClearSiteDataForAllPlugins(uint64_t callbackID)
 {
-    OwnPtr<ClearSiteDataState> state = adoptPtr(m_pendingClearSiteData.take(callbackID));
+    OwnPtr<ClearSiteDataState> state = m_pendingClearSiteData.take(callbackID);
     ASSERT(state);
 
     didClearSiteData(callbackID);

Modified: trunk/Source/WebKit2/UIProcess/Plugins/WebPluginSiteDataManager.h (145327 => 145328)


--- trunk/Source/WebKit2/UIProcess/Plugins/WebPluginSiteDataManager.h	2013-03-10 16:46:16 UTC (rev 145327)
+++ trunk/Source/WebKit2/UIProcess/Plugins/WebPluginSiteDataManager.h	2013-03-10 17:41:13 UTC (rev 145328)
@@ -78,10 +78,10 @@
     void didClearSiteDataForAllPlugins(uint64_t callbackID);
 
     class GetSitesWithDataState;
-    HashMap<uint64_t, GetSitesWithDataState*> m_pendingGetSitesWithData;
+    HashMap<uint64_t, OwnPtr<GetSitesWithDataState> > m_pendingGetSitesWithData;
 
     class ClearSiteDataState;
-    HashMap<uint64_t, ClearSiteDataState*> m_pendingClearSiteData;
+    HashMap<uint64_t, OwnPtr<ClearSiteDataState> > m_pendingClearSiteData;
 #endif
 };
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to