Title: [121437] trunk/Source/WebCore
- Revision
- 121437
- Author
- [email protected]
- Date
- 2012-06-28 09:43:23 -0700 (Thu, 28 Jun 2012)
Log Message
[GTK] [WK2] Memory leak in ResourceHandleSoup.cpp
https://bugs.webkit.org/show_bug.cgi?id=90168
Patch by Sudarsana Nagineni <[email protected]> on 2012-06-28
Reviewed by Martin Robinson.
Fixed a memory leak in WebCoreSynchronousLoader by using adoptGRef
instead of just getting new reference of GMainLoop.
No new tests. No change in behavior.
* platform/network/soup/ResourceHandleSoup.cpp:
(WebCore::WebCoreSynchronousLoader::WebCoreSynchronousLoader):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (121436 => 121437)
--- trunk/Source/WebCore/ChangeLog 2012-06-28 16:35:33 UTC (rev 121436)
+++ trunk/Source/WebCore/ChangeLog 2012-06-28 16:43:23 UTC (rev 121437)
@@ -1,3 +1,18 @@
+2012-06-28 Sudarsana Nagineni <[email protected]>
+
+ [GTK] [WK2] Memory leak in ResourceHandleSoup.cpp
+ https://bugs.webkit.org/show_bug.cgi?id=90168
+
+ Reviewed by Martin Robinson.
+
+ Fixed a memory leak in WebCoreSynchronousLoader by using adoptGRef
+ instead of just getting new reference of GMainLoop.
+
+ No new tests. No change in behavior.
+
+ * platform/network/soup/ResourceHandleSoup.cpp:
+ (WebCore::WebCoreSynchronousLoader::WebCoreSynchronousLoader):
+
2012-06-27 Dana Jansens <[email protected]>
[chromium] Do not multiply bounds by contentsScale in TiledLayerChromium and CanvasLayerTextureUpdater
Modified: trunk/Source/WebCore/platform/network/soup/ResourceHandleSoup.cpp (121436 => 121437)
--- trunk/Source/WebCore/platform/network/soup/ResourceHandleSoup.cpp 2012-06-28 16:35:33 UTC (rev 121436)
+++ trunk/Source/WebCore/platform/network/soup/ResourceHandleSoup.cpp 2012-06-28 16:43:23 UTC (rev 121437)
@@ -88,7 +88,7 @@
loadingSynchronousRequest = true;
GRefPtr<GMainContext> innerMainContext = adoptGRef(g_main_context_new());
g_main_context_push_thread_default(innerMainContext.get());
- m_mainLoop = g_main_loop_new(innerMainContext.get(), false);
+ m_mainLoop = adoptGRef(g_main_loop_new(innerMainContext.get(), false));
adjustMaxConnections(1);
}
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes